<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: dfaure, chinmoyr.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />broulik 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/D19585">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Currently, <tt style="background: #ebebeb; font-size: 13px;">slotReport</tt> is called all the way at the end when in <tt style="background: #ebebeb; font-size: 13px;">STATE_SETTING_DIR_ATTRIBUTES</tt>. However, <tt style="background: #ebebeb; font-size: 13px;">slotReport</tt> does nothing in this state.<br />
Instead, move it to before changing away from <tt style="background: #ebebeb; font-size: 13px;">STATE_COPYING_FILES</tt> so that progress is correctly reported for a final time.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Opened Dolphin, Ctrl+C on a file, Ctrl+V it into the same folder, let it sit at the "Override?" prompt until the job view in Plasma showed up. Chose a different filename and continued.</p>

<p>Before, the job would finish without having reported any progress as the report timer never fired since that one file was copied super quickly. With this patch, the final progress is correct. The URL isn't updated in case of a rename but I bet it just needs a <tt style="background: #ebebeb; font-size: 13px;">m_bURLDirty</tt> call somewhere in the conflict handler</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/D19585">https://phabricator.kde.org/D19585</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/core/copyjob.cpp</div></div></div><br /><div><strong>To: </strong>broulik, dfaure, chinmoyr<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>