<table><tr><td style="">sitter created this revision.<br />sitter added a reviewer: broulik.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />sitter 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/D21761">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>it was the only remaining user of that dialog helper.<br />
the connection between the two was fairly meh. the engine would invoke the<br />
kded module over dbus, the module would then process the request, find that<br />
only one desktop entry is specified, resulting in one action being found<br />
and then running that action on behalf of the engine. this is overly<br />
complicated for no additional gain. the engine is the sole user of this<br />
functionality AND it already had knowledge of the service system, so<br />
soliduiserver in this case doesn't actually add anything. it's just a more<br />
involved call chain.</p>

<p>the useful code of soliduiserver is now in the hotplug dataengine directly<br />
and the soliduiserver no longer supports the actions dialog. hotplug is now<br />
the sole authority for hotplugging.</p>

<p>soliduiserver:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">include cleanup</li>
<li class="remarkup-list-item">all classes no longer in use have been removed</li>
<li class="remarkup-list-item">kdelibs4support is no longer a link target</li>
<li class="remarkup-list-item">new link targets i18n + widgetsaddons (previously pulled in by kdelibs4support)</li>
</ul>

<p>hotplug:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">hotplugjob now directly executes service actions using classes imported from soliduiserver</li>
<li class="remarkup-list-item">no longer links qtdbus</li>
<li class="remarkup-list-item">new translation domain plasma_engine_hotplug (for i18n call in imported service classes)</li>
<li class="remarkup-list-item">new link against ki18n for that reason</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><p>running an action works same as before, internally it no longer relies on dbus/kded</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21761">https://phabricator.kde.org/D21761</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>dataengines/hotplug/CMakeLists.txt<br />
dataengines/hotplug/Messages.sh<br />
dataengines/hotplug/deviceaction.cpp<br />
dataengines/hotplug/deviceaction.h<br />
dataengines/hotplug/deviceserviceaction.cpp<br />
dataengines/hotplug/deviceserviceaction.h<br />
dataengines/hotplug/hotplugjob.cpp<br />
soliduiserver/CMakeLists.txt<br />
soliduiserver/deviceaction.cpp<br />
soliduiserver/deviceaction.h<br />
soliduiserver/deviceactionsdialog.cpp<br />
soliduiserver/deviceactionsdialog.h<br />
soliduiserver/deviceactionsdialogview.ui<br />
soliduiserver/devicenothingaction.cpp<br />
soliduiserver/devicenothingaction.h<br />
soliduiserver/deviceserviceaction.cpp<br />
soliduiserver/deviceserviceaction.h<br />
soliduiserver/soliduiserver.cpp<br />
soliduiserver/soliduiserver.h</div></div></div><br /><div><strong>To: </strong>sitter, broulik<br /><strong>Cc: </strong>plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>