Review Request: Allow "~" as folder name (bug: 117473)

[Mark]

On Sun, Aug 29, 2010 at 9:45 PM, Mark <markg85 at gmail.com> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5186/
>   Review request for kdelibs.
> By Mark.
>
> *Updated 2010-08-29 19:45:14.152762*
> Changes
>
> Updated patch to keep the current behavior only add a special case when just "~" is provided.
>
>   Description
>
> This patch fixes https://bugs.kde.org/show_bug.cgi?id=117473 and allows "~" to be the folder name.
> this is also consistent with new file names that could already have the "~" name.
>
>   Testing
>
> Tested the patch and seems to work fine.
>
>   *Bugs: * 117473 <https://bugs.kde.org/show_bug.cgi?id=117473>
> Diffs (updated)
>
>    - trunk/KDE/kdelibs/kfile/knewfilemenu.cpp (1169591)
>
> View Diff <http://reviewboard.kde.org/r/5186/diff/>
>

What has to be done with this bug?
Commit it as posted by me as a "temp" fix? (will add a // TODO ... above
it).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100908/069d70ee/attachment.htm>