Bug 221371

Summary:	regression from 4.3.4: man: and info: syntax no longer working
Product:	[Plasma] krunner	Reporter:	Martin Koller <kollix>
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	mpyne
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Fix bug by checking for network addresses before using the more stringent path checks

Description Martin Koller 2010-01-05 12:24:11 UTC

Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

In 4.3.4 entering "man:ls" or "info:ls" opened the man or info page in konqueror.
In 4.3.85 (4.4. beta2) this no longer works and simply does nothing.

I noted that "man:/ls" or "info:/ls" works, but I was used to the shorter "man:ls" syntax, which is still working when entered directly into konqueror addressbar, so it's obviously a regression.

Comment 1 Michael Pyne 2010-01-10 00:38:45 UTC

man:foo URLs now come up as a UnknownContext for Plasma::RunnerContext::type() instead of NetworkContext (which seems to have been what would have worked before). Maybe something in the plasma libraries themselves changed to make that happen?

Comment 2 Michael Pyne 2010-01-10 01:00:10 UTC

Created attachment 39729 [details]
Fix bug by checking for network addresses before using the more stringent path checks

This is my proposed fix.

Comment 3 Michael Pyne 2010-01-12 05:19:05 UTC

SVN commit 1073382 by mpyne:

Backport fix for bug 221371 (man: and info:foo doesn't work in KRunner, 4.3 regression)
to KDE SC 4.4.

BUG:221371


 M  +7 -6      runnercontext.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1073382