Rework and cleanup of VFS
ClosedPublic

Authored by abika on Oct 11 2016, 4:02 PM.

Details

Summary

Removed queue manager; Part of cleaning VFS (WIP)

Maybe re-add it in future. But KDE has its one queue/nofify dialog for KIO jobs.

Removed KIOJobWrapper. Part of cleaning VFS (WIP)

Calculating space on non-local filesystem does not work because its running in a different thread.
It did now work before either.

Removed VirtualCopyJob. Part of cleaning VFS (WIP)

Support for copy/move to virt:/ filesystem is limited (as before).
And btw: moving to virt doesn't make any sense and is not possible. Currently it is the same as copying

Merged normal_vfs and ftp_vfs into default_vfs. Part of cleaning VFS (WIP)

This code is compiling but there are a lot TODOs, not everything is working and performance is terrible.
More progress later...

Rework and merge of move() and copy() in panelfunc. Part of cleaning VFS (WIP)

+ KrVFSHandler is singleton now and can create copy jobs.
Job signals should be handled in VFS

Removed PreservingCopyJob. Part of cleaning VFS (WIP)

Preserving option was never used, KIO always preserves attributes when copying.

Cleaning up vfs. Part of cleaning VFS (WIP)

Minor code cleanup. Part of cleaning VFS (WIP)

Moved all classes related to archive handling to new folder. Part of cleaning VFS (WIP)

Removed unused mime type detection switch. Part of cleaning VFS (WIP)

Simplify usage of vfs. Part of cleaning VFS (WIP)

Compiling, bugs are cleaned later...

Rework of entire vfs class and subclasses. Part of cleaning VFS

Big diff. This needs extensive testing.

Added TODO comments. Part of cleaning VFS

Diff Detail

Repository
R167 Krusader
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
abika updated this revision to Diff 7313.Oct 11 2016, 4:02 PM
abika retitled this revision from to Rework and cleanup of VFS.
abika updated this object.
abika edited the test plan for this revision. (Show Details)
abika added a reviewer: Krusader.
abika updated this revision to Diff 7314.Oct 11 2016, 4:11 PM

Rebased

Closed by commit R167:637c34e2d6d9: Removed queue manager; Part of cleaning VFS (WIP) (authored by Alexander Bikadorov <bikaejkb@mailbox.tu-berlin.de>). · Explain WhyOct 11 2016, 4:43 PM
This revision was automatically updated to reflect the committed changes.
abika reopened this revision.Oct 11 2016, 4:49 PM

No, Phabricator, no!

Closed by commit R167:53298d89189e: Added TODO comments. Part of cleaning VFS (authored by Alexander Bikadorov <bikaejkb@mailbox.tu-berlin.de>, committed by Alexander Bikadorov <alex.bikadorov@kdemail.net>). · Explain WhyOct 17 2016, 2:47 PM
This revision was automatically updated to reflect the committed changes.