D17617: Display error instead of silently failing when asked to create folder that already exists

David Faure noreply at phabricator.kde.org
Sun Dec 16 20:48:29 GMT 2018


dfaure added a comment.


  What might be easier is:
  
  - if the user entered something without '/', use KIO::mkdir()   (which will fail with ERR_DIR_ALREADY_EXIST if the dir already exists)
  - if the user entered something with '/', use KIO::mkpath() as the code currently does.
  
  Much simpler and faster than an (async and racy) existence check before hand.

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D17617

To: ngraham, #dolphin, elvisangelaccio
Cc: dfaure, emateli, elvisangelaccio, Codezela, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181216/b74f74a2/attachment.htm>


More information about the kfm-devel mailing list