Check for datainput validity when updating datainput map

Broken presentations might contain control bindings to datainputs that
do not exist, so check that datainput name is actually listed in the
global datainput map.

Change-Id: I4d5fb1fa85464c6a60576d9f5d1e2a671f48512b
Reviewed-by: Tomi Korpipää <tomi.korpipaa@qt.io>
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>

1 files changed, 5 insertions, 1 deletions

diff --git a/src/Authoring/Client/Code/Core/Doc/Doc.cpp b/src/Authoring/Client/Code/Core/Doc/Doc.cpp
index 8659923f..ecb16fb8 100644
--- a/src/Authoring/Client/Code/Core/Doc/Doc.cpp
+++ b/src/Authoring/Client/Code/Core/Doc/Doc.cpp
@@ -3061,7 +3061,11 @@ void CDoc::UpdateDatainputMapForInstance(qt3dsdm::Qt3DSDMInstanceHandle inInstan
             else if (propName == QLatin1String("@timeline"))
                 item.dataType = {qt3dsdm::DataModelDataType::Value::RangedNumber, true};
 
-            g_StudioApp.m_dataInputDialogItems[diName]->ctrldElems.append(item);
+            // Check for DI name validity because we might have broken
+            // presentations with property control bindings set to non-existent
+            // datainputs
+            if (g_StudioApp.m_dataInputDialogItems.contains(diName))
+                g_StudioApp.m_dataInputDialogItems[diName]->ctrldElems.append(item);
         }
     }
 }