<table><tr><td style="">jtamate updated this revision to Diff 40779.<br />jtamate marked 3 inline comments as done.<br />jtamate edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-2sltfyfgdkvnwxo/">(Show Details)</a><br />jtamate edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-d3eh65dvvohq4is/">(Show Details)</a><br />jtamate added a comment.
</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/D15180">View Revision</a></tr></table><br /><div><div><p>I'm not sure if I have to keep compatibility in the dbus calls, therefore the old one is still there.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">When a non KIO friendly program opens a non local file, the file is copied to an user temporary folder<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> by kioexec.<br />
Watch any creation or modification of that temporary file, because some programs delete the old file and create a new one when saving the file, When the file is modified, the user is asked if he/she wants to upload the modified file.<br />
Included two new dbus calls:<br />
 *to watch the temporary files, and keeping what application and how many times has been launched</span>.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">W</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> *to unw</span>atch <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">any creation or modification in that temporary folder.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the files, and to avoid deleting the files too early in case of immediate exit of the applications,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">for example because they ope</span>n th<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">at way</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">e file in a new tab</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">even with programs that delete</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">use a counter of</span> the<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> old</span> file<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> and create a new one when saving the file</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s one application has opened</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the user is asked if he wants to upload</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">and only delete the temporary files and directories when</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">modified file</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">application really exists</span>.<br />
<br />
Delete recursively the temporary directories and any file they could have<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> in the destructor</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">, so no temporary directories with backup files are left</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Tested logging out and in several times and opening <span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">two or three </span>remote files with xed and libreoffice.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D15180?vs=40737&id=40779">https://phabricator.kde.org/D15180?vs=40737&id=40779</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15180">https://phabricator.kde.org/D15180</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kioexec/kioexecd.cpp<br />
src/kioexec/kioexecd.h<br />
src/kioexec/main.cpp</div></div></div><br /><div><strong>To: </strong>jtamate, Frameworks, broulik, ngraham, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>