<table><tr><td style="">rthomsen updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jb7jusxuyqcmgpl/" rel="noreferrer">(Show Details)</a><br />rthomsen updated the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-admlgk4i564eak2/" rel="noreferrer">(Show Details)</a><br />rthomsen updated this revision to Diff 8118.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D3142" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">This diff changes `ExtractionFailedPatterns` from a `QStringList` to a `QHash<QString,QVariant>`</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">This diff removes the `ExtractionFailedPatterns` regexp in `CliProperties`</span>. <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">This way we show custom</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">Instead we take care of the</span> error <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">messages to</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">handling in</span> the <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">user</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">individual plugins</span> by <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">passing an error message associated with the `QRegularExpression`</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">adding a pure virtual `readExtractLine()` method to `CliInterface` (similar</span> to <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">`CliInterface`.<br />
<br />
A</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">`readListLine()`).</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">new pattern `ListFailedPatterns` was added which is used for `ListJob`</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">This way we can provide custom error messages to `Part`</span>, <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">before it just</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">instead of the generic `Extraction failed due to an unexpected error` (this message was also</span> used <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">`ExtractionFailedPatterns`.<br />
</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">for ListJob).</span></div></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div> - Open Zip archive using Xz compression with clizip.<br />
<span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);"> - Open AES-encrypted Zip archive with clizip.<br />
</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rARK Ark</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D3142?vs=7621&id=8118" rel="noreferrer">https://phabricator.kde.org/D3142?vs=7621&id=8118</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3142" rel="noreferrer">https://phabricator.kde.org/D3142</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kerfuffle/cliinterface.cpp<br />
kerfuffle/cliinterface.h<br />
kerfuffle/cliproperties.cpp<br />
kerfuffle/cliproperties.h<br />
plugins/cli7zplugin/cliplugin.cpp<br />
plugins/cli7zplugin/cliplugin.h<br />
plugins/clirarplugin/cliplugin.cpp<br />
plugins/clirarplugin/cliplugin.h<br />
plugins/cliunarchiverplugin/cliplugin.cpp<br />
plugins/cliunarchiverplugin/cliplugin.h<br />
plugins/clizipplugin/cliplugin.cpp<br />
plugins/clizipplugin/cliplugin.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>rthomsen, elvisangelaccio<br /><strong>Cc: </strong>kde-utils-devel, tctara<br /></div>