<table><tr><td style="">dfaure created this revision.<br />dfaure added reviewers: elvisangelaccio, broulik.<br />Herald added a subscriber: kde-utils-devel.<br />Herald added a project: Ark.<br />dfaure requested review of this revision.
</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/D18256">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Using the part URL at closing time is really dangerous: if the user used<br />
"save as" (e.g. for an embedded PDF), we'll be deleting the file that<br />
the user just saved!!</p>

<p>BUG: 402529</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>launch ARK and open a ZIP file containing a PDF, click on the PDF,<br />
an embedded viewer appears. From there, use "Save As..." to save the PDF<br />
somewhere on your local disk. Close the embedded viewer, close ark.<br />
It doesn't delete the saved file anymore.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R36 Ark</div></div></div><br /><div><strong>BRANCH</strong><div><div>Applications/18.12</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18256">https://phabricator.kde.org/D18256</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>part/arkviewer.cpp<br />
part/arkviewer.h</div></div></div><br /><div><strong>To: </strong>dfaure, elvisangelaccio, broulik<br /><strong>Cc: </strong>kde-utils-devel, Ark, tctara<br /></div>