Revert accidental commit

This reverts commit f0f48da209de08d66fd9e3b311e379687c7ba699.

1 files changed, 15 insertions, 40 deletions

diff --git a/rigify/feature_set_list.py b/rigify/feature_set_list.py
index 0367a70d..a50bd010 100644
--- a/rigify/feature_set_list.py
+++ b/rigify/feature_set_list.py
@@ -16,8 +16,6 @@
 #
 #======================= END GPL LICENSE BLOCK ========================
 
-from typing import List
-
 import bpy
 from bpy.props import StringProperty
 import os
@@ -46,8 +44,7 @@ def get_install_path(*, create=False):
     return INSTALL_PATH
 
 
-def get_installed_modules_names() -> List[str]:
-    """Return a list of module names of all feature sets in the file system."""
+def get_installed_list():
     features_path = get_install_path()
     if not features_path:
         return []
@@ -63,20 +60,6 @@ def get_installed_modules_names() -> List[str]:
     return sets
 
 
-def get_enabled_modules_names() -> List[str]:
-    """Return a list of module names of all enabled feature sets."""
-    rigify_prefs = bpy.context.preferences.addons[__package__].preferences
-    installed_module_names = get_installed_modules_names()
-    rigify_feature_sets = rigify_prefs.rigify_feature_sets
-
-    enabled_module_names = { 
-        fs.module_name for fs in rigify_feature_sets 
-        if fs.enabled and fs.module_name in installed_module_names
-    }
-
-    return enabled_module_names
-
-
 def get_module(feature_set):
     return importlib.import_module('.'.join([*NAME_PREFIX, feature_set]))
 
@@ -105,17 +88,17 @@ def get_info_dict(feature_set):
     return {}
 
 
-def call_function_safe(module_name, func_name, args=[], kwargs={}):
-    module = get_module_safe(module_name)
+def call_function_safe(feature_set, name, args=[], kwargs={}):
+    module = get_module_safe(feature_set)
 
     if module:
-        func = getattr(module, func_name, None)
+        func = getattr(module, name, None)
 
         if callable(func):
             try:
                 return func(*args, **kwargs)
             except Exception:
-                print(f"Rigify Error: Could not call function '{func_name}' of feature set '{module_name}': exception occurred.\n")
+                print("Rigify Error: Could not call function '%s' of feature set '%s': exception occurred.\n" % (name, feature_set))
                 traceback.print_exc()
                 print("")
 
@@ -145,7 +128,7 @@ def feature_set_items(scene, context):
         ('rigify', 'Rigify Built-in', 'Rigs bundled with Rigify'),
     ]
 
-    for fs in get_enabled_modules_names():
+    for fs in get_installed_list():
         ui_name = get_ui_name(fs)
         items.append((fs, ui_name, ui_name))
 
@@ -239,7 +222,8 @@ class DATA_OT_rigify_add_feature_set(bpy.types.Operator):
 
             addon_prefs.update_external_rigs()
 
-            addon_prefs.active_feature_set_index = len(addon_prefs.rigify_feature_sets)-1
+            new_index = addon_prefs.rigify_feature_sets.find(get_ui_name(fixed_dirname))
+            addon_prefs.active_feature_set_index = new_index
 
         return {'FINISHED'}
 
@@ -250,6 +234,8 @@ class DATA_OT_rigify_remove_feature_set(bpy.types.Operator):
     bl_description = "Remove external feature set (rigs, metarigs, ui templates)"
     bl_options = {"REGISTER", "UNDO", "INTERNAL"}
 
+    featureset: StringProperty(maxlen=1024, options={'HIDDEN', 'SKIP_SAVE'})
+
     @classmethod
     def poll(cls, context):
         return True
@@ -259,29 +245,18 @@ class DATA_OT_rigify_remove_feature_set(bpy.types.Operator):
 
     def execute(self, context):
         addon_prefs = context.preferences.addons[__package__].preferences
-        feature_sets = addon_prefs.rigify_feature_sets
-        active_idx = addon_prefs.active_feature_set_index
-        active_fs = feature_sets[active_idx]
 
-        # Call the unregister callback of the set being removed.
-        if active_fs.enabled:
-            call_register_function(active_fs.module_name, register=False)
+        # Call the unregister callback of the set being removed
+        call_register_function(self.featureset, False)
 
-        # Remove the feature set's folder from the file system.
         rigify_config_path = get_install_path()
         if rigify_config_path:
-            set_path = os.path.join(rigify_config_path, active_fs.module_name)
+            set_path = os.path.join(rigify_config_path, self.featureset)
             if os.path.exists(set_path):
                 rmtree(set_path)
 
-        # Remove the feature set's entry from the addon preferences.
-        feature_sets.remove(active_idx)
-
-        # Remove the feature set's entries from the metarigs and rig types.
-        addon_prefs.update_external_rigs()
-
-        # Update active index.
-        addon_prefs.active_feature_set_index -= 1
+        addon_prefs.update_external_rigs(force=True)
+        addon_prefs.active_feature_set_index = 0
 
         return {'FINISHED'}