Review Request: Multiple actions support for Nepomuk Search Runner

Ryan Bitanga ryan.bitanga at
Wed May 13 09:47:37 CEST 2009

This is an automatically generated e-mail. To reply, visit:

Review request for Plasma.


Reposting patch for multiple actions support for the Nepomuk search runner. It adds open with and service menu actions to the nepomuk search runner. 

Now uses the new KFileItemActions and KFileItemListProperties and no longer depends on libkonq.

Minor style issue: the two new classes in KIO don't have camel case includes yet, and I wasn't sure if I should add them myself.

Another possible point for improvement: move the parseMenu method to AbstractRunner as something like an addActionsFromMenu method to reuse code between runners (useful for the locations runner and the window management runner which I'm thinking of moving into kdereview)


  /trunk/KDE/kdebase/workspace/plasma/runners/nepomuksearch/nepomuksearchrunner.h 967101 
  /trunk/KDE/kdebase/workspace/plasma/runners/nepomuksearch/nepomuksearchrunner.cpp 967101 
  /trunk/KDE/kdebase/workspace/plasma/runners/nepomuksearch/plasma-runner-nepomuksearch.desktop 967101 
  /trunk/KDE/kdebase/workspace/plasma/runners/nepomuksearch/queryclientwrapper.cpp 967101 





More information about the Plasma-devel mailing list