Review Request 122897: Fix 7z zip not listing files after adding on new created archive.

[Giuseppe Silletti]

> On Mar. 17, 2015, 9:58 a.m., Raphael Kubo da Costa wrote:
> > Thanks a lot for the patch and sorry for being absent.
> > Does this mean Ark is currently unable to add/remove files from 7z archives, or is it that the file list is not updated after that is done? Are you using the new p7zip (9.38.1)?

It is that the file list is not updated after adding a file. I'm using the last version of p7zip.


- Giuseppe


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


On Mar. 10, 2015, 10:17 p.m., Giuseppe Silletti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122897/
> -----------------------------------------------------------
> 
> (Updated Mar. 10, 2015, 10:17 p.m.)
> 
> 
> Review request for KDE Utils and Raphael Kubo da Costa.
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> Ark doesn't access to a new 7z archive properly. I've made the substituteListVariables function in cliinterface.cpp virtual, and then added it in 7z cliplugin.cpp. In this file i've overridden  substituteListVariables, and then i've appended the .7z extension to the filename() function. So now ark can find the right path of 7z archives.
> 
> 
> Diffs
> -----
> 
>   kerfuffle/cliinterface.h b4d79ed 
>   plugins/cli7zplugin/cliplugin.h 9f122e4 
>   plugins/cli7zplugin/cliplugin.cpp 500f121 
> 
> Diff: https://git.reviewboard.kde.org/r/122897/diff/
> 
> 
> Testing
> -------
> 
> I've added some files in a new archive and it works.
> 
> 
> Thanks,
> 
> Giuseppe Silletti
> 
>

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