broulik abandoned this revision.
REPOSITORY
R288 KJobWidgets
REVISION DETAIL
https://phabricator.kde.org/D14290
To: broulik, dfaure
Cc: elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns
dfaure added a comment.
Why is this called with an invalid URL?
The bug report says "copy/move a big file to another device/partition." --
surely the destination should be valid? It sounds like this patch is masking a
real bug...
REPOSITORY
R288 KJobWidgets
REVISION DETAIL
dfaure requested changes to this revision.
This revision now requires changes to proceed.
REPOSITORY
R288 KJobWidgets
REVISION DETAIL
https://phabricator.kde.org/D14290
To: broulik, dfaure
Cc: elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns
elvisangelaccio added a comment.
+1
REPOSITORY
R288 KJobWidgets
REVISION DETAIL
https://phabricator.kde.org/D14290
To: broulik, dfaure
Cc: elvisangelaccio, kde-frameworks-devel, michaelh, ngraham, bruns
broulik created this revision.
broulik added a reviewer: dfaure.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: kde-frameworks-devel.
broulik requested review of this revision.
REVISION SUMMARY
It checks whether the file is not in `TEMP` but it