Merge remote-tracking branch 'upstream/master' into pui-feature-plugin

docs/docs/extend/plugins/ui.md

@@ -18,9 +18,9 @@ When rendering certain content in the user interface, the rendering functions ar
 
 ## Custom Panels
 
-Many of the pages in the InvenTree web interface are built using a series of "panels" which are displayed on the page. Custom panels can be added to these pages, by implementing the `get_custom_panels` method:
+Many of the pages in the InvenTree web interface are built using a series of "panels" which are displayed on the page. Custom panels can be added to these pages, by implementing the `get_ui_panels` method:
 
-::: plugin.base.ui.mixins.UserInterfaceMixin.get_custom_panels
+::: plugin.base.ui.mixins.UserInterfaceMixin.get_ui_panels
     options:
       show_bases: False
       show_root_heading: False

src/backend/InvenTree/plugin/base/ui/api.py

@@ -9,6 +9,7 @@ from rest_framework.views import APIView
 
 import plugin.base.ui.serializers as UIPluginSerializers
 from common.settings import get_global_setting
+from InvenTree.exceptions import log_error
 from plugin import registry
 
 
@@ -31,17 +32,22 @@ class PluginPanelList(APIView):
         if get_global_setting('ENABLE_PLUGINS_INTERFACE'):
             # Extract all plugins from the registry which provide custom panels
             for _plugin in registry.with_mixin('ui', active=True):
-                # Allow plugins to fill this data out
-                plugin_panels = _plugin.get_custom_panels(
-                    target_model, target_id, request
-                )
+                try:
+                    # Allow plugins to fill this data out
+                    plugin_panels = _plugin.get_ui_panels(
+                        target_model, target_id, request
+                    )
 
-                if plugin_panels and type(plugin_panels) is list:
-                    for panel in plugin_panels:
-                        panel['plugin'] = _plugin.slug
+                    if plugin_panels and type(plugin_panels) is list:
+                        for panel in plugin_panels:
+                            panel['plugin'] = _plugin.slug
 
-                        # TODO: Validate each panel before inserting
-                        panels.append(panel)
+                            # TODO: Validate each panel before inserting
+                            panels.append(panel)
+                except Exception:
+                    # Custom panels could not load
+                    # Log the error and continue
+                    log_error(f'{_plugin.slug}.get_ui_panels')
 
         return Response(
             UIPluginSerializers.PluginPanelSerializer(panels, many=True).data

src/backend/InvenTree/plugin/base/ui/mixins.py

@@ -51,6 +51,10 @@ class UserInterfaceMixin:
 
     - All content is accessed via the API, as requested by the user interface.
     - This means that content can be dynamically generated, based on the current state of the system.
+
+    The following custom UI methods are available:
+    - get_ui_panels: Return a list of custom panels to be injected into the UI
+
     """
 
     class MixinMeta:
@@ -63,8 +67,8 @@ class UserInterfaceMixin:
         super().__init__()
         self.add_mixin('ui', True, __class__)  # type: ignore
 
-    def get_custom_panels(
-        self, instance_type: str, instance_id: int, request: Request
+    def get_ui_panels(
+        self, instance_type: str, instance_id: int, request: Request, **kwargs
     ) -> list[CustomPanel]:
         """Return a list of custom panels to be injected into the UI.
 

src/backend/InvenTree/plugin/samples/integration/user_interface_sample.py

@@ -44,7 +44,7 @@ class SampleUserInterfacePlugin(SettingsMixin, UserInterfaceMixin, InvenTreePlug
         },
     }
 
-    def get_custom_panels(self, instance_type: str, instance_id: int, request):
+    def get_ui_panels(self, instance_type: str, instance_id: int, request, **kwargs):
         """Return a list of custom panels to be injected into the UI."""
         panels = []