fix(bulk-model-sync-lib): do not attempt to remove properties that ar…

…e null Fixes: MODELIX-746
modelix · Feb 20, 2024 · fcac1dd · fcac1dd
1 parent a72dfc0
commit fcac1dd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/bulk-model-sync-lib/src/commonMain/kotlin/org/modelix/model/sync/bulk/ModelImporter.kt b/bulk-model-sync-lib/src/commonMain/kotlin/org/modelix/model/sync/bulk/ModelImporter.kt
@@ -219,7 +219,7 @@ class ModelImporter(
 
         val toBeRemoved = node.getPropertyRoles().toSet()
             .subtract(nodeData.properties.keys)
-            .filter { it != NodeData.idPropertyKey }
+            .filter { it != NodeData.idPropertyKey && node.getPropertyValue(it) != null }
         toBeRemoved.forEach { node.setPropertyValue(it, null) }
     }