Accept local paths
Needs ReviewPublic

Authored by broulik on Jun 22 2019, 4:01 PM.

Details

Reviewers
hein
Group Reviewers
Plasma
Summary

Alternate approach to D16087

BUG: 401579

Test Plan

5.12
No longer get broken recent folders without icon in Dolphin

Diff Detail

Repository
R119 Plasma Desktop
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Jun 22 2019, 4:01 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 22 2019, 4:01 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jun 22 2019, 4:01 PM
ngraham edited the summary of this revision. (Show Details)Jun 22 2019, 4:19 PM
apol added a subscriber: apol.Jun 23 2019, 9:26 AM
apol added inline comments.
applets/kicker/plugin/actionlist.cpp
256
if (QDir::isAbsolutePath(resource)) {
    resource = QUrl::fromLocalFile(resource);
}
meven added a subscriber: meven.Fri, Jul 26, 8:47 AM
meven added inline comments.
applets/kicker/plugin/actionlist.cpp
256

This should be done in applets/kicker/plugin/kastatsfavoritesmodel.cpp as well line 162 and line 275.

I have a patch ready @broulik I can take over if you'd like.