馃崚 applets/kicker: use correct fallback in AppEntry ctor
Original commit message(s)
- applets/kicker: use correct fallback in AppEntry ctor
Constructing an AppEntry (for non-preferred:// schemes), we first try serviceByStorageId on the path part of the url, and if that does not find anything, we try again with the value of m_id... except that m_id only stores the supplied id in case of preferred:// urls, and is left default-initialized otherwise, so this always looks up the service for empty string, which is not particularly useful.
Instead, look it up by the id that was passed to the ctor. This makes malformed .desktop file names containing the character ":" somewhat work again. (This is disallowed by the spec, but some software may produce them including, unfortunately, kmenuedit.) They're still broken in all sorts of weird ways, but at least they will show up when favorited like in 6.6, rather than just fail.
(cherry picked from commit eac02018)
Co-authored-by: Christoph Wolk cwo.kde@posteo.net