Differential D6112

update kcm statistics in kcmshell too
ClosedPublic
Actions

Authored by mart on Jun 6 2017, 10:39 AM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R126:e003e86628be: update kcm statistics in kcmshell too
Summary

also when opening modules with kcmshell (used a lot with krunner)
the statistics need to be updated to keep the list on systemsettings startup
up to date and have complete usage data of kcm modules

Test Plan

opened systemsettings and a module with kcmsell, the list gets updated
on the fly

Diff Detail

Repository
R126 KDE CLI Utilities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mart created this revision.Jun 6 2017, 10:39 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJun 6 2017, 10:39 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson added a subscriber: davidedmundson.Jun 6 2017, 10:45 AM
davidedmundson added inline comments.
kcmshell/main.cpp
298

?

mart updated this revision to Diff 15207.Jun 6 2017, 10:59 AM

remove dead code

mart marked an inline comment as done.Jun 6 2017, 10:59 AM
mart added inline comments.
kcmshell/main.cpp
298

eww, sorry :)

davidedmundson accepted this revision.Jun 6 2017, 11:12 AM
This revision is now accepted and ready to land.Jun 6 2017, 11:12 AM
Closed by commit R126:e003e86628be: update kcm statistics in kcmshell too (authored by mart). · Explain WhyJun 6 2017, 11:49 AM
This revision was automatically updated to reflect the committed changes.
mart marked an inline comment as done.

Revision Contents
Changeset List

PathPackages
MCMakeLists.txt (1 line)
Mkcmshell/CMakeLists.txt (1 line)
Mkcmshell/main.cpp (13 lines)

Diff 15212

View Options

CMakeLists.txt

Loading...
View Options

kcmshell/CMakeLists.txt

Loading...
View Options

kcmshell/main.cpp

Loading...