Review Request: ftp server URL throws "invalid protocol" - Bug: 209031

Mark markg85 at
Thu Aug 26 23:58:01 BST 2010

This is an automatically generated e-mail. To reply, visit:

(Updated 2010-08-26 22:58:00.704925)

Review request for Dolphin, kdelibs and David Faure.


And so there is a new diff that uses Dawit Alemayehu suggestion.
Is it good enough now to be committed? (i don't have the commit power so please feel free to commit)


Read the bug report for the.. bug..

As for the fix. I had a hard time finding the right place to implement this since all was using const KUrl thus not editable... When searching deeper i ended up in making the fix in either KUrlNavigator or in KUrl itself. I decided to go for KUrlNavigator for no particular reason... I could just as well have tried it in KUrl. I hope this KUrlNavigator was the right place (either way, please do explain where is the right place and why the other one isn't the right place). 

This addresses bug 209031.

Diffs (updated)

  /tags/KDE/4.5.0/kdelibs/kfile/kurlnavigator.cpp 1168303 



I did one simple tests on it:
typing "" in the url bar and pressing enter. It changed the url to "" and the ftp opened just perfectly. I browsed a bit on it to see if it stays working fins and it seems to do just that.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list