<table><tr><td style="">marten created this revision.<br />marten added a reviewer: Plasma.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />marten requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13360">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is now the preferred way of providing plugin metadata.  It's not just a cosmetic change because there is an actual problem with the original:  the desktop file kded_touchpad.desktop sets X-KDE-DBus-ModuleName=touchpad which is presumably intended to register the KDED module under that name on DBus;  all of the clients access it with the interface path "/modules/touchpad".  However, this key appears to be being ignored, and the module is registered under the path "/modules/kded_touchpad" which can be confirmed with qdbusviewer.  This incorrect path means that touchpad control via the Plasma applet does not work, and 'kcmshell5 kcmkded' does not show the touchpad module status correctly.</p>

<p>This change updates the KDED plugin to use JSON metadata.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built plasma-desktop with this change.  Observed correct registration of the module name on DBus, correct operation of the Plasma touchpad applet, and of "kcmshell5 kcmkded" and "kcmshell5 kcm_touchpad".</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13360">https://phabricator.kde.org/D13360</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcms/touchpad/src/CMakeLists.txt<br />
kcms/touchpad/src/kcm/xlib/touchpadconfigxlib.cpp<br />
kcms/touchpad/src/kded/kded.cpp<br />
kcms/touchpad/src/kded/kdedactions.cpp<br />
kcms/touchpad/src/plugins.cpp<br />
kcms/touchpad/src/plugins.h</div></div></div><br /><div><strong>To: </strong>marten, Plasma<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>