Merge pull request #4310 from ynput/bugfix/loaders_can_be_disabled

General: Loader and Creator plugins can be disabled

openpype/pipeline/create/legacy_create.py

@@ -20,6 +20,7 @@ class LegacyCreator(object):
     family = None
     defaults = None
     maintain_selection = True
+    enabled = True
 
     dynamic_subset_keys = []
 
@@ -76,11 +77,10 @@ def apply_settings(cls, project_settings, system_settings):
         print(">>> We have preset for {}".format(plugin_name))
         for option, value in plugin_settings.items():
             if option == "enabled" and value is False:
-                setattr(cls, "active", False)
                 print("  - is disabled by preset")
             else:
-                setattr(cls, option, value)
                 print("  - setting `{}`: `{}`".format(option, value))
+            setattr(cls, option, value)
 
     def process(self):
         pass

openpype/pipeline/load/plugins.py

@@ -30,6 +30,7 @@ class LoaderPlugin(list):
     representations = list()
     order = 0
     is_multiple_contexts_compatible = False
+    enabled = True
 
     options = []
 
@@ -73,11 +74,10 @@ def apply_settings(cls, project_settings, system_settings):
         print(">>> We have preset for {}".format(plugin_name))
         for option, value in plugin_settings.items():
             if option == "enabled" and value is False:
-                setattr(cls, "active", False)
                 print("  - is disabled by preset")
             else:
-                setattr(cls, option, value)
                 print("  - setting `{}`: `{}`".format(option, value))
+            setattr(cls, option, value)
 
     @classmethod
     def get_representations(cls):

openpype/pipeline/workfile/build_workfile.py

@@ -120,6 +120,8 @@ def build_workfile(self):
         # Prepare available loaders
         loaders_by_name = {}
         for loader in discover_loader_plugins():
+            if not loader.enabled:
+                continue
             loader_name = loader.__name__
             if loader_name in loaders_by_name:
                 raise KeyError(

openpype/pipeline/workfile/workfile_template_builder.py

@@ -239,6 +239,8 @@ def get_creators_by_name(self):
         if self._creators_by_name is None:
             self._creators_by_name = {}
             for creator in discover_legacy_creator_plugins():
+                if not creator.enabled:
+                    continue
                 creator_name = creator.__name__
                 if creator_name in self._creators_by_name:
                     raise KeyError(

openpype/tools/creator/model.py

@@ -23,6 +23,8 @@ def reset(self):
         items = []
         creators = discover_legacy_creator_plugins()
         for creator in creators:
+            if not creator.enabled:
+                continue
             item_id = str(uuid.uuid4())
             self._creators_by_id[item_id] = creator
 

openpype/tools/loader/widgets.py

@@ -451,6 +451,8 @@ def on_context_menu(self, point):
         repre_loaders = []
         subset_loaders = []
         for loader in available_loaders:
+            if not loader.enabled:
+                continue
             # Skip if its a SubsetLoader.
             if issubclass(loader, SubsetLoaderPlugin):
                 subset_loaders.append(loader)
@@ -1352,6 +1354,8 @@ def on_context_menu(self, point):
 
         filtered_loaders = []
         for loader in available_loaders:
+            if not loader.enabled:
+                continue
             # Skip subset loaders
             if issubclass(loader, SubsetLoaderPlugin):
                 continue