From kde-devel Sat Jun 21 14:46:34 2008 From: Albert Astals Cid Date: Sat, 21 Jun 2008 14:46:34 +0000 To: kde-devel Subject: Re: Bug in KonqPopupMenuPrivate? Re: Help needed with KDiff3 service Message-Id: <200806211646.34822.aacid () kde ! org> X-MARC-Message: https://marc.info/?l=kde-devel&m=121405968304369 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 <<