<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, hein.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik 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/D9838" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>First connect to the signal, then register the object, and only at last register the service.<br />
Otherwise it might expose a not yet fully operational service on DBus.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>I made an app that listens for Unity service to be registered and send its badge info once it's there. Without this it would fire the signal before we connected to it resulting in the badge not showing on Plasma startup, with this patch it works.<br />
iirc this is also what Thiago always said you should do when he did the threaded dbus changes in Qt 5.6</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/D9838" rel="noreferrer">https://phabricator.kde.org/D9838</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>