<table><tr><td style="">marten created this revision.<br />Restricted Application added a project: Frameworks.
</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/D7841" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>As described in bug <a href="https://bugs.kde.org/show_bug.cgi?id=384500" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=384500</a>, there appears to be a problem when the receiving application of a file needs a temporary copy to be made (because of %F/%f in its desktop file).  The kded file watching module is told to watch the file too early, before the ioslave has even started to copy it.  This means that when the copy is complete it will receive a dirty signal (on file creation) and the user will immediately be prompted to re-upload.</p>

<p>This change moves the file watch operation to after the file copy job is complete.  At this point the file is in a stable state and hence the dirty signal and the prompt will not happen, unless the file really is subsequently modified.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built and installed kio with this change, checked remote file opening as per the bug.  Verified that the kded module is not told to watch the file until after the copy is complete, and that there is no re-upload prompt unless the file is actually modified.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7841" rel="noreferrer">https://phabricator.kde.org/D7841</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kioexec/main.cpp</div></div></div><br /><div><strong>To: </strong>marten, Frameworks<br /><strong>Cc: </strong>elvisangelaccio, dfaure<br /></div>