<table><tr><td style="">mart created this revision.<br />mart added a reviewer: Plasma.<br />Restricted Application added projects: Plasma, Frameworks.<br />Restricted Application added subscribers: Frameworks, 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/D4082" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Search for the ServiceTypes key in the desktop file before<br />
parsing it "for real", because how it's parsed depends from<br />
the service type definition, so if the servicetype is defined<br />
at the bottom of the file or after keys dependent from the type,<br />
those keys would be parsed incorrectly</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>things using plugins like plasmashell still start correctly,<br />
dropping on the desktop a text file now proposes to create a notes widget.<br />
The notes widget has ServiceTypes defined *after* X-Plasma-DropMimetypes<br />
which is a stringlist, that would be misinterpreted as a string<br />
otherwise</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>BRANCH</strong><div><div>phab/prse2servicetypes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4082" rel="noreferrer">https://phabricator.kde.org/D4082</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/lib/plugin/desktopfileparser.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>mart, Plasma<br /><strong>Cc: </strong>plasma-devel, Frameworks, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>