[Differential] [Commented On] D3262: Get .desktop file name for launcher URL and activities string from KWindowInfo.

[graesslin (Martin Gräßlin)]

graesslin added inline comments.

INLINE COMMENTS

> hein wrote in xwindowtasksmodel.cpp:490-499
> > In KWin we also ensure that the desktopFile ends with .desktop prior to passing to KDesktopFile
> 
> This shouldn't be necessary. "Passing to KDesktopFile" in this code's case is calling KDesktopFile::isDesktopFile, which is documented to do the following:
> 
> "The check is performed looking at the file extension (the file is not opened). Currently, the only valid extension is ".desktop". "
> 
> I.e. checking for the extension isn't necessary, that's what isDesktopFile() is doing. Basically, we try to find a KService first, and if that fails we check if it's a .desktop file and whether it exists, then use it.

exactly. So if you get a name which does not have the .desktop file extension it won't be used.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D3262

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: hein, #plasma, graesslin
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161107/5b1144e0/attachment.html>