<table><tr><td style="">broulik created this revision.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</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/D6148" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>We used to always delete the backing .desktop file if we changed a property in a Link (to a document) hoping that setUrl would call populate. However, it won't do that if the URL didn't change, so if you only changed the icon, the .desktop file was removed and never re-created, breaking the applet.</p>

<p>BUG: 380922<br />
FIXED-IN: 5.10.2</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>(Possibly for 5.8 branch also)</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Dropped app on desktop, clicked it, worked</li>
<li class="remarkup-list-item">Changed icon for app, worked</li>
<li class="remarkup-list-item">Changed name for app, worked</li>
</ul>

<ul class="remarkup-list">
<li class="remarkup-list-item">Dropped file on desktop, clicked it, worked</li>
<li class="remarkup-list-item">Changed icon for file link, now works, didn't before</li>
<li class="remarkup-list-item">Changed URL for file link, still works</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6148" rel="noreferrer">https://phabricator.kde.org/D6148</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>applets/icon/iconapplet.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>