From f623c8f7bc5847a771b86e6ba134a0ca704e480b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= <sybren@blender.org>
Date: Fri, 26 Jan 2024 11:05:00 +0100
Subject: [PATCH] Anim: remove 'Solo Visibility' bone collections operator

Remove the 'Solo Visibility' operator that singles out a bone collection.
It was a workaround for not having a true 'solo' flag, which is there
now.
---
 scripts/startup/bl_operators/anim.py          | 43 -------------------
 .../startup/bl_ui/properties_data_armature.py |  8 ----
 2 files changed, 51 deletions(-)

diff --git a/scripts/startup/bl_operators/anim.py b/scripts/startup/bl_operators/anim.py
index 8d6bb1d8949..b2ac59c0ad1 100644
--- a/scripts/startup/bl_operators/anim.py
+++ b/scripts/startup/bl_operators/anim.py
@@ -524,48 +524,6 @@ class ARMATURE_OT_copy_bone_color_to_selected(Operator):
         return {'FINISHED'}
 
 
-class ARMATURE_OT_collection_solo_visibility(Operator):
-    """Hide all other bone collections and show the active one. """ \
-        """Note that it is necessary to also show the ancestors of the active """ \
-        """bone collection in order to ensure its visibility"""
-    bl_idname = "armature.collection_solo_visibility"
-    bl_label = "Solo Visibility"
-    bl_options = {'REGISTER', 'UNDO'}
-
-    name: StringProperty(name='Bone Collection')
-
-    @classmethod
-    def poll(cls, context):
-        return context.object and context.object.type == 'ARMATURE' and context.object.data
-
-    def execute(self, context):
-        arm = context.object.data
-
-        # Find the named bone collection.
-        if self.name:
-            try:
-                solo_bcoll = arm.collections[self.name]
-            except KeyError:
-                self.report({'ERROR'}, "Bone collection %r not found" % self.name)
-                return {'CANCELLED'}
-        else:
-            solo_bcoll = arm.collections.active
-            if not solo_bcoll:
-                self.report({'ERROR'}, "Armature has no active Bone collection, nothing to solo")
-                return {'CANCELLED'}
-
-        # Hide everything first.
-        for bcoll in arm.collections_all:
-            bcoll.is_visible = False
-
-        # Show the named bone collection and all its ancestors.
-        while solo_bcoll:
-            solo_bcoll.is_visible = True
-            solo_bcoll = solo_bcoll.parent
-
-        return {'FINISHED'}
-
-
 class ARMATURE_OT_collection_show_all(Operator):
     """Show all bone collections"""
     bl_idname = "armature.collection_show_all"
@@ -673,7 +631,6 @@ classes = (
     ClearUselessActions,
     UpdateAnimatedTransformConstraint,
     ARMATURE_OT_copy_bone_color_to_selected,
-    ARMATURE_OT_collection_solo_visibility,
     ARMATURE_OT_collection_show_all,
     ARMATURE_OT_collection_remove_unused,
 )
diff --git a/scripts/startup/bl_ui/properties_data_armature.py b/scripts/startup/bl_ui/properties_data_armature.py
index ec6b7089776..d0ee133fb29 100644
--- a/scripts/startup/bl_ui/properties_data_armature.py
+++ b/scripts/startup/bl_ui/properties_data_armature.py
@@ -143,15 +143,8 @@ class ARMATURE_MT_collection_context_menu(Menu):
     def draw(self, context):
         layout = self.layout
 
-        arm = context.armature
-        active_bcoll = arm.collections.active
-
-        props = layout.operator("armature.collection_solo_visibility")
-        props.name = active_bcoll.name if active_bcoll else ""
         layout.operator("armature.collection_show_all")
-
         layout.separator()
-
         layout.operator("armature.collection_remove_unused", text="Remove Unused")
 
 
@@ -175,7 +168,6 @@ class ARMATURE_MT_collection_tree_context_menu(Menu):
 
         layout.separator()
 
-        layout.operator("armature.collection_solo_visibility")
         layout.operator("armature.collection_show_all")
 
         layout.separator()