Appended comments on unit test.

Added 'valid' property in QStateAction.

5 files changed, 88 insertions, 30 deletions

diff --git a/src/common/qdbusactiongroup.cpp b/src/common/qdbusactiongroup.cpp
index 2b3528e..1038014 100644
--- a/src/common/qdbusactiongroup.cpp
+++ b/src/common/qdbusactiongroup.cpp
@@ -71,13 +71,22 @@ QDBusActionGroup::~QDBusActionGroup()
 */
 QStateAction *QDBusActionGroup::action(const QString &name)
 {
+    QStateAction *act = actionImpl(name);
+    if (act == 0) {
+        return addAction(name.toLatin1(), true);
+    } else {
+        return act;
+    }
+}
+
+QStateAction *QDBusActionGroup::actionImpl(const QString &name)
+{
     Q_FOREACH(QStateAction *act, m_actions) {
         if (act->text() == name) {
             return act;
         }
     }
-
-    return NULL;
+    return 0;
 }
 
 /*!
@@ -162,48 +171,66 @@ void QDBusActionGroup::setActionGroup(GDBusActionGroup *ag)
 
         gchar **actionNames = g_action_group_list_actions(m_actionGroup);
         for(int i=0; actionNames[i] != NULL; i++) {
-            addAction(actionNames[i]);
+            addAction(actionNames[i], true);
         }
         g_strfreev(actionNames);
     }
 }
 
 /*! \internal */
-void QDBusActionGroup::addAction(const char *actionName)
+QStateAction *QDBusActionGroup::addAction(const char *actionName, bool create)
 {
-    QStateAction *act = new QStateAction(actionName, this);
-
-    act->setEnabled(g_action_group_get_action_enabled(m_actionGroup, actionName));
-
-    GVariant *actState = g_action_group_get_action_state(m_actionGroup, actionName);
-    if (actState) {
-        act->setState(Converter::parseGVariant(actState));
+    bool isNew = false;
+    QStateAction *act = actionImpl(actionName);
+    if (act == 0) {
+        if (create) {
+            act = new QStateAction(actionName, this);
+            isNew = true;
+        } else {
+            return 0;
+        }
     }
 
-    QObject::connect(act, SIGNAL(triggered()), this, SLOT(onActionTriggered()));
+    if (g_action_group_has_action(m_actionGroup, actionName)) {
+        act->setEnabled(g_action_group_get_action_enabled(m_actionGroup, actionName));
 
-    // remove any older action with the same name
-    removeAction(actionName);
+        GVariant *actState = g_action_group_get_action_state(m_actionGroup, actionName);
+        if (actState) {
+            act->setState(Converter::parseGVariant(actState));
+        }
+        act->setValid(true);
+    } else {
+        act->setValid(false);
+    }
 
-    m_actions.insert(act);
-    Q_EMIT countChanged(m_actions.count());
+    if (isNew) {
+        QObject::connect(act, SIGNAL(triggered()), this, SLOT(onActionTriggered()));
+        m_actions.insert(act);
+        Q_EMIT countChanged(m_actions.count());
+    }
+    return act;
 }
 
 /*! \internal */
 void QDBusActionGroup::onActionTriggered()
 {
-    QAction *act = qobject_cast<QAction*>(QObject::sender());
-    g_action_group_activate_action(m_actionGroup, act->text().toLatin1(), NULL);
+    QStateAction *act = qobject_cast<QStateAction*>(QObject::sender());
+    if (act->isValid()) {
+        g_action_group_activate_action(m_actionGroup, act->text().toLatin1(), NULL);
+    }
 }
 
 /*! \internal */
-void QDBusActionGroup::removeAction(const char *actionName)
+void QDBusActionGroup::removeAction(const char *actionName, bool erase)
 {
     Q_FOREACH(QStateAction *act, m_actions) {
         if (act->text() == actionName) {
-            m_actions.remove(act);
-            delete act;
-            Q_EMIT countChanged(m_actions.count());
+            if (erase) {
+                m_actions.remove(act);
+                delete act;
+            } else {
+                act->setValid(false);
+            }
             break;
         }
     }
@@ -236,15 +263,14 @@ void QDBusActionGroup::clear()
 void QDBusActionGroup::onActionAdded(GDBusActionGroup *, gchar *actionName, gpointer data)
 {
     QDBusActionGroup *self = reinterpret_cast<QDBusActionGroup*>(data);
-    self->addAction(actionName);
+    self->addAction(actionName, true);
 }
 
 /*! \internal */
 void QDBusActionGroup::onActionRemoved(GDBusActionGroup *, gchar *actionName, gpointer data)
 {
     QDBusActionGroup *self = reinterpret_cast<QDBusActionGroup*>(data);
-    self->removeAction(actionName);
-
+    self->removeAction(actionName, false);
 }
 
 /*! \internal */
diff --git a/src/common/qdbusactiongroup.h b/src/common/qdbusactiongroup.h
index 28e4f25..f2b2033 100644
--- a/src/common/qdbusactiongroup.h
+++ b/src/common/qdbusactiongroup.h
@@ -43,6 +43,8 @@ public:
 
     int count() const;
 
+    Q_INVOKABLE QStateAction *action(const QString &actionName);
+
 Q_SIGNALS:
     void busTypeChanged(BusType type);
     void busNameChanged(const QString &busNameChanged);
@@ -50,10 +52,11 @@ Q_SIGNALS:
     void statusChanged(ConnectionStatus status);
     void countChanged(int count);
 
+
 public Q_SLOTS:
     void start();
     void stop();
-    QStateAction *action(const QString &actionName);
+
 
 protected:
     virtual void serviceAppear(GDBusConnection *connection);
@@ -74,10 +77,11 @@ private:
     void setIntBusType(int busType);
 
     void setActionGroup(GDBusActionGroup *ag);
-    void addAction(const char *actionName);
-    void removeAction(const char *actionName);
+    QStateAction *addAction(const char *actionName, bool create);
+    void removeAction(const char *actionName, bool erase);
     void updateAction(const char *actionName, GVariant *state);
     void clear();
+    QStateAction *actionImpl(const QString &actionName);
 
     // glib slots
     static void onActionAdded(GDBusActionGroup *ag, gchar *actionName, gpointer data);
diff --git a/src/common/qmenumodel.cpp b/src/common/qmenumodel.cpp
index accc47d..2321f1c 100644
--- a/src/common/qmenumodel.cpp
+++ b/src/common/qmenumodel.cpp
@@ -70,7 +70,7 @@ void QMenuModel::setMenuModel(GMenuModel *other)
         g_object_unref(m_menuModel);
     }
 
-    m_menuModel = other;    
+    m_menuModel = other;
 
     if (m_menuModel) {
         // this will trigger the menu load
diff --git a/src/common/qstateaction.cpp b/src/common/qstateaction.cpp
index a9ecabc..8e3c7c3 100644
--- a/src/common/qstateaction.cpp
+++ b/src/common/qstateaction.cpp
@@ -58,6 +58,25 @@ QVariant QStateAction::state() const
     return m_state;
 }
 
+/*!
+    \qmlproperty int QStateAction::isValid
+    This property return if the current Action is valid or not
+    A valid Action is a action which has a DBus action linked
+*/
+bool QStateAction::isValid() const
+{
+    return m_valid;
+}
+
+/*! \internal */
+void QStateAction::setValid(bool valid)
+{
+    if (m_valid != valid) {
+        m_valid = valid;
+        Q_EMIT validChanged(m_valid);
+    }
+}
+
 /*! \internal */
 void QStateAction::setState(const QVariant &state)
 {
diff --git a/src/common/qstateaction.h b/src/common/qstateaction.h
index 3ed6bf3..4019c64 100644
--- a/src/common/qstateaction.h
+++ b/src/common/qstateaction.h
@@ -27,17 +27,26 @@ class QStateAction : public QAction
 {
     Q_OBJECT
     Q_PROPERTY(QVariant state READ state WRITE setState NOTIFY stateChanged)
+    Q_PROPERTY(bool valid READ isValid NOTIFY validChanged)
 public:
     QStateAction(const QString &text="", QObject *parent=0);
 
     QVariant state() const;
     void setState(const QVariant &state);
 
+    bool isValid() const;
+
 Q_SIGNALS:
     void stateChanged(QVariant state);
+    void validChanged(bool valid);
 
 private:
-    QVariant m_state;
+    QVariant m_state;    
+    bool m_valid;
+
+    void setValid(bool valid);
+
+    friend class QDBusActionGroup;
 };
 
 #endif