Fixed a bug in IfThenElse/Switch for the following sequence:

1. set binary to valid data and then "expand_all()"
2. clear binary data (invalidate) so that the structure is showed in the dvc. For IfThenElse/Switch this means, that an extra level dvc level is inserted.
3. set binary to valid data again and then "expand_all()"
4. try to modify an item. This produced an error "... wxDataViewMainWindow::DoItemChanged(): invalid item" and is now fixed.

Problem was, that when inserting an extra level an removing it again to the parent_dvc_item was not correctly calculated again.

Author: timrid

construct_editor/helper/wrapper.py

@@ -477,6 +477,18 @@ def __init__(
         self._dvc_item: t.Optional[dv.DataViewItem] = None
         self._dvc_item_expanded: bool = False
 
+    def get_debug_infos(self) -> str:
+        s = ""
+        s += f"{'.'.join(self.path)}\n"
+        s += f"  - name={str(self.name)}\n"
+        s += f"  - construct={str(self.construct)}\n"
+        s += f"  - entry={self}\n"
+        s += f"  - dvc_item={str(self.dvc_item)}\n"
+        s += f"  - parent={self.parent}\n"
+        s += f"  - parent_dvc_item={self.get_parent_dvc_item()}\n"
+        s += f"  - subentries={self.subentries}"
+        return s
+
     # default "parent" ########################################################
     @property
     def parent(self) -> Optional["EntryConstruct"]:
@@ -974,6 +986,7 @@ def _get_subentry(self) -> "Optional[EntryConstruct]":
         """Evaluate the conditional function to detect the type of the subentry"""
         obj = self.obj
         if obj is None:
+            self._dvc_item = None  # reset dvc_item, so that the subentries can correctly identify its parents dvc_item
             return None
         else:
             metadata = get_gui_metadata(obj)
@@ -1066,6 +1079,7 @@ def _get_subentry(self) -> "Optional[EntryConstruct]":
         """Evaluate the conditional function to detect the type of the subentry"""
         obj = self.obj
         if obj is None:
+            self._dvc_item = None  # reset dvc_item, so that the subentries can correctly identify its parents dvc_item
             return None
         else:
             metadata = get_gui_metadata(obj)
@@ -1528,6 +1542,7 @@ def _get_subentry(self) -> "Optional[EntryConstruct]":
         """Evaluate the conditional function to detect the type of the subentry"""
         obj = self.obj
         if obj is None:
+            self._dvc_item = None  # reset dvc_item, so that the subentries can correctly identify its parents dvc_item
             return None
         else:
             metadata = get_gui_metadata(obj)
@@ -1607,6 +1622,7 @@ def _get_subentry(self) -> "Optional[EntryConstruct]":
         """Evaluate the conditional function to detect the type of the subentry"""
         obj = self.obj
         if obj is None:
+            self._dvc_item = None  # reset dvc_item, so that the subentries can correctly identify its parents dvc_item
             return None
         else:
             metadata = get_gui_metadata(obj)

construct_editor/main.py

@@ -293,6 +293,7 @@ def on_gallery_selection_changed(self, event):
     def on_clear_binary_clicked(self, event):
         self.example_selector_lbx.SetSelection(wx.NOT_FOUND)
         self.construct_hex_editor.binary = bytes()
+        self.construct_hex_editor.construct_editor.expand_all()
 
     def on_example_selection_changed(self, event):
         selection = self.gallery_selector_lbx.GetStringSelection()
@@ -301,6 +302,7 @@ def on_example_selection_changed(self, event):
 
         # Set example binary
         self.construct_hex_editor.binary = example_binary
+        self.construct_hex_editor.construct_editor.expand_all()
 
     def on_load_binary_file_clicked(self, event):
         with wx.FileDialog(