Don't check for QVariant::isValid() when setting up observed properties.

An invalid QVariant can be the result of an undefined property, whose
uses still need to be tracked.

Change-Id: I66c74ae0a36b2f597a2774221bafac7fb3e994d8
Reviewed-by: Joerg Bornemann <joerg.bornemann@digia.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/lib/buildgraph/buildgraph.cpp b/src/lib/buildgraph/buildgraph.cpp
index 4f0648d39..d382384be 100644
--- a/src/lib/buildgraph/buildgraph.cpp
+++ b/src/lib/buildgraph/buildgraph.cpp
@@ -153,10 +153,10 @@ static void setupProductScriptValue(ScriptEngine *engine, QScriptValue &productS
     const QVariantMap &propMap = product->properties->value();
     for (QVariantMap::ConstIterator it = propMap.constBegin(); it != propMap.constEnd(); ++it) {
         const QVariant &value = it.value();
-        if (value.isValid()
-                && (!value.canConvert<QVariantMap>() || it.key() != QLatin1String("modules")))
+        if (!value.canConvert<QVariantMap>() || it.key() != QLatin1String("modules")) {
             engine->setObservedProperty(productScriptValue, it.key(),
                                         engine->toScriptValue(value), observer);
+        }
     }
 }