Review Request 125293: Fix drag'n'drop extraction of CLI plugins

[Ragnar Thomsen]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125293/
-----------------------------------------------------------

Review request for KDE Utils, Elvis Angelaccio and Raphael Kubo da Costa.


Bugs: 208384
    http://bugs.kde.org/show_bug.cgi?id=208384


Repository: ark


Description
-------

This fixes drag'n'drop extraction of selected entries from Ark to e.g. Dolphin. Previously, the entries would be extracted with full path, which is not what the user expects and not how libarchive handles tar-based archives. This was due to the CLI plugins not supporting root nodes (i.e. removing a part of the path). This is now circumvented by extracting to a QTemporaryDir, removing the root node from the path, and finally moving the files to their final destination. The moving to final destination is done in a new member function in CliInterface (moveToFinalDest). This function checks if the destination file exists and prompts the user for action if that is the case.

Things to consider:
QTemporaryDir creates a directory in /tmp. What happens/should happen if this partition fills up?


Diffs
-----

  kerfuffle/cliinterface.h 052851a 
  kerfuffle/cliinterface.cpp 1e936a6 

Diff: https://git.reviewboard.kde.org/r/125293/diff/


Testing
-------

Extracting files from zip, 7z and rar archives by selecting and dragging them to Dolphin.


Thanks,

Ragnar Thomsen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20150918/9544c0d1/attachment.html>