Plasma/libplasma2/API Review/RunnerManager: Difference between revisions

From KDE Community Wiki
(Created page with "{| cellspacing="0" cellpadding="5" border="1" style="border: 1px solid gray; border-collapse: collapse; text-align: left; width: 100%;" class="sortable" |- style="background: ...")
 
No edit summary
Line 9: Line 9:


{{FeatureDone|Keep| explicit RunnerManager(QObject *parent=0);
{{FeatureDone|Keep| explicit RunnerManager(QObject *parent=0);
{{FeatureTodo|Remove?| explicit RunnerManager(KConfigGroup &config, QObject *parent=0); <- don’t like it much, but probaly the cleanest way, (alternative would be init())}}
{{FeatureTodo|Remove?| explicit RunnerManager(KConfigGroup &config, QObject *parent); <- don’t like it much, but probaly the cleanest way, (alternative would be init())}}
{{FeatureDone|Keep| ~RunnerManager();}}
{{FeatureDone|Keep| ~RunnerManager();}}
{{FeatureDone|Keep|  AbstractRunner *runner(const QString &name) const;}}
{{FeatureDone|Keep|  AbstractRunner *runner(const QString &name) const;}}
Line 19: Line 19:
{{FeatureTodo|Subclass| QStringList singleModeAdvertisedRunnerIds() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass?}}
{{FeatureTodo|Subclass| QStringList singleModeAdvertisedRunnerIds() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass?}}


{{FeatureTodo|Move| QString runnerName(const QString &id) const;->use KpluginInfo}}
{{FeatureDone|Keep| QList<AbstractRunner *> runners() const;}}
{{FeatureTodo|Review RunnerContext| RunnerContext *searchContext() const;}}


{{FeatureDone|Keep| QList<QueryMatch> matches() const;}}
{{FeatureDone|Keep| void run(const QueryMatch &match);}}
{{FeatureDone|Keep| void run(const QString &matchId);}}
{{FeatureDone|Keep| QList<QAction*> actionsForMatch(const QueryMatch &match);}}
{{FeatureDone|Keep| QString query() const;}}
{{FeatureTodo|Slot|void reloadConfiguration();-> not that nice having it exposed, but should be a slot}}
{{FeatureDone|Keep| void setAllowedRunners(const QStringList &runners);}}
{{FeatureDone|Keep|  QStringList allowedRunners() const;}}
{{FeatureTodo|Move| void loadRunner(const KService::Ptr service);}}
{{FeatureTodo|Move|  void loadRunner(const QString &path);}}
{{FeatureTodo|Move|  static KPluginInfo::List listRunnerInfo(const QString &parentApp = QString());}}
{{FeatureDone|Keep|QMimeData * mimeDataForMatch(const QueryMatch &match) const;}}
{{FeatureDone|Keep|QMimeData * mimeDataForMatch(const QString &matchId) const;}}
{{FeatureTodo|Remove|slot void setupMatchSession(); -> Was done for the window runner, since xlib wasn’t threadsafe, Xcb is, so this can go}}
{{FeatureTodo|Remove|slot void matchSessionComplete(); -> Was done for the window runner, since xlib wasn’t threadsafe, Xcb is, so this can go}}
{{FeatureDone|Keep|slot void launchQuery(const QString &term, const QString &runnerId);}}
{{FeatureDone|Keep|slot void launchQuery(const QString &term);}}
{{FeatureTodo|Remove| bool execQuery(const QString &term, const QString &runnerName);}}
{{FeatureTodo|Remove| bool execQuery(const QString &term);}}
{{FeatureDone|Keep| slot void reset();}}
{{FeatureDone|Keep| void matchesChanged(const QList<Plasma::QueryMatch> &matches);}}
{{FeatureDone|Keep| void queryFinished();}}


|}
|}

Revision as of 11:49, 2 April 2013

Status Action Method
DONE Keep uint id() const;


{{FeatureDone|Keep| explicit RunnerManager(QObject *parent=0);

TO DO Remove? explicit RunnerManager(KConfigGroup &config, QObject *parent); <- don’t like it much, but probaly the cleanest way, (alternative would be init()) <{{{3}}}>
DONE Keep ~RunnerManager();
DONE Keep AbstractRunner *runner(const QString &name) const;
TO DO Subclass AbstractRunner *singleModeRunner() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>
TO DO Subclass void setSingleModeRunnerId(const QString &id); -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>
TO DO Subclass QString singleModeRunnerId() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>
TO DO Subclass bool singleMode() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>
TO DO Subclass void setSingleMode(bool singleMode); -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>
TO DO Subclass QStringList singleModeAdvertisedRunnerIds() const; -> should Single runner mode stuff go into a SingleModeRunnerManagersubclass? <{{{3}}}>


TO DO Move QString runnerName(const QString &id) const;->use KpluginInfo <{{{3}}}>
DONE Keep QList<AbstractRunner *> runners() const;
TO DO Review RunnerContext RunnerContext *searchContext() const; <{{{3}}}>


DONE Keep QList<QueryMatch> matches() const;
DONE Keep void run(const QueryMatch &match);
DONE Keep void run(const QString &matchId);
DONE Keep QList<QAction*> actionsForMatch(const QueryMatch &match);
DONE Keep QString query() const;


TO DO Slot void reloadConfiguration();-> not that nice having it exposed, but should be a slot <{{{3}}}>


DONE Keep void setAllowedRunners(const QStringList &runners);
DONE Keep QStringList allowedRunners() const;


TO DO Move void loadRunner(const KService::Ptr service); <{{{3}}}>
TO DO Move void loadRunner(const QString &path); <{{{3}}}>
TO DO Move {{{2}}} <{{{3}}}>


DONE Keep QMimeData * mimeDataForMatch(const QueryMatch &match) const;
DONE Keep QMimeData * mimeDataForMatch(const QString &matchId) const;


TO DO Remove slot void setupMatchSession(); -> Was done for the window runner, since xlib wasn’t threadsafe, Xcb is, so this can go <{{{3}}}>
TO DO Remove slot void matchSessionComplete(); -> Was done for the window runner, since xlib wasn’t threadsafe, Xcb is, so this can go <{{{3}}}>


DONE Keep slot void launchQuery(const QString &term, const QString &runnerId);
DONE Keep slot void launchQuery(const QString &term);


TO DO Remove bool execQuery(const QString &term, const QString &runnerName); <{{{3}}}>
TO DO Remove  bool execQuery(const QString &term); <{{{3}}}>
DONE Keep slot void reset();
DONE Keep void matchesChanged(const QList<Plasma::QueryMatch> &matches);
DONE Keep void queryFinished();