<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/120204/">https://git.reviewboard.kde.org/r/120204/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Any chance to get you to rebase this patch against the frameworks branch? I would really like for this to get merged for Applications 15.08, which will be KF5-based.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">The rebasing will not be trivial, since AddDialog is now a QDialog with a KFileWidget, and the dialog in MainWindow::newArchive() is now a QFileDialog.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Also your new CreateDialog will have to be a QDialog+KFileWidget, due to QFileDialog not supporting extensions.</p></pre>
<br />
<p>- Ragnar Thomsen</p>
<br />
<p>On February 8th, 2015, 12:05 a.m. CET, Elvis Angelaccio wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDE Utils and Raphael Kubo da Costa.</div>
<div>By Elvis Angelaccio.</div>
<p style="color: grey;"><i>Updated Feb. 8, 2015, 12:05 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=253694">253694</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ark
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch allows the users to create password-protected archives using Ark.
The supported plugins are <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">zip</code>, <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">7z</code> and <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">rar</code>.
The input fields for the passwords are displayed only when the relevent mime filters are selected in the <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">AddDialog</code> window.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Create <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">.zip</code>, <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">.7z</code> and <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">.rar</code> archives using the Ark <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">AddDialog</code> dialog (right click on an entry in Dolphin -> Compress -> Compress to...).
Those archives are created as expected and their entries are marked as <em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Password protected</em>.
If the user clicks on those entries, the standard password dialog is prompted to the user.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plugins/cli7zplugin/kerfuffle_cli7z.desktop.cmake <span style="color: grey">(29b355af72b02d767e099a6e7a61778a70ae4476)</span></li>
<li>plugins/clirarplugin/cliplugin.cpp <span style="color: grey">(f5f7b184cb9f7ee16efd74968f2e86dc83668b4d)</span></li>
<li>plugins/clirarplugin/kerfuffle_clirar.desktop.cmake <span style="color: grey">(3091d9d384952fbe27a43049c66c2ee998017957)</span></li>
<li>plugins/clizipplugin/cliplugin.cpp <span style="color: grey">(1be2a10f722865c23fba6df5c25993742ce38d69)</span></li>
<li>plugins/clizipplugin/kerfuffle_clizip.desktop.cmake <span style="color: grey">(32ceced7b913c4d4a0d2007b4b3ea7b4ec850e35)</span></li>
<li>app/mainwindow.cpp <span style="color: grey">(aee823174a0b731e125381be09181938cdb6dd7d)</span></li>
<li>kerfuffle/CMakeLists.txt <span style="color: grey">(8c56b4f46090c924f0848ad229b6a3779af1f1f3)</span></li>
<li>kerfuffle/adddialog.h <span style="color: grey">(7c9f03e916da3a410482be2b45ff30923dfc1f9f)</span></li>
<li>kerfuffle/adddialog.cpp <span style="color: grey">(2950ee6e22c90c770bb3b9128243f9786989ee52)</span></li>
<li>kerfuffle/adddialog.ui <span style="color: grey">(a1c595b89202b1f2b44dc4f9c2faf68f5f655eb1)</span></li>
<li>kerfuffle/addtoarchive.h <span style="color: grey">(fa4608e0c0df9ea9cec3aa9e7323fb53dd1b12d3)</span></li>
<li>kerfuffle/addtoarchive.cpp <span style="color: grey">(d13e0958468e3bb1aa85606e4fd4d3cb8ef6d3c0)</span></li>
<li>kerfuffle/archive.h <span style="color: grey">(515788f978c1d29146c25dcb57a9dad2d46371f7)</span></li>
<li>kerfuffle/archive.cpp <span style="color: grey">(c2c5b18528def63405a8703475b1c592ca945973)</span></li>
<li>kerfuffle/archiveinterface.h <span style="color: grey">(801318b7842d78a0de54700ffdb43a945f513fbf)</span></li>
<li>kerfuffle/archiveinterface.cpp <span style="color: grey">(3319de074225ccf8cb0fe54ffadca5882cf29342)</span></li>
<li>kerfuffle/cliinterface.h <span style="color: grey">(b4d79ed2f3ffbbbe28f7087f1fbff36de5138f36)</span></li>
<li>kerfuffle/cliinterface.cpp <span style="color: grey">(eab1c6a3200543989fd501e4d54e3e0b06b48999)</span></li>
<li>kerfuffle/createdialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kerfuffle/createdialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kerfuffle/createdialog.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>kerfuffle/kerfufflePlugin.desktop <span style="color: grey">(3999d24f414ede1bb377f47b8c7d46dc44459b7d)</span></li>
<li>part/archivemodel.h <span style="color: grey">(7f8c527fc331778e0ec74dce047eba5c56507499)</span></li>
<li>part/archivemodel.cpp <span style="color: grey">(43262689ad989e3a9233e279db6dcd532084a031)</span></li>
<li>part/part.cpp <span style="color: grey">(09fe1cbfcc7f4345fe12932055dcb041f50abb7b)</span></li>
<li>plugins/cli7zplugin/cliplugin.cpp <span style="color: grey">(500f121de97ef395110658a6ca495298bc797b46)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/120204/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/01/13/3bed4e8a-477e-4675-9920-1390e89d28ad__passwd-dialog-0.png">passwd-dialog-0.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/01/13/21c65550-60da-4984-838c-cf4e6579d894__passwd-dialog-1.png">passwd-dialog-1.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/01/13/f0a7764e-ae16-4e8c-81a7-d99de68a7a61__passwd-dialog-2.png">passwd-dialog-2.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/01/13/d894c54d-bb58-429c-b291-66bfef03aed6__passwd-dialog-3.png">passwd-dialog-3.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/01/13/2347d4d2-65b4-484c-80d1-e5c0f8f461c1__passwd-dialog-4.png">passwd-dialog-4.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/02/07/65c72521-3374-4156-bb74-a856fb2e8615__show-password.png">show-password.png</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>