D18563: Don't allow '/' in new directory's name

Nathaniel Graham noreply at phabricator.kde.org
Sun Jan 27 17:41:47 GMT 2019


ngraham requested changes to this revision.
ngraham added reviewers: Frameworks, Dolphin.
ngraham added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> knewfilemenu.cpp:907
>      if (name.contains(QLatin1Char('/'))) {
> +        // Allow creating directory tree on Windows
>          // If the name contains any slashes, use mkpath so that a/b/c works.

If Windows allows slashes in the filename, shouldn't they just be a part of the filename and not create a directory tree? If so, then we don't even need this `#ifdef` condition at all; `KIO::mkdir(url);` will take care of the behavior on each individual platform

REPOSITORY
  R241 KIO

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

To: shubham, ngraham, #frameworks, #dolphin
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190127/ba2166a1/attachment.html>


More information about the Kde-frameworks-devel mailing list