django-smartbase-admin 1.4.2__tar.gz → 1.4.4__tar.gz

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: django-smartbase-admin
-Version: 1.4.2
+Version: 1.4.4
 Summary: 
 Home-page: https://smartbase-sk.github.io/django-smartbase-admin-docs/
 License: MIT

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "django-smartbase-admin"
-version = "1.4.2"
+version = "1.4.4"
 description = ""
 authors = ["SmartBase <info@smartbase.sk>"]
 readme = "README.md"

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/actions/admin_action_list.py

@@ -573,11 +573,44 @@ class SBAdminListAction(SBAdminAction):
         if not action.is_enabled(action_row):
             return None
 
+        sub_actions = [
+            descriptor
+            for sub_action in action.sub_actions or []
+            if (
+                descriptor := self._materialize_row_action(
+                    sub_action, row, obj_id, raw_row=raw_row
+                )
+            )
+            is not None
+        ]
+        if action.sub_actions and not sub_actions:
+            return None
+
+        if getattr(action, "sub_actions", None):
+            sub_actions = [
+                descriptor
+                for sub_action in action.sub_actions
+                if (
+                    descriptor := self._materialize_row_action(
+                        sub_action, row, obj_id, raw_row=raw_row
+                    )
+                )
+                is not None
+            ]
+            if not sub_actions:
+                return None
+            return {
+                "title": str(action.get_title(action_row) or ""),
+                "icon": action.get_icon(action_row),
+                "css_class": action.get_css_class(action_row) or "",
+                "sub_actions": sub_actions,
+            }
+
         url = action.url
         if url and MODIFIER_OBJECT_ID in url:
             url = url.replace(MODIFIER_OBJECT_ID, str(obj_id))
 
-        return {
+        descriptor = {
             "url": url,
             "title": str(action.get_title(action_row) or ""),
             "icon": action.get_icon(action_row),
@@ -586,6 +619,9 @@ class SBAdminListAction(SBAdminAction):
             "is_method_action": bool(action.action_id) and not action.open_in_modal,
             "open_in_new_tab": bool(action.open_in_new_tab),
         }
+        if action.sub_actions:
+            descriptor["sub_actions"] = sub_actions
+        return descriptor
 
     def get_json_data(self):
         data = self.get_data()

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/admin/admin_base.py

@@ -2,6 +2,7 @@ import json
 import logging
 import urllib.parse
 from collections.abc import Iterable
+from copy import copy
 from functools import partial
 from typing import Any
 from urllib.parse import urlparse
@@ -345,9 +346,11 @@ class SBAdminFormFieldWidgetsMixin:
 
 class SBAdminBaseFormInit(SBAdminFormFieldWidgetsMixin, FormFieldsetMixin):
     view = None
+    sbadmin_action_id = None
 
     def __init__(self, *args, **kwargs):
         self.view = kwargs.pop("view", self.view)
+        self.sbadmin_action_id = kwargs.pop("sbadmin_action_id", self.sbadmin_action_id)
         threadsafe_request = kwargs.pop(
             "request", SBAdminThreadLocalService.get_request()
         )
@@ -736,14 +739,10 @@ class SBAdminThirdParty(SBAdminInlineAndAdminCommon, SBAdminBaseView):
         extra_context.update(self.get_global_context(request))
         return super().changelist_view(request, extra_context)
 
-    def get_action_url(self, action, modifier="template") -> str:
+    def get_action_url(self, action, modifier="template", object_id=None) -> str:
         return reverse(
             "sb_admin:sb_admin_base",
-            kwargs={
-                "view": self.get_id(),
-                "action": action,
-                "modifier": modifier,
-            },
+            kwargs=self.get_action_url_kwargs(action, modifier, object_id),
         )
 
 
@@ -931,9 +930,26 @@ class SBAdmin(
         self.get_form(request)()
 
     def _register_inline_autocomplete(self, request) -> None:
-        for inline in self.get_inline_instances(request, obj=None):
+        obj = None
+        object_id = getattr(request.request_data, "object_id", None)
+        if object_id is not None:
+            obj = self.get_object(request, object_id)
+        for inline in self.get_inline_instances(request, obj=obj):
             inline._register_inline_autocomplete(request)
 
+    def init_actions(self, request) -> None:
+        super().init_actions(request)
+        object_id = getattr(getattr(request, "request_data", None), "object_id", None)
+        if object_id is None:
+            return
+        try:
+            obj = self.get_object(request, object_id)
+        except PermissionDenied:
+            return
+        inline_instances = self.get_inline_instances(request, obj=obj)
+        for inline in inline_instances:
+            inline.init_actions(request)
+
     def get_sbadmin_tabs(self, request, object_id) -> Iterable:
         return self.sbadmin_tabs
 
@@ -962,16 +978,12 @@ class SBAdmin(
     def get_menu_label(self) -> str:
         return self.menu_label or self.model._meta.verbose_name_plural
 
-    def get_action_url(self, action, modifier="template") -> str:
+    def get_action_url(self, action, modifier="template", object_id=None) -> str:
         if not hasattr(self, action):
             raise ImproperlyConfigured(f"Action {action} does not exist on {self}")
         return reverse(
             "sb_admin:sb_admin_base",
-            kwargs={
-                "view": self.get_id(),
-                "action": action,
-                "modifier": modifier,
-            },
+            kwargs=self.get_action_url_kwargs(action, modifier, object_id),
         )
 
     def get_detail_url(self, object_id=None) -> str:
@@ -1281,27 +1293,98 @@ class SBAdminInline(
         return [*(self.sbadmin_inline_list_actions or [])]
 
     def get_sbadmin_inline_list_actions_processed(self, request) -> list:
+        object_id = (
+            self.parent_instance.pk
+            if self.parent_instance is not None and self.parent_instance.pk is not None
+            else None
+        )
         return self.process_inline_actions(
-            request, self.get_sbadmin_inline_list_actions(request)
+            request,
+            self._bind_parent_row_action_modals(
+                self.get_sbadmin_inline_list_actions(request)
+            ),
+            object_id=object_id,
+        )
+
+    def init_actions(self, request) -> None:
+        # inline supports only own sbadmin_inline_list_actions but also sbadmin_fieldsets_actions
+        object_id = (
+            self.parent_instance.pk
+            if self.parent_instance is not None and self.parent_instance.pk is not None
+            else None
         )
+        all_actions = [*self.get_sbadmin_inline_list_actions_processed(request)]
+        if object_id is not None:
+            all_actions.extend(
+                self.get_sbadmin_fieldsets_actions_processed(request, object_id)
+            )
+        self.register_action_autocomplete_views(request, all_actions)
+
+    def _bind_parent_row_action_modals(self, actions: list) -> list:
+        if self.parent_instance is None:
+            return actions
+        parent_admin = self.admin_site._registry.get(self.parent_model)
+        if parent_admin is None:
+            return actions
+        return [
+            self._bind_parent_row_action_modal(action, parent_admin)
+            for action in actions
+        ]
+
+    def _bind_parent_row_action_modal(self, action, parent_admin):
+        # we need to add view on action to be parent_admin
+        # this ->
+        #         SBAdminFormViewAction(
+        #             title=_("Úprava kreditu"),
+        #             target_view=UserManualCreditView,
+        #             open_in_modal=True,
+        #             css_class="btn btn-primary",
+        #         )
+        # needs to become ->
+        #         SBAdminFormViewAction(
+        #             ...
+        #             view=parent_admin
+        #             ...
+        #         )
+        sub_actions = getattr(action, "sub_actions", None)
+        if sub_actions:
+            resolved_sub_actions = [
+                self._bind_parent_row_action_modal(sub_action, parent_admin)
+                for sub_action in sub_actions
+            ]
+            if resolved_sub_actions != sub_actions:
+                action = copy(action)
+                action.sub_actions = resolved_sub_actions
+            return action
+        if getattr(action, "view", None) is not None:
+            return action
+        target_view = getattr(action, "target_view", None)
+        if target_view is None:
+            return action
+        from django_smartbase_admin.engine.modal_view import RowActionModalView
 
-    def get_action_url(self, action, modifier="template") -> str:
+        try:
+            is_row_action_modal = issubclass(target_view, RowActionModalView)
+        except TypeError:
+            is_row_action_modal = False
+        if not is_row_action_modal:
+            return action
+        action = copy(action)
+        action.view = parent_admin
+        return action
+
+    def get_action_url(self, action, modifier="template", object_id=None) -> str:
         return reverse(
             "sb_admin:sb_admin_base",
-            kwargs={
-                "view": self.get_id(),
-                "action": action,
-                "modifier": modifier,
-            },
+            kwargs=self.get_action_url_kwargs(action, modifier, object_id),
         )
 
     def _register_inline_autocomplete(self, request) -> None:
-        form_class = self.get_formset(request, None).form
+        form_class = self.get_formset(
+            request, getattr(self, "parent_instance", None)
+        ).form
         self.initialize_form_class(form_class, request)
         form_class()
-        self.register_action_autocomplete_views(
-            request, self.get_sbadmin_inline_list_actions_processed(request)
-        )
 
     def get_parent_instance_from_request(self):
         # Try to get parent instance from request referrer

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/admin/site.py

@@ -212,7 +212,7 @@ class SBAdminSite(admin.AdminSite):
                     name="sb_admin_base",
                 ),
                 path(
-                    "<str:view>/<str:action>/<str:modifier>/<int:id>",
+                    "<str:view>/<str:action>/<str:modifier>/<str:object_id>",
                     self.admin_view(SBAdminEntrypointView.as_view()),
                     name="sb_admin_base",
                 ),

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/admin/widgets.py

@@ -36,6 +36,10 @@ from django_smartbase_admin.engine.admin_base_view import (
     SBADMIN_PARENT_INSTANCE_PK_VAR,
     SBADMIN_PARENT_INSTANCE_LABEL_VAR,
 )
+from django_smartbase_admin.engine.const import (
+    ACTION_AUTOCOMPLETE_MODIFIER_SEPARATOR,
+    Action,
+)
 from django_smartbase_admin.engine.filter_widgets import (
     AutocompleteFilterWidget,
     SBAdminTreeWidgetMixin,
@@ -740,8 +744,21 @@ class SBAdminAutocompleteWidget(
         base_id = super().get_id()
         if self.form:
             base_id += f"_{self.form.__class__.__name__}"
+            action_id = getattr(self.form, "sbadmin_action_id", None)
+            if action_id:
+                separator = ACTION_AUTOCOMPLETE_MODIFIER_SEPARATOR
+                base_id = f"{action_id}{separator}{base_id}"
         return base_id
 
+    def get_autocomplete_url(self):
+        request = SBAdminThreadLocalService.get_request()
+        object_id = getattr(getattr(request, "request_data", None), "object_id", None)
+        return self.view.get_action_url(
+            Action.AUTOCOMPLETE.value,
+            modifier=self.get_id(),
+            object_id=object_id,
+        )
+
     def init_widget_dynamic(
         self, form, form_field, field_name, view, request, default_create_data=None
     ):

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/audit/tests/test_action_processing.py

@@ -6,6 +6,7 @@ from unittest.mock import patch
 from django import forms
 from django.core.exceptions import ImproperlyConfigured
 from django.test import RequestFactory, TestCase
+from django.utils.translation import gettext_lazy as _
 
 from django_smartbase_admin.actions.admin_action_list import SBAdminListAction
 from django_smartbase_admin.engine.actions import (
@@ -13,7 +14,10 @@ from django_smartbase_admin.engine.actions import (
     SBAdminFormViewAction,
     SBAdminRowAction,
 )
-from django_smartbase_admin.engine.admin_base_view import SBAdminBaseView
+from django_smartbase_admin.engine.admin_base_view import (
+    SBAdminBaseListView,
+    SBAdminBaseView,
+)
 from django_smartbase_admin.engine.const import (
     IGNORE_LIST_SELECTION,
     MODIFIER_OBJECT_ID,
@@ -41,8 +45,11 @@ class FakeAdminView(SBAdminBaseView):
         self.row_actions = row_actions or []
         self.has_action_permission = has_action_permission
 
-    def get_action_url(self, action, modifier="template"):
-        return f"/actions/{action}/{modifier}/"
+    def get_action_url(self, action, modifier="template", object_id=None):
+        url = f"/actions/{action}/{modifier}/"
+        if object_id is not None:
+            url = f"{url}{object_id}/"
+        return url
 
     def has_permission_for_action(self, request, action):
         return self.has_action_permission
@@ -132,6 +139,51 @@ class RowActionIntegrationTests(TestCase):
     def setUp(self):
         self.request = RequestFactory().get("/")
 
+    def test_permission_gated_target_view_row_action_is_materialized(self):
+        class ArticleAdmin(FakeAdminView, SBAdminBaseListView):
+            def get_sbadmin_row_actions(self, request):
+                if not request.user.has_perm("blog.publish_article"):
+                    return []
+                return [
+                    SBAdminRowAction(
+                        target_view=PublishArticleView,
+                        title=_("Publish"),
+                        icon="Check-correct",
+                        view=self,
+                    )
+                ]
+
+            def get_sbadmin_row_actions_processed(self, request):
+                return self.process_row_actions(
+                    request, self.get_sbadmin_row_actions(request)
+                )
+
+        allowed_request = RequestFactory().get("/")
+        allowed_request.user = SimpleNamespace(
+            has_perm=lambda perm: perm == "blog.publish_article"
+        )
+        denied_request = RequestFactory().get("/")
+        denied_request.user = SimpleNamespace(has_perm=lambda perm: False)
+
+        view = ArticleAdmin()
+        view.init_actions(denied_request)
+        view.init_actions(allowed_request)
+
+        allowed_first_view = ArticleAdmin()
+        allowed_first_view.init_actions(allowed_request)
+        denied_context = TestListAction(
+            allowed_first_view, denied_request
+        ).get_template_data()
+
+        self.assertTrue(hasattr(view, "PublishArticleView"))
+        self.assertNotIn(
+            "_row_actions",
+            [
+                column["field"]
+                for column in denied_context["tabulator_definition"]["tableColumns"]
+            ],
+        )
+
     def test_row_actions_are_processed_into_table_column_and_row_descriptors(self):
         view = FakeAdminView()
         view.row_actions = [
@@ -183,7 +235,7 @@ class RowActionIntegrationTests(TestCase):
             rows[0]["_row_actions"],
             [
                 {
-                    "url": "/actions/PublishArticleView/7/",
+                    "url": "/actions/PublishArticleView/template/7/",
                     "title": "Publish Draft Article",
                     "icon": "Check-correct",
                     "css_class": "btn btn-small btn-only-icon",
@@ -192,7 +244,7 @@ class RowActionIntegrationTests(TestCase):
                     "open_in_new_tab": False,
                 },
                 {
-                    "url": "/actions/action_archive_article/7/",
+                    "url": "/actions/action_archive_article/template/7/",
                     "title": "Archive",
                     "icon": "Delete",
                     "css_class": "btn-icon draft",
@@ -216,6 +268,92 @@ class RowActionIntegrationTests(TestCase):
             ["/articles/8/"],
         )
 
+    def test_row_action_accepts_sub_actions_without_direct_interaction_mode(self):
+        view = FakeAdminView()
+        child = SBAdminRowAction(
+            action_id="action_archive_article",
+            title="Archive",
+            view=view,
+        )
+
+        action = SBAdminRowAction(title="More", sub_actions=[child])
+
+        self.assertEqual(action.sub_actions, [child])
+        self.assertIsNone(action.icon)
+        self.assertIsNone(child.icon)
+        self.assertIsNone(action.url)
+        self.assertIsNone(action.action_id)
+        self.assertIsNone(action.target_view)
+
+    def test_row_action_sub_actions_are_materialized_per_row(self):
+        view = FakeAdminView()
+        view.row_actions = [
+            SBAdminRowAction(
+                title=lambda row: f"Actions for {row['title']}",
+                icon="More",
+                sub_actions=[
+                    SBAdminRowAction(
+                        action_id="action_archive_article",
+                        title=lambda row: f"Archive {row['title']}",
+                        icon="Delete",
+                        view=view,
+                        enabled_if=lambda row: row.get("status") != "archived",
+                    ),
+                    SBAdminRowAction(
+                        url=f"/articles/{MODIFIER_OBJECT_ID}/audit/",
+                        title="Audit",
+                        icon="History",
+                        enabled_field="status",
+                        enabled_value="draft",
+                    ),
+                ],
+            )
+        ]
+        rows = [
+            {"id": 7, "title": "Draft Article", "status": "Draft"},
+            {"id": 8, "title": "Archived Article", "status": "Archived"},
+        ]
+        raw_rows_by_pk = {
+            7: {"id": 7, "title": "Draft Article", "status": "draft"},
+            8: {"id": 8, "title": "Archived Article", "status": "archived"},
+        }
+
+        TestListAction(view, self.request).inject_row_actions(
+            rows, raw_rows_by_pk=raw_rows_by_pk
+        )
+
+        self.assertEqual(
+            rows[0]["_row_actions"],
+            [
+                {
+                    "title": "Actions for Draft Article",
+                    "icon": "More",
+                    "css_class": "btn btn-small btn-only-icon",
+                    "sub_actions": [
+                        {
+                            "url": "/actions/action_archive_article/template/7/",
+                            "title": "Archive Draft Article",
+                            "icon": "Delete",
+                            "css_class": "btn btn-small btn-only-icon",
+                            "open_in_modal": False,
+                            "is_method_action": True,
+                            "open_in_new_tab": False,
+                        },
+                        {
+                            "url": "/articles/7/audit/",
+                            "title": "Audit",
+                            "icon": "History",
+                            "css_class": "btn btn-small btn-only-icon",
+                            "open_in_modal": False,
+                            "is_method_action": False,
+                            "open_in_new_tab": False,
+                        },
+                    ],
+                }
+            ],
+        )
+        self.assertEqual(rows[1]["_row_actions"], [])
+
     def test_nested_plugin_materializes_row_actions_for_child_rows(self):
         request = RequestFactory().get("/")
         request.request_data = SimpleNamespace(additional_data={})
@@ -249,12 +387,12 @@ class RowActionIntegrationTests(TestCase):
 
         self.assertEqual(
             result[0]["_row_actions"][0]["url"],
-            "/actions/action_archive_article/1/",
+            "/actions/action_archive_article/template/1/",
         )
         child = result[0][CHILDREN_FIELD][0]
         self.assertEqual(
             child["_row_actions"][0]["url"],
-            "/actions/action_archive_article/2/",
+            "/actions/action_archive_article/template/2/",
         )
         self.assertTrue(child[LAST_CHILD_FIELD])
 
@@ -275,12 +413,36 @@ class RowActionIntegrationTests(TestCase):
         )
 
         self.assertEqual(action.action_modifier, MODIFIER_OBJECT_ID)
-        self.assertEqual(processed[0].url, "/actions/PublishArticleView/123/")
+        self.assertEqual(processed[0].url, "/actions/PublishArticleView/template/123/")
         self.assertEqual(
             processed_without_object[0].url,
             f"/actions/PublishArticleView/{IGNORE_LIST_SELECTION}/",
         )
 
+    def test_detail_action_without_object_modifier_keeps_template_modifier(self):
+        view = FakeAdminView()
+        action = SBAdminCustomAction(
+            title="Contact",
+            view=view,
+            action_id="contact",
+        )
+
+        processed = view.process_detail_actions(self.request, [action], object_id=123)
+
+        self.assertEqual(processed[0].url, "/actions/contact/template/")
+
+    def test_detail_row_modal_action_infers_current_object(self):
+        view = FakeAdminView()
+        action = SBAdminFormViewAction(
+            target_view=PublishArticleView,
+            title="Publish",
+            view=view,
+        )
+
+        processed = view.process_detail_actions(self.request, [action], object_id=123)
+
+        self.assertEqual(processed[0].url, "/actions/PublishArticleView/template/123/")
+
     def test_form_view_actions_are_registered_before_permission_filtering(self):
         view = FakeAdminView(has_action_permission=False)
         action = SBAdminFormViewAction(
@@ -293,7 +455,40 @@ class RowActionIntegrationTests(TestCase):
 
         self.assertEqual(processed, [])
         self.assertTrue(hasattr(view, "PublishArticleView"))
-        self.assertEqual(action.url, "/actions/PublishArticleView/template/")
+        self.assertIsNone(action.url)
+
+    def test_form_view_action_uses_declared_action_id(self):
+        class CustomActionIdView(RowActionModalView):
+            action_id = "custom_action_id"
+            form_class = forms.Form
+
+        view = FakeAdminView()
+        action = SBAdminFormViewAction(
+            target_view=CustomActionIdView,
+            title="Publish",
+            view=view,
+        )
+
+        processed = view.process_list_actions(self.request, [action])
+
+        self.assertTrue(hasattr(view, "custom_action_id"))
+        self.assertEqual(processed[0].action_id, "custom_action_id")
+        self.assertEqual(processed[0].url, "/actions/custom_action_id/template/")
+        self.assertIsNone(action.action_id)
+        self.assertIsNone(action.url)
+
+    def test_row_action_materializes_explicit_url_without_mutating_definition(self):
+        view = FakeAdminView()
+        action = SBAdminCustomAction(
+            title="Open",
+            url=f"/articles/{MODIFIER_OBJECT_ID}/",
+            action_modifier=MODIFIER_OBJECT_ID,
+        )
+
+        processed = view.process_row_actions(self.request, [action])
+
+        self.assertEqual(action.url, f"/articles/{MODIFIER_OBJECT_ID}/")
+        self.assertEqual(processed[0].url, f"/articles/{MODIFIER_OBJECT_ID}/")
 
     def test_nested_form_view_actions_are_registered(self):
         view = FakeAdminView()
@@ -314,7 +509,8 @@ class RowActionIntegrationTests(TestCase):
         processed = view.process_detail_actions(self.request, [parent], object_id=123)
 
         self.assertEqual(
-            processed[0].sub_actions[0].url, "/actions/PublishArticleView/123/"
+            processed[0].sub_actions[0].url,
+            "/actions/PublishArticleView/template/123/",
         )
         self.assertTrue(hasattr(view, "PublishArticleView"))
 
@@ -330,6 +526,21 @@ class RowActionIntegrationTests(TestCase):
                 url="/articles/__object_id__/",
             )
 
+        with self.assertRaises(ImproperlyConfigured):
+            SBAdminRowAction(
+                title="Broken",
+                icon="Close",
+                url="/articles/__object_id__/",
+                sub_actions=[
+                    SBAdminRowAction(
+                        action_id="action_archive_article",
+                        title="Archive",
+                        icon="Delete",
+                        view=FakeAdminView(),
+                    )
+                ],
+            )
+
 
 class ModalActionIntegrationTests(TestCase):
     def test_modal_action_errors_render_as_form_errors(self):

{django_smartbase_admin-1.4.2 → django_smartbase_admin-1.4.4}/src/django_smartbase_admin/audit/tests/test_delegate_to_action.py

@@ -21,6 +21,7 @@ def _make_request_data(action_name, modifier="template"):
     rd = MagicMock()
     rd.action = action_name
     rd.modifier = modifier
+    rd.object_id = None
     return rd
 
 
@@ -96,6 +97,7 @@ class TestDelegateToAction(TestCase):
         base_view = SBAdminBaseView.__new__(SBAdminBaseView)
         inner = base_view.delegate_to_action_view(mock_action)
         self.assertTrue(getattr(inner, "_is_sbadmin_action", False))
+        self.assertTrue(getattr(inner, "_sbadmin_keep_route_modifier_argument", False))
 
     @patch(PATCH_FROM_REQUEST)
     def test_decorator_permission_propagated_to_action(self, mock_from_request):
@@ -122,6 +124,38 @@ class TestDelegateToAction(TestCase):
         action_arg = view.has_permission_for_action.call_args.args[1]
         self.assertEqual(action_arg.permission, "delete")
 
+    @patch(PATCH_FROM_REQUEST)
+    def test_object_id_is_passed_to_method_action_for_template_modifier(
+        self, mock_from_request
+    ):
+        seen = {}
+
+        @sbadmin_action
+        def row_action(request, modifier):
+            seen["modifier"] = modifier
+            return HttpResponse("ok")
+
+        view = MagicMock()
+        view.row_action = row_action
+        view.has_permission_for_action.return_value = True
+        view.init_view_dynamic = MagicMock()
+
+        rd = _make_request_data("row_action", modifier="template")
+        rd.object_id = "123"
+        rd.selected_view = view
+        mock_from_request.return_value = rd
+
+        response = SBAdminViewService.delegate_to_action(
+            self.factory.get("/"),
+            view="v",
+            action="row_action",
+            modifier="template",
+            object_id="123",
+        )
+
+        self.assertEqual(response.status_code, 200)
+        self.assertEqual(seen["modifier"], "123")
+
 
 class TestDefaultActionPermission(TestCase):
     """Custom actions with no explicit ``permission`` must default to