Review Request 124111: Support overwrite prompt of unrar v5

[Raphael Kubo da Costa]

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


Looking at it again, I can see a few more improvements that can be made:

* There's no need to have `checkForFileName()` as a separate function. Its use cannot be dissociated from `handleFileExistsMessage()`, so you can get rid of `m_storedFileName`, move the contents of `checkForFileName()` to the beginning of `handleFileExistsMessage()` and make the relationship between this part of the code much more clear.
* Also to having the need of a separate member variable (`m_storedFileName`), you can change `checkForFileExists()` and move the log message to the `handle` method instead.

And now that the frameworks branch has been merged into master, I guess there's no need to worry about it not working on KDE4 anymore.


kerfuffle/cliinterface.cpp (line 663)
<https://git.reviewboard.kde.org/r/124111/#comment56793>

    This isn't being used anymore.


- Raphael Kubo da Costa


On June 17, 2015, 12:35 p.m., Ragnar Thomsen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124111/
> -----------------------------------------------------------
> 
> (Updated June 17, 2015, 12:35 p.m.)
> 
> 
> Review request for KDE Utils, Elvis Angelaccio and Raphael Kubo da Costa.
> 
> 
> Bugs: 349131
>     http://bugs.kde.org/show_bug.cgi?id=349131
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> The overwrite prompt of unrar v5 changed compared to v3 and v4. This causes Ark to hang when extracting a rar archive with unrar v5 when a destination file exists. In unrar v5 the overwrite prompt looks like:
> 
>     Would you like to replace the existing file file1.txt
>         32 bytes, modified on 15-05-2015 14:26
>     with a new one
>         32 bytes, modified on 15-05-2015 14:26
>         
> and in unrar v3 and v4:
> 
>     file1.txt already exists. Overwrite it ?
> 
> Ark currently only supports the v3/v4 prompt. This patch adds support for the v5 prompt by defining a QStringList containing regexps for both prompts and iterating through the QStringList in CliInterface::checkForFileExistsMessage().
> 
> 
> Diffs
> -----
> 
>   kerfuffle/cliinterface.h e93d443 
>   kerfuffle/cliinterface.cpp 2ce6a06 
>   plugins/cli7zplugin/cliplugin.cpp 1c1389f 
>   plugins/clilhaplugin/cliplugin.cpp 3e2b338 
>   plugins/clirarplugin/cliplugin.cpp 489abd6 
>   plugins/clizipplugin/cliplugin.cpp c927581 
> 
> Diff: https://git.reviewboard.kde.org/r/124111/diff/
> 
> 
> Testing
> -------
> 
> Extracting rar archives using both v4 and v5 unrar works as expected, also when destination exists. Extraction using clizip and cli7z also works as expected.
> 
> 
> Thanks,
> 
> Ragnar Thomsen
> 
>

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