feat(plugins): display experimental plugins only to flagged sections

fiesta/apps/pickup_system/apps.py

@@ -19,7 +19,6 @@ class PickupSystemConfig(BasePluginAppConfig):
     verbose_name = _("Pickup System")
     emoji = "🤼"
     description = _("Tools for managing pickup of your students.")
-    feature_state = BasePluginAppConfig.FeatureState.EXPERIMENTAL
     order = 30
 
     configuration_model = "pickup_system.PickupSystemConfiguration"

fiesta/apps/plugins/utils.py

@@ -9,13 +9,17 @@
 
 
 @lru_cache
-def all_plugin_apps() -> tuple[BasePluginAppConfig, ...]:
+def all_plugin_apps(filter_f: callable = None) -> tuple[BasePluginAppConfig, ...]:
     """Returns all django app configs considered as PluginApps -- inheriting from PluginAppConfig."""
     from django.apps import apps
 
     return tuple(
         sorted(
-            filter(lambda a: isinstance(a, BasePluginAppConfig), apps.get_app_configs()), key=attrgetter("verbose_name")
+            filter(
+                filter_f,
+                filter(lambda a: isinstance(a, BasePluginAppConfig), apps.get_app_configs()),
+            ),
+            key=attrgetter("verbose_name"),
         )
     )
 

fiesta/apps/sections/views/plugins.py

@@ -7,6 +7,7 @@
 
 from apps.fiestaforms.views.htmx import HtmxFormViewMixin
 from apps.plugins.models import BasePluginConfiguration, Plugin
+from apps.plugins.plugin import BasePluginAppConfig
 from apps.plugins.utils import all_plugin_apps
 from apps.sections.forms.plugin_configuration import get_plugin_configuration_form
 from apps.sections.forms.plugin_state import ChangePluginStateForm, SetupPluginSettingsForm
@@ -27,6 +28,12 @@ def get_context_data(self, **kwargs):
         def by_label(label: str) -> Plugin | None:
             return self.request.in_space_of_section.plugins.filter(app_label=label).first()
 
+        def filter_app(app: BasePluginAppConfig) -> bool:
+            return (
+                self.request.in_space_of_section.allow_experimental_plugins
+                or app.feature_state != BasePluginAppConfig.FeatureState.EXPERIMENTAL
+            )
+
         ctx = super().get_context_data(**kwargs)
         ctx.update(
             plugins=[
@@ -52,7 +59,7 @@ def by_label(label: str) -> Plugin | None:
                         )
                     ),
                 )
-                for app in all_plugin_apps()
+                for app in all_plugin_apps(filter_f=filter_app)
                 if (plugin := by_label(app.label)) or True
             ],
             PluginState=Plugin.State,