Review Request 119505: Instance our onw QFileDialog instead of using getExistingDirectoryUrl

Àlex Fiestas afiestas at kde.org
Sun Aug 3 00:47:56 UTC 2014


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

(Updated ago. 3, 2014, 12:47 a.m.)


Review request for KDE Frameworks.


Repository: kio


Description
-------

getExistingDirectoryUrl only works for local files so we have to implement it on our own for now.

As for Qt, QFileDialog::getExistingDirectoryUrl calls QUrl.toLocalFile, if we remove that, further down the code path _qt_get_directory is called which checks that the file exists but since Qt is not able to talk all our kios (for example smb) it will return false.
So at the moment we are forced to implement it ourselves.


Diffs (updated)
-----

  src/widgets/kurlrequester.cpp f2837ad 

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


Testing
-------


Thanks,

Àlex Fiestas

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140803/11905c36/attachment.html>


More information about the Kde-frameworks-devel mailing list