Remove circular build dependencywip/qtify

Duplicate enum defined in studio3d module on runtime side to avoid
including the entire class from studio3d.

Task-id: QT3DS-3396
Change-Id: I193b34614a8bbf991f717f8302061298b56e451e
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>

5 files changed, 19 insertions, 11 deletions

diff --git a/src/Runtime/Source/runtime/Qt3DSApplication.h b/src/Runtime/Source/runtime/Qt3DSApplication.h
index cdec96a1..b74191f2 100644
--- a/src/Runtime/Source/runtime/Qt3DSApplication.h
+++ b/src/Runtime/Source/runtime/Qt3DSApplication.h
@@ -37,7 +37,6 @@
 #include "Qt3DSKernelTypes.h"
 #include "Qt3DSMetadata.h"
 #include "QtQml/qjsengine.h"
-#include "q3dsdatainput.h"
 
 namespace Q3DStudio {
 class IRuntimeFactory;
@@ -104,6 +103,13 @@ enum DataInputType {
     DataInputTypeVariant
 };
 
+// Duplicated from Q3DSDataInput class on viewer side
+enum class DataInputValueRole {
+    Value = 0,
+    Min = 1,
+    Max = 2
+};
+
 struct DataInputDef
 {
     QVector<DataInputControlledAttribute> controlledAttributes;
diff --git a/src/Runtime/Source/runtime/Qt3DSIScriptBridge.h b/src/Runtime/Source/runtime/Qt3DSIScriptBridge.h
index 2aa59339..8f8925d8 100644
--- a/src/Runtime/Source/runtime/Qt3DSIScriptBridge.h
+++ b/src/Runtime/Source/runtime/Qt3DSIScriptBridge.h
@@ -33,7 +33,7 @@
 #include "EASTL/vector.h"
 #include "EASTL/string.h"
 #include "foundation/Qt3DSRefCounted.h"
-#include "q3dsdatainput.h"
+#include "Qt3DSApplication.h"
 
 #include <QtCore/qvariant.h>
 
@@ -157,7 +157,8 @@ public: // Elements
     virtual void FireEvent(const char *element, const char *evtName) = 0;
     virtual void SetDataInputValue(
             const QString &name, const QVariant &value,
-            Q3DSDataInput::ValueRole property = Q3DSDataInput::ValueRole::Value) = 0;
+            qt3ds::runtime::DataInputValueRole property
+            = qt3ds::runtime::DataInputValueRole::Value) = 0;
     virtual void createElement(const QString &parentElementPath, const QString &slideName,
                                const QHash<QString, QVariant> &properties,
                                qt3ds::render::IQt3DSRenderer *renderer) = 0;
diff --git a/src/Runtime/Source/runtime/Qt3DSQmlEngine.cpp b/src/Runtime/Source/runtime/Qt3DSQmlEngine.cpp
index 86f68bdd..2251a2b8 100644
--- a/src/Runtime/Source/runtime/Qt3DSQmlEngine.cpp
+++ b/src/Runtime/Source/runtime/Qt3DSQmlEngine.cpp
@@ -416,7 +416,7 @@ public:
     bool GetAttribute(const char *element, const char *attName, char *value) override;
     void FireEvent(const char *element, const char *evtName) override;
     void SetDataInputValue(const QString &name, const QVariant &value,
-                           Q3DSDataInput::ValueRole valueRole) override;
+                           qt3ds::runtime::DataInputValueRole valueRole) override;
     void createElement(const QString &parentElementPath, const QString &slideName,
                        const QHash<QString, QVariant> &properties,
                        qt3ds::render::IQt3DSRenderer *renderer) override;
@@ -632,13 +632,13 @@ void CQmlEngineImpl::FireEvent(const char *element, const char *evtName)
 
 void CQmlEngineImpl::SetDataInputValue(
         const QString &name, const QVariant &value,
-        Q3DSDataInput::ValueRole valueRole = Q3DSDataInput::ValueRole::Value)
+        qt3ds::runtime::DataInputValueRole valueRole = qt3ds::runtime::DataInputValueRole::Value)
 {
     qt3ds::runtime::DataInputMap &diMap = m_Application->dataInputMap();
     if (diMap.contains(name)) {
         qt3ds::runtime::DataInputDef &diDef = diMap[name];
         switch (valueRole) {
-        case Q3DSDataInput::ValueRole::Value: { // switch (valueRole)
+        case qt3ds::runtime::DataInputValueRole::Value: { // switch (valueRole)
             diDef.value = value;
             const QVector<qt3ds::runtime::DataInputControlledAttribute> &ctrlAtt
                     = diDef.controlledAttributes;
@@ -816,11 +816,11 @@ void CQmlEngineImpl::SetDataInputValue(
             }
             break;
         }
-        case Q3DSDataInput::ValueRole::Max: { // switch (valueRole)
+        case qt3ds::runtime::DataInputValueRole::Max: { // switch (valueRole)
             diDef.max = value.toFloat();
             break;
         }
-        case Q3DSDataInput::ValueRole::Min: { // switch (valueRole)
+        case qt3ds::runtime::DataInputValueRole::Min: { // switch (valueRole)
             diDef.min = value.toFloat();
             break;
         }
diff --git a/src/Runtime/Source/runtime/q3dsqmlscript.cpp b/src/Runtime/Source/runtime/q3dsqmlscript.cpp
index 91963804..94133a67 100644
--- a/src/Runtime/Source/runtime/q3dsqmlscript.cpp
+++ b/src/Runtime/Source/runtime/q3dsqmlscript.cpp
@@ -383,7 +383,7 @@ QString Q3DSQmlScript::getParent(const QString &handle)
 }
 
 void Q3DSQmlScript::setDataInputValue(const QString &name, const QVariant &value,
-                                      Q3DSDataInput::ValueRole valueRole)
+                                      qt3ds::runtime::DataInputValueRole valueRole)
 {
     m_api.SetDataInputValue(name, value, valueRole);
 }
diff --git a/src/Runtime/Source/runtime/q3dsqmlscript.h b/src/Runtime/Source/runtime/q3dsqmlscript.h
index b05fad03..cf41db54 100644
--- a/src/Runtime/Source/runtime/q3dsqmlscript.h
+++ b/src/Runtime/Source/runtime/q3dsqmlscript.h
@@ -41,7 +41,7 @@
 #include "Qt3DSKernelTypes.h"
 #include "Qt3DSEvent.h"
 #include "q3dsqmlbehavior.h"
-#include "q3dsdatainput.h"
+#include "Qt3DSApplication.h"
 
 namespace Q3DStudio {
 class CQmlEngine;
@@ -75,7 +75,8 @@ public:
     QVector3D matrixToEuler(const QMatrix4x4 &matrix);
     QString getParent(const QString &handle);
     void setDataInputValue(const QString &name, const QVariant &value,
-                           Q3DSDataInput::ValueRole valueRole = Q3DSDataInput::ValueRole::Value);
+                           qt3ds::runtime::DataInputValueRole valueRole
+                           = qt3ds::runtime::DataInputValueRole::Value);
 
     struct EventData {
         QJSValue function;