Differential D3076

Fix random order in extension menus
ClosedPublic
Actions

Authored by antonanikin on Oct 17 2016, 2:52 AM.

Details

Reviewers
kfunk
Group Reviewers
KDevelop
Commits
R33:be5cd5b57ca2: Fix random order in extension menus
R32:be5cd5b57ca2: Fix random order in extension menus
Summary

This patch fixes random order of extension menu items between different runs of KDevelop.

Old behavior leads for example to random reordering of "Analyze With" submenu:

  1. "Cppcheck" actions, "Vera++" actions - first run
  2. "Vera++" actions, "Cppcheck" actions - some other run.
Test Plan

Tested with master branch

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
antonanikin updated this revision to Diff 7429.Oct 17 2016, 2:52 AM
antonanikin retitled this revision from to Fix random order in extension menus.
antonanikin updated this object.
antonanikin edited the test plan for this revision. (Show Details)
antonanikin added a reviewer: KDevelop.
antonanikin set the repository for this revision to R33 KDevPlatform.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptOct 17 2016, 2:52 AM
kfunk accepted this revision.Oct 17 2016, 6:28 AM
kfunk added a reviewer: kfunk.
kfunk added a subscriber: kfunk.

Looks good to me.

Please add a comment (why we're sorting this) to the source code before pushing.

To 5.0 branch.

This revision is now accepted and ready to land.Oct 17 2016, 6:28 AM
antonanikin updated this revision to Diff 7450.Oct 17 2016, 8:17 AM
antonanikin edited edge metadata.
Closed by commit R33:be5cd5b57ca2: Fix random order in extension menus (authored by antonanikin). · Explain WhyOct 17 2016, 8:18 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mshell/plugincontroller.cpp (31 lines)

Diff 7451

View Options

shell/plugincontroller.cpp

Loading...