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