[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: Bug in KonqPopupMenuPrivate? Re: Help needed with KDiff3 service
From:       Albert Astals Cid <aacid () kde ! org>
Date:       2008-06-21 14:46:34
Message-ID: 200806211646.34822.aacid () kde ! org
[Download RAW message or body]

A Dissabte 21 Juny 2008, David Faure va escriure:
> On Saturday 21 June 2008, Albert Astals Cid wrote:
> > A Dissabte 21 Juny 2008, David Faure va escriure:
> > > On Saturday 21 June 2008, Albert Astals Cid wrote:
> > > > It seems a bug in KonqPopupMenuPrivate, if you set
> > > > ServiceTypes=KonqPopupMenu/Plugin,inode/directory
> > > > it will appear on all directories, but if you set it on all/all it
> > > > does not, i'd say the KMimeTypeTrader call on
> > > > KonqPopupMenuPrivate::addPlugins needs to handle inheritancy when
> > > > it's not.
> > > >
> > > > David can you confirm it is a bug?
> > >
> > > No bug. all/all is a kde3 name (kept in servicemenus though iirc).
> >
> > Why keditfiletype all/all works then?
>
> Depends on your definition of "works" :)

Does not give me the same of keditfiletype foo/bar

> The mimetype is defined in kdelibs/mimetypes/kde.xml, but we should
> probably remove it since it has no meaning - no file ever matches it.

Agreed, if it never matches anything, just remove it, maybe all/allfiles has 
to go too as afair i tried to use it and did not seem to match either.

Albert

>
> OTOH I agree that it's a better name than
> "application/octet-stream,inode/directory" so maybe we should define
> all/allfiles as an alias for application/octet-stream, at least. But it
> would be KDE only unless shared-mime-info gets this alias...
>
> > > Note however that the konqpopupmenu support for "real" plugins (.so) is
> > > untested in kde4, I re-enabled it only recently and didn't have a
> > > chance to test it, so you might have to fix bugs in it (but not this
> > > one :).
> >
> > Not me, i just was lending a hand to Joachim :D
>
> I know, it was a general "you", I didn't necessarily mean "you" :)


 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]