Review Request 120606: Properly parse URL in KUrlNavigator

Jan Grulich jgrulich at redhat.com
Thu Oct 16 13:56:12 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120606/
-----------------------------------------------------------

Review request for KDE Frameworks and David Faure.


Repository: kio


Description
-------

When using an URL with a scheme, like sftp://foo@bar.com/home/foo/, then KUrlNavigator doesn't properly parse it. At the beginning it tries to count the number of slashes from sftp://foo@bar.com, which is 2, then it tries to contruct buttons using names from particular sections separated by slashes, but when we use only QUrl::path() for URL above, we will always get only "/home/foo/" path and therefore we will have bigger index then the number of sections, which leads to have same URLs for all buttons. We need to parse sections from full URL including first two slashes.


Diffs
-----

  src/filewidgets/kurlnavigator.cpp e96d914 

Diff: https://git.reviewboard.kde.org/r/120606/diff/


Testing
-------

Tested in dolphin and works fine now.


File Attachments
----------------

Screenshot of the problem
  https://git.reviewboard.kde.org/media/uploaded/files/2014/10/16/eda6fb7e-a75e-4c28-ac87-5d16a39a366d__kurlnavigator.png


Thanks,

Jan Grulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20141016/991dccf3/attachment.html>


More information about the Kde-frameworks-devel mailing list