Plasma/libplasma2/API Review/DataEngine

From KDE Community Wiki
Revision as of 17:26, 25 March 2013 by Notmart (talk | contribs)
Status Action Method
TO DO Remove explicit DataEngine(QObject *parent , KService::Ptr service); <{{{3}}}>
TO DO Add explicit DataEngine(QObject *parent, const KPluginInfo &info); <{{{3}}}>
TO DO Remove DataEngine(QObject *parent, const QVariantList &args); <{{{3}}}>
DONE Keep ~DataEngine();
TO DO Remove? virtual void init(); can be removed is there will be only one constructor <{{{3}}}>
DONE Keep virtual QStringList sources() const;
DONE Keep virtual Service *serviceForSource(const QString &source);
TO DO Add KPluginInfo pluginInfo() const; <{{{3}}}>
TO DO Remove QString name() const; <{{{3}}}>
TO DO Remove QString icon() const; <{{{3}}}>
TO DO Remove QString pluginName() const; <{{{3}}}>
DONE Keep void connectSource(const QString &source, QObject *visualization, uint pollingInterval, Plasma::IntervalAlignment intervalAlignment ) const;
DONE Keep void connectAllSources(QObject *visualization, uint pollingInterval, Plasma::IntervalAlignment intervalAlignment) const;
DONE Keep void disconnectSource(const QString &source, QObject *visualization) const;
DONE Keep DataContainer *containerForSource(const QString &source);
TO DO Remove DataEngine::Data query(const QString &source) const; <{{{3}}}>
DONE Keep bool isValid() const;
DONE Keep bool isEmpty() const;
DONE Keep  Package package() const;
TO DO Remove {{{2}}} <{{{3}}}>
TO DO Move  static QStringList listAllEngines(const QString &parentApp); -> in PluginLoader <{{{3}}}>
TO DO Move  static KPluginInfo::List listEngineInfo(const QString &parentApp); -> in PluginLoader <{{{3}}}>
TO DO Move static KPluginInfo::List listEngineInfoByCategory(const QString &category, const QString &parentApp); <{{{3}}}>