----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125476/#review86204 -----------------------------------------------------------
Ship it! Looks good to me. - Aleix Pol Gonzalez On Oct. 1, 2015, 10:58 p.m., Sergio Martins wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125476/ > ----------------------------------------------------------- > > (Updated Oct. 1, 2015, 10:58 p.m.) > > > Review request for kdelibs and David Faure. > > > Repository: kio > > > Description > ------- > > - Passing openUrl.toString() is wrong, it should have been openUrl.path(), > since the former has a scheme, and QFileDialog will call > QUrl::fromLocalFile() on it. > - QFileDialog::getExistingDirectory() internally calls > QFileDialog::getExistingDirectoryUrl() and converts the result to a local > file string, which we then were re-converting to QUrl again, so instead just > call getExistingDirectoryUrl() directly. > > > Diffs > ----- > > src/widgets/kurlrequester.cpp 1b3bbdf > > Diff: https://git.reviewboard.kde.org/r/125476/diff/ > > > Testing > ------- > > > Thanks, > > Sergio Martins > >
