Bonsai: refresh material data unconditionally instead of forcing a redraw

falken10vdl reviewed 16b1b4e7b1 on #8843 and pointed out that tagging
every area for redraw was overkill. The actual problem was that the
Object Material panel and the scene Materials list read from plain
python caches (ObjectMaterialData and MaterialsData) that only get
invalidated when the Materials editing UI list is reloaded, which
never happens while you are not in editing mode. The redraw itself was
never the issue, closing the rename dialog already triggers one.

Removed the tag_redraw loop from RenameMaterial and instead call the
existing bonsai.bim.module.material.data.refresh() function from
core.rename_material, unconditionally, through a new tool.Material.refresh()
method. This is the same invalidate-on-next-load mechanism already used
by every other module's Data classes, just wired up for this operator
too, instead of introducing a new one.

Also updates the core tests to prescribe the new unconditional refresh()
call, and adds tool-layer coverage for tool.Material.refresh().

Generated with the assistance of an AI coding tool.

(cherry picked from commit 7ab0628c54)

src/bonsai/test/core/test_material.py

@@ -97,6 +97,7 @@ class TestRenameMaterial:
     def test_renaming_a_material(self, ifc, material):
         ifc.run("material.edit_material", material="material", attributes={"Name": "name"}).should_be_called()
         material.is_editing_materials().should_be_called().will_return(False)
+        material.refresh().should_be_called()
         subject.rename_material(ifc, material, material="material", name="name")
 
     def test_renaming_a_material_and_reloading_imported_materials(self, ifc, material):
@@ -104,6 +105,7 @@ class TestRenameMaterial:
         material.is_editing_materials().should_be_called().will_return(True)
         material.get_active_material_type().should_be_called().will_return("material_type")
         material.import_material_definitions("material_type").should_be_called()
+        material.refresh().should_be_called()
         subject.rename_material(ifc, material, material="material", name="name")
 
 

src/bonsai/test/tool/test_material.py

@@ -139,6 +139,17 @@ class TestImportMaterialDefinitions(NewFile):
         assert props.materials[0].total_elements == 0
 
 
+class TestRefresh(NewFile):
+    def test_run(self):
+        from bonsai.bim.module.material.data import MaterialsData, ObjectMaterialData
+
+        MaterialsData.is_loaded = True
+        ObjectMaterialData.is_loaded = True
+        subject.refresh()
+        assert MaterialsData.is_loaded is False
+        assert ObjectMaterialData.is_loaded is False
+
+
 class TestIsEditingMaterials(NewFile):
     def test_run(self):
         props = tool.Material.get_material_props()