Review Request: KOpenWithDialog: Quote paths selected in the file dialog

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103602/#review10148
-----------------------------------------------------------


This review has been submitted with commit cfc52e240ae5a15f049f578c50b9c58eadb3203a by Ingomar Wesp to branch KDE/4.8.

- Commit Hook


On Jan. 1, 2012, 9:49 p.m., Ingomar Wesp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103602/
> -----------------------------------------------------------
> 
> (Updated Jan. 1, 2012, 9:49 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> KOpenWithDialog expects the input in its line edit to be shell-quoted. Currently, however, when the user picks a path via the provided file dialog, whitespace and other special characters in the path are inserted verbatim. This leads to an error when the user tries to select an executable whose path contains such characters.
> 
> This tiny patch addresses the issue by adding a private slot that runs all paths coming from the file dialog through KShell::quoteArg().
> 
> I believe this solution is better than the one I originally proposed in the linked bug report, since it avoids the ambiguity of having to guess whether whitespace belongs to the path or an argument to the executable.
> 
> 
> This addresses bug 281952.
>     http://bugs.kde.org/show_bug.cgi?id=281952
> 
> 
> Diffs
> -----
> 
>   kio/kfile/kopenwithdialog.h 86c02ab 
>   kio/kfile/kopenwithdialog.cpp b5ebbbf 
> 
> Diff: http://git.reviewboard.kde.org/r/103602/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ingomar Wesp
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120127/8d02223e/attachment.htm>