[PATCH] Make man:foo, info:foo, etc. work again from KRunner

[Michael Pyne]

Hi all,

Attached is a patch to fix bug 221371 (man: and info: syntax no longer 
working) which is caused by a strengthening of the syntax checks done in 
RunnerContext in kdelibs/plasma. It used to be that near the end of the chain 
of checks if a context hadn't been defined that a KUrl would be constructed 
and made either a network path or a local path.

This was changed a few weeks ago to only assume a path if there was a slash or 
backslash, but this was done before the network check was made to.

This caused the context passed to the location runner to change from a 
NetworkContext to an UnknownContext which stopped any matches from being 
emitted (since these URLs don't match the possibilities allowed in 
LocationRunner for UnknownContext URLs).

The attached patch changes the ambiguity test to be performed after the 
network URL check to restore the old behavior while still not ambiguously 
assuming local paths.

Note that it was always possible to workaround with man:/foo and info:/foo, 
but the other style is frequently used as examples of "cool things you can do 
in KDE" and are valid URLs anyways so I'd like that syntax to also work. I'd 
also like to get this in before 4.4 so I would appreciate a prompt review if 
possible so I can contact release-team.

Thanks!

Regards,
 - Michael Pyne
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs-fix-runner-protocol.patch
Type: text/x-patch
Size: 1530 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20100109/fe8bba45/attachment.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/plasma-devel/attachments/20100109/fe8bba45/attachment.sig