----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106475/#review19779 -----------------------------------------------------------
This review has been submitted with commit 8e9bfe11f54e3fa5000a86773b21bdcdcff44bd5 by Alex Fiestas to branch KDE/4.9. - Commit Hook On Sept. 17, 2012, 10:39 a.m., Alex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106475/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2012, 10:39 a.m.) > > > Review request for kdelibs and David Faure. > > > Description > ------- > > This patch addresses the following bug: > When an user clicks in a KUrlRequestUrl("smb://") to open the dialog, the > KFIleDialog will be opened with the "current working dir" instead of smb:// > > Debugging the issue, I reached KFileWidget::getStartUrl, in that method there > is a check that looks if the directory is not empty, in case it is the > directory (startDir) is discarded in favor of some magic code that looks for > recent folders blablabla. > > this code apparently was added to support the case where an url is composed > only by a fileName, for example when Konqueror Save as (commit 4d3933d4). So > the solution I thought is to check if the schema is not empty, in case both > directory and schema are empty the KUrl will be discarded as well, instead if > any of them is not empty the directory will be used. > > > Diffs > ----- > > kfile/kfilewidget.cpp 7069a49 > > Diff: http://git.reviewboard.kde.org/r/106475/diff/ > > > Testing > ------- > > Used Konqueror (as indicated in the commit) with both khtml and webkit. > Tested samba-mounter > Tested bluedevil > > Everything seems to work. > > > Thanks, > > Alex Fiestas > >