Hidden KDED desktop file crashing systemsettings - where to fix?

[David Faure]

On Tuesday 02 November 2010, Jonathan Marten wrote:
> c) use a KServiceTypeTrader query (which hopefully ignores hidden
> desktop files) instead of listing them.

Yes, this is the right way, since it matches what kded itself does.

kdelibs/kded/kded.cpp:
     const KService::List kdedModules = KServiceTypeTrader::self()->query("KDEDModule");
     for(KService::List::ConstIterator it = kdedModules.begin(); it != kdedModules.end(); ++it)
     {
         KService::Ptr service = *it;
           ...
      }

and yes, it skips Hidden=true files, since they are not in ksycoca at all.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).