Support for tryExec in sidebar plugins

[David Faure]

On Monday 19 July 2004 21:05, Marcus Camen wrote:
> On Monday 19 July 2004 20:43, David Faure wrote:
> > On Monday 19 July 2004 20:33, Marcus Camen wrote:
> > > Hi *,
> > >
> > > AFAIK for kdm session files (like twm.desktop) we read the 'TryExec'
> > > key in the .desktop file and check if the given executable is
> > > available or not. The same applies for KService which is used by the
> > > Kontact plugin system (e.g. David and Daniel committed a modified
> > > newsticker plugin today).
> > >
> > > The attached patch implements a similar functionality in
> > > kdebase/konqueror/sidebar/sidebar_widget.cpp. It affects the 'Add
> > > New' popup menu in a way that sidebar plugins which have
> > > (TryExec==foo && executable foo does not exist) will not be shown.
> > > Therefore a user will not be able to add non-functional plugins.
> >
> > I like the idea but I think it should be implemented with
> > KDesktopFile::tryExec()
> 
> Yeah, I ripped from kdm/kfrontend/kgreeter.h  ;)
Ossi: you know what to fix :)

> Patch with KDesktopFile is attached.
Looks fine to me, assuming you tested it, of course :)

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).