[kded kcm] Fix estimating dbusModuleName of kded plugins

Authored by kossebau on Jun 19 2018, 9:36 PM.

Description

[kded kcm] Fix estimating dbusModuleName of kded plugins

Summary:
The entry X-KDE-DBus-ModuleName has never been used by kded code,
at least is not using it these days, so the check for that metadata entry
and using its value is broken.

Instead Kded::loadModule() used KPluginMetaData::pluginId() to set the
module name, which then is used by KDEDModule for its D-Bus object path.

The branch for handling an empty name is removed as that fallback is
covered by KPluginMetaData already.

Test Plan:
KCM now lists all kded plugins, no longer complains about modules it could not
relate.

Reviewers: dfaure, davidedmundson

Reviewed By: dfaure

Subscribers: plasma-devel

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D13623

Details

Committed
kossebauJun 19 2018, 10:31 PM
Reviewer
dfaure
Differential Revision
D13623: [kded kcm] Fix estimating dbusModuleName of kded plugins
Parents
R119:24803dc9dc40: Fix tooltip woes
Branches
Unknown
Tags
Unknown