[patch] KFileWidget: fix changing the current dir using url combo box

Jarosław Staniek js at iidea.pl
Fri Apr 11 16:04:29 BST 2008


for review:

Fix changing the current dir using the 'recent url' combo box.
Upon accepting new url in the 'directory' url combo box, append '/' if needed: 
the combo does not add it, but tokenize() expects it because uses 
KUrl::setFileName(), which would silently remove the last segment of the path.

I guess it was a crossplatform issue.
Without the patch, if you select /home/js/foo/bar from the url combo box, and 
click on file.txt, the resulting url would be /home/js/foo/file.txt, not the 
expected /home/js/foo/bar/file.txt.

-- 
regards / pozdrawiam, Jaroslaw Staniek
  Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
  Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi)
  KDE Libraries for MS Windows (http://windows.kde.org)
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: kfilewidget-urlcombo.patch
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080411/e833ae57/attachment.ksh>


More information about the kde-core-devel mailing list