Differential D13308

Filter out duplicate plugins
ClosedPublic
Actions

Authored by elvisangelaccio on Jun 3 2018, 5:59 PM.

Details

Reviewers
apol
Commits
R495:727ce79c815d: Filter out duplicate plugins
Summary

If purpose is installed both as distro package and in a custom prefix,
we don't want to show each plugin twice. This patch filters out
duplicate plugins based on their pluginId.

Test Plan
  • Apply D13124 and make sure each purpose plugin shows up only once in the 'Share' context menu.

Diff Detail

Repository
R495 Purpose Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
elvisangelaccio created this revision.Jun 3 2018, 5:59 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJun 3 2018, 5:59 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
elvisangelaccio requested review of this revision.Jun 3 2018, 5:59 PM
elvisangelaccio edited the summary of this revision. (Show Details)Jun 3 2018, 6:07 PM
apol accepted this revision.Jun 4 2018, 1:09 AM
This revision is now accepted and ready to land.Jun 4 2018, 1:09 AM
Closed by commit R495:727ce79c815d: Filter out duplicate plugins (authored by elvisangelaccio). · Explain WhyJun 4 2018, 8:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/alternativesmodel.cpp (13 lines)

Diff 35557

View Options

src/alternativesmodel.cpp

Loading...