D18563: Don't create directory tree when a new folder has a '/' in the name

Mark Gaiser noreply at phabricator.kde.org
Tue Jan 29 20:39:22 GMT 2019


markg added a comment.


  In D18563#401979 <https://phabricator.kde.org/D18563#401979>, @shubham wrote:
  
  > @ndavis UNIX does not allow / in any valid identifier name.
  >  Also looking at the bug report, it feels like people wanted to create a folder named "a/b/c" which UNEXPECTEDLY created directory tree. So in this, I have fix both the problems, first one which is not valid and the latter one which is unexpected behaviour.
  
  
  Define unexpected.
  
  People that don't know much about technology but merely happen to use KDE (so that would be the ones that also call for help if a console opens) the making of a folder names "a/b/c" causing the tree to me made is indeed unexpected.
  The more technical inclined people, the power users, will know that "/" is the directory separator and that typing "a/b/c" probably makes the whole tree. For those (and i bet those are still the majority of Plasma users) you would introduce a regression with this patch.
  
  I'm not saying it should go or stay. I only ever used it a couple of times and one of them being when debugging something years ago.
  I can imagine it to be useful for people who got used to it and now rely on it for their workflow.
  I can also imagine it not feeling intuitive.
  
  Fyi, mkdir "a/b/c" also doesn't work. mkdir -p "a/b/c" does!

REPOSITORY
  R241 KIO

BRANCH
  dir

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

To: shubham, ngraham, #frameworks, #dolphin, dfaure, elvisangelaccio, pino
Cc: markg, ndavis, dfaure, elvisangelaccio, pino, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190129/5e8f2a3a/attachment.htm>


More information about the kfm-devel mailing list