davidedmundson added inline comments.

INLINE COMMENTS
> graesslin wrote in main.cpp:212
> Why are you querying KService? Scripts are packages and at least KWin 
> internally they are queried using:
> 
>   
> KPackage::PackageLoader::self()->listPackages(QStringLiteral("KWin/Script"), 
> scriptFolder);

it's copy paste from kcm_kwinscripts

If you are doing that in kwin you have a bug as you're ignoring the 
KWin-Exclude-Listing flag and potentially loading things that you can't 
configure.

REPOSITORY
  rKWIN KWin

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

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

To: davidedmundson, #plasma
Cc: graesslin, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, 
jensreuterberg, abetts, sebas

Reply via email to