Review Request 112547: avoid creating jobs on urls with double slashes
Aleix Pol Gonzalez
aleixpol at kde.org
Tue Sep 17 11:14:32 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112547/
-----------------------------------------------------------
(Updated Sept. 17, 2013, 11:14 a.m.)
Status
------
This change has been discarded.
Review request for KDE Frameworks.
Description
-------
It happens that sometimes urls get a double shash instead of one, to trash:/foo becoming trash://foo (an hostname)
this happens because if url.path() == "/"
the line
itemURL.setPath(itemURL.path() + '/' + fileName);
just doubles the /
I'm not sure the patch is correct, if path() is guaranteed to always have a trailing slash it may be simpler (altough qurl documentation doesn't suggest so)
Diffs
-----
staging/kio/src/core/job.cpp e066ef4
Diff: http://git.reviewboard.kde.org/r/112547/diff/
Testing
-------
Thanks,
Marco Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130917/83571c36/attachment.html>
More information about the Kde-frameworks-devel
mailing list