fix: bugs with changing item ids using item()

Author: ragardner

docs/CHANGELOG.md

@@ -6,6 +6,7 @@
 - [#267](https://github.com/ragardner/tksheet/issues/267) as it was not fixed in version `7.4.5`
 
 #### Fixed:
+- Bugs with treeview mode `item()` `iid` parameter, used for changing item ids
 - `see()` function scrolls to wrong position if the row index auto-resized after scrolling
 
 #### Improved:

tksheet/sheet.py

@@ -5238,14 +5238,6 @@ def item(
         """
         if item not in self.RI.tree:
             raise ValueError(f"Item '{item}' does not exist.")
-        if isinstance(iid, str):
-            if iid in self.RI.tree:
-                raise ValueError(f"Cannot rename '{iid}', it already exists.")
-            self.RI.tree[item].iid = iid
-            self.RI.tree[iid] = self.RI.tree.pop(item)
-            self.RI.tree_rns[iid] = self.RI.tree_rns.pop(item)
-            if iid in self.RI.tree_open_ids:
-                self.RI.tree_open_ids[iid] = self.RI.tree_open_ids.pop(item)
         if isinstance(text, str):
             self.RI.tree[item].text = text
         if isinstance(values, list):
@@ -5273,6 +5265,21 @@ def item(
                         )
             else:
                 self.RI.tree_open_ids.discard(item)
+        if isinstance(iid, str):
+            if iid in self.RI.tree:
+                raise ValueError(f"Cannot rename '{iid}', it already exists.")
+            for ciid in self.RI.tree[item].children:
+                self.RI.tree[ciid].parent = iid
+            if self.RI.tree[item].parent:
+                parent_node = self.RI.parent_node(item)
+                item_index = parent_node.children.index(item)
+                parent_node.children[item_index] = iid
+            self.RI.tree[item].iid = iid
+            self.RI.tree[iid] = self.RI.tree.pop(item)
+            self.RI.tree_rns[iid] = self.RI.tree_rns.pop(item)
+            if item in self.RI.tree_open_ids:
+                self.RI.tree_open_ids.discard(item)
+                self.RI.tree_open_ids.add(iid)
         get = not (isinstance(iid, str) or isinstance(text, str) or isinstance(values, list) or isinstance(open_, bool))
         self.set_refresh_timer(redraw=not get and redraw)
         if get: