Review Request 125339: Add an "Open" action

[Ragnar Thomsen]

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

(Updated Oct. 7, 2015, 10:18 p.m.)


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


Changes
-------

* Part::UpdateActions(): Improve readability by defining some variables and using them in the setEnabled() calls. Only enable `Open` and `Open with` if entry is previewable.
* Remove isPreviewable(): Move the code to UpdateActions() instead.
* Part::slotOpenExtractedEntry(): Check for writability of archive. If not writable, use QFile to set temporarily extracted file to readonly.
* Part::slotWatchedFileModified(): Don't display full path of file in KMessageBox. Instead only display relative path in archive.
* Part::slotAddFiles(): Add comments for hard-to-grasp code.


Bugs: 201162 and 208330
    http://bugs.kde.org/show_bug.cgi?id=201162
    http://bugs.kde.org/show_bug.cgi?id=208330


Repository: ark


Description
-------

Add an Open action, which opens an archive entry with the associated application. This was previously only possible through the "Open With" action, but this required several clicks. Currently, the Preview action is still the default, i.e. when the user clicks an archive entry the Preview action is called.

QSignalMapper is used to connect the signals from the Open, Open With and Preview actions to the same slot.

As for Preview, the file is extracted to a QTemporaryDir and then opened. The temporarily extracted file is monitored for changes using QFileSystemWatcher and on change a KMessageBox is opened to query the user if the archive should be updated with the modified file. If the user accepts slotAddFiles() is called.

The previously unimplemented path argument of slotAddFiles() is used to allow for updating a file in a subdirectory within the archive.

Things to consider:
1. Should we set some default keyboard shortcuts for "Open File" and "Open File With"? If yes, which?


Diffs (updated)
-----

  part/ark_part.rc da04d47 
  part/part.h 1d733c0 
  part/part.cpp 8854fdd 

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


Testing
-------

Tested with tar-based, zip, rar and 7z archives.


File Attachments
----------------

Context menu
  https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/b0b7c44d-2b40-4772-b8d0-a81f6895518a__open-action-1.png
Action menu
  https://git.reviewboard.kde.org/media/uploaded/files/2015/10/05/deab8bb1-722c-48b8-bce3-64190afc6a53__open-action-2.png


Thanks,

Ragnar Thomsen

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