dfaure added a comment.
This calls for unittests. UndoManager has unittests here in kio, but if this requires kio_desktop then the test needs to be where kio_desktop is. INLINE COMMENTS > copyjob.cpp:2095 > ++m_processedFiles; > - emit q->copyingDone(q, *m_currentStatSrc, > finalDestUrl(*m_currentStatSrc, dest), QDateTime() /*mtime unknown, and not > needed*/, true, true); > + // Bug 391606: finalDestUrl needs the resolved URL to slook up the > trash file name > + // which is why we're passing m_currentSrcURL instead of > m_currentStatSrc to it slook? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D11987 To: broulik, #frameworks, dfaure Cc: michaelh, ngraham, bruns
