Multiedit fix N:M listing of childrens

opengisch · Feb 10, 2024 · 3741aec · 3741aec
1 parent 502f5f6
commit 3741aec
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/linking_relation_editor/gui/linking_relation_editor_widget.py b/linking_relation_editor/gui/linking_relation_editor_widget.py
@@ -10,6 +10,7 @@
 
 import os
 from enum import IntEnum
+import copy
 
 from qgis.core import (
     Qgis,
@@ -426,10 +427,10 @@ def updateUiMultiEdit(self):
                         treeWidgetItem.addChild(treeWidgetItemChild)
                         featureIdsMixedValues.add(featureChildChild.id())
 
-                        if treeWidgetItem in multimapChildFeatures:
-                            multimapChildFeatures[treeWidgetItem].append(featureChildChild.id())
+                        if id(treeWidgetItem) in multimapChildFeatures:
+                            multimapChildFeatures[id(treeWidgetItem)].append(featureChildChild.id())
                         else:
-                            multimapChildFeatures[treeWidgetItem] = [featureChildChild.id()]
+                            multimapChildFeatures[id(treeWidgetItem)] = [featureChildChild.id()]
 
                 else:
                     treeWidgetItemChild = self.createMultiEditTreeWidgetItem(
@@ -453,10 +454,10 @@ def updateUiMultiEdit(self):
         # See https://github.com/qgis/QGIS/pull/45703
         #
         if self.nmRelation().isValid():
-            for featureIdMixedValue in featureIdsMixedValues[:]:
+            for featureIdMixedValue in copy.copy(featureIdsMixedValues):
                 mixedValues = True
                 for parentTreeWidgetItem in parentTreeWidgetItems:
-                    if featureIdMixedValue in multimapChildFeatures[parentTreeWidgetItem]:
+                    if featureIdMixedValue in multimapChildFeatures[id(parentTreeWidgetItem)]:
                         mixedValues = True
                         break