<table><tr><td style="">kossebau requested changes to this revision.<br />kossebau added a comment.<br />This revision now requires changes to proceed.
</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/D5405" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Having slept over this one night, I still think that this change should be not the way to go to fix the seen issues.</p>

<p>The actual problem is bad usage in application code, usually due to lack of knowledge what is needed.<br />
The documented API of KAboutData, especially  <a href="https://api.kde.org/frameworks/kcoreaddons/html/classKAboutData.html#a112d2fc20c31e7847995930e030cc67b" class="remarkup-link" target="_blank" rel="noreferrer">AboutData::setDesktopFileName()</a>, is very explicite what needs to be done and when. This patch would change this behaviour and runs a chance to break things for application code which relies on the documented behaviour.<br />
Changing this behaviour to help application code whose authors never bothered to get things only means screwing over those application developers who did the effort to read up in the API dox and write proper code.</p>

<p>If we were to redesign this API, the behaviour proposed in this patch would be fine IMHO. So IMHO please add a comment to change things for KF6, as that the desktopfilename should be auto-generated not in the constructor and then be fixed, but from the currently set component name and org domain. Unless set explicitely to a value.</p>

<p>What could be done here, is to add some example code to the class API dox, for the section "Detailed Description", which would serve as some template code how to use the existing API for application metadata, together with the needed desktopfile name and/or D-Bus names. And then write a blog post to try to catch more people's awareness of this issue, pointing out also all applications which have been fixed already.</p>

<p>Some additional unit tests for KAboutData around the issue would be good to have as well.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5405" rel="noreferrer">https://phabricator.kde.org/D5405</a></div></div><br /><div><strong>To: </strong>stikonas, mpyne, ltoscano, kossebau, aacid<br /><strong>Cc: </strong>plasma-devel, kde-frameworks-devel, Frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol<br /></div>