Consider UNC paths as NetworkShare context
ClosedPublic

Authored by broulik on Sep 16 2019, 8:24 AM.

Details

Summary

Location runner uses KUriFilter which translates "\\foo\bar" to smb://foo/bar" but we don't want to go through all of the URI filtering in KRunner itself.

Test Plan

Searched for "\\foo\bar" and got it as "NetworkShare" with higher score in location runner than before.

Diff Detail

Repository
R308 KRunner
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Sep 16 2019, 8:24 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptSep 16 2019, 8:24 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik requested review of this revision.Sep 16 2019, 8:24 AM
ngraham accepted this revision.Sep 16 2019, 3:17 PM
ngraham added a subscriber: ngraham.

Works great, looks great.

This revision is now accepted and ready to land.Sep 16 2019, 3:17 PM
This revision was automatically updated to reflect the committed changes.