Bug 363337 - Recent docs/history shortcuts in kicker/kickoff fail when clicked with "Could not find any application or handler..." error
Summary: Recent docs/history shortcuts in kicker/kickoff fail when clicked with "Coul...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Application Menu (Kicker) (show other bugs)
Version: 5.12.4
Platform: unspecified Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL: http://i.imgur.com/twoUpUZ.png
Keywords:
: 363671 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-05-20 17:26 UTC by Rik Mills
Modified: 2018-07-11 08:47 UTC (History)
10 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.12.7


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Rik Mills 2016-05-20 17:26:01 UTC
Recent docs/history shortcuts in kicker/kickoff fail when clicked with  "Could not find any application or handler..." for the file; error in a plasma error popup (see url link).

Seems to be associated with running plasma with latest frameworks 5.22. Testing with previous frameworks 5.21 with same plasma version does not produce the error. 

Reproducible: Always

Steps to Reproduce:
1. Install plasma 5.6.4 or above with frameworks v 5.22
2. Activate kicker/kickoff menu and navigate to history/recent document lists 
3. Click desired recent item of ANY type

Actual Results:  
Error "Could not find any application or handler..." for the file, in a plasma error popup (see url link)

Expected Results:  
Should launch application for file association, and load recent document.

Tested on.

With bug occurring:
- kubuntu plasma 5.6.4 backport testing packages with frameworks 5.22
- kde-neon user and dev editions with plasma 5.6.4 and 5.6.90, and frameworks 5.22
- archlinux plasma 5.6.4, with frameworks 5.22 from their 'testing' repo

Without bug occurring:
- archlinux plasma 5.6.4 with frameworks 5.21
Comment 1 Rik Mills 2016-05-21 14:28:57 UTC
Reverting this diff and rebuilding kio seems to fix the problem. 

https://quickgit.kde.org/?p=kio.git&a=blobdiff&h=ec1b4bdeecf2cff58765ae5edb3d8ec6a4a7c972&hp=b9b232dc80e9fa9b47f65203aa72b52280124fc1&hb=2e6980e9ba3b91798dad0db9f9d78317b2d7e4a2&f=src%2Fwidgets%2Fkrun.cpp

Clicking on recent/history items now opens them in the associated application.

Not sure if it breaks anything else intended with the commit

https://quickgit.kde.org/?p=kio.git&a=commitdiff&h=2e6980e9ba3b91798dad0db9f9d78317b2d7e4a2

or if that should be retained or modified, and problem fixed from another angle.
Comment 2 Eike Hein 2016-05-21 15:46:15 UTC
CC'ing David who authored the commit.
Comment 3 Kai Uwe Broulik 2016-05-21 15:58:19 UTC
Now I know why "Send mail to" in krunner is broken, KProtocolManager::supportsProtocol("mailto") returns false because... well, this thing no longer exists.
Comment 4 David Faure 2016-05-21 21:31:49 UTC
Try porting to KProtocolInfo::isKnownProtocol, which will return true for mailto as soon as one mail client is installed.
Comment 5 Rik Mills 2016-05-22 14:06:33 UTC
Previously, Chakra with plasma 5.6.3 and frameworks 5.21 was unaffected, as should be the case if a FW 5.21 problem.

After: 

https://chakraos.org/news/index.php?/archives/178-Plasma-5.6.4,-Applications-16.04.1-and-Frameworks-5.22.0-available-in-Chakra.html

upgrade to plasma 5.6.4 and frameworks 5.22, 

can confirm Chakra also shows this bug.
Comment 6 Eike Hein 2016-05-22 15:27:24 UTC
Kicker doesn't do any protocol stuff though, it just hands KService URLs to KRun.
Comment 7 David Faure 2016-05-23 21:13:20 UTC
Found it. The URL being given to KRun is wrong, it has no scheme at all. The error message would show file:/// in front if it did. This is a QString->QUrl conversion issue.

With no scheme, KRun can't check that the protocol supports reading, that's why the error message is new, but the bug is in whichever code is assembling this URL.
I'll add asserts to catch this earlier in the future ;-)
Comment 8 Eike Hein 2016-05-24 09:51:30 UTC
Thanks David!
Comment 9 Eike Hein 2016-05-24 09:51:42 UTC
Git commit 8a9772d8673a58583317b4906a9352d6bf44a8e2 by Eike Hein.
Committed on 24/05/2016 at 09:50.
Pushed by hein into branch 'Plasma/5.6'.

Fix opening recent docs on newer KF5.

The KActivities model can return scheme-less local paths, and execution
didn't go through the code path already handling this.

M  +1    -1    applets/kicker/plugin/recentusagemodel.cpp

http://commits.kde.org/plasma-desktop/8a9772d8673a58583317b4906a9352d6bf44a8e2
Comment 10 David Faure 2016-05-24 19:49:27 UTC
Git commit 9c1c48161416a2dd70fdb1dfeb92ed0b7dde75d9 by David Faure.
Committed on 24/05/2016 at 19:48.
Pushed by dfaure into branch 'master'.

Give clearer error message when KRun(URL) is given a URL without scheme.

M  +4    -3    src/widgets/krun.cpp

http://commits.kde.org/kio/9c1c48161416a2dd70fdb1dfeb92ed0b7dde75d9
Comment 11 Kai Uwe Broulik 2016-05-29 17:09:02 UTC
*** Bug 363671 has been marked as a duplicate of this bug. ***
Comment 12 Daniel Noga 2018-06-09 09:34:36 UTC
I found the same bug on Kubuntu 18.04 and openSUSE Leap 15.


1) Clean install openSUSE Leap 15.0 or Kubuntu 18.04 with KDE desktop on VirtualBox
1.1) On openSUSE in left bottom corner right click on Start menu (SUSE logo) -> select Alternatives -> choose Application launcher (second in list). On Kubuntu it is default.
2) In Application launcher -> tab Computer -> select Documents (search:/documents)
3) error appears: "could not find any application or handler for search:/documents"
Comment 13 Eike Hein 2018-06-11 06:25:58 UTC
This is not the same bug, I think you're describing an openSUSE bug. Kickoff as shipped does not have a "Documents" item on the Computer tab, which means openSUSE added it, and if it uses an URL scheme the system doesn't understand, it's probably missing a plugin package for it, which they should install. Please report downstream to openSUSE. :)
Comment 14 Fabian Vogt 2018-06-11 08:14:07 UTC
This is added by kio itself, so not a downstream issue:

https://github.com/KDE/kio/blob/ed714cb876bcaa56377c9e5a777533e83b8cedc8/src/filewidgets/kfileplacesmodel.cpp#L289
Comment 15 Eike Hein 2018-06-11 08:18:32 UTC
Thanks Fabian!

Shouldn't this be baloosearch:/?
Comment 16 Luca Beltrame 2018-06-11 08:48:57 UTC
The URLs from the places in the Dolphin panel for search use indeed search:/ but interestingly, this only works from Dolphin itself. Typing "search:/" elsewhere (including the Dolphin URL bar) yields "Unsupported protocol".

Is Dolphin doing something extra there? CCing Elvis as IIRC he has some experience on Dolphin itself.
Comment 17 Daniel Noga 2018-07-10 13:38:54 UTC
Is it fixed by https://phabricator.kde.org/D13841 or that was different issue?
Comment 18 Fabian Vogt 2018-07-10 13:42:15 UTC
(In reply to Daniel Noga from comment #17)
> Is it fixed by https://phabricator.kde.org/D13841 or that was different
> issue?

It might be - can someone confirm?
Comment 19 Nate Graham 2018-07-10 19:27:18 UTC
The remaining search:/ issue was indeed fixed by https://phabricator.kde.org/D13841
Comment 20 Nate Graham 2018-07-10 20:45:55 UTC
Oops--*will* be fixed. :)
Comment 21 Kai Uwe Broulik 2018-07-11 08:47:35 UTC
Git commit 3fe808ef8e9763edd5c349f8556b7ee2c0645702 by Kai Uwe Broulik.
Committed on 11/07/2018 at 08:47.
Pushed by broulik into branch 'Plasma/5.12'.

[Kicker] Use KFilePlaces::convertedUrl for ComputerModel

Fixes timeline and search URLs in "Computer" tab
FIXED-IN: 5.12.7

Differential Revision: https://phabricator.kde.org/D14029

M  +1    -1    applets/kicker/plugin/computermodel.cpp

https://commits.kde.org/plasma-desktop/3fe808ef8e9763edd5c349f8556b7ee2c0645702