Dev nshp2 (#132)

* row actions

* row actions

* row actions

* row actions

* row actions

* feat: added fitDataStretchGrow layout and widthGrow support for Tabulator

* feat: added fitDataStretchGrow layout and widthGrow support for Tabulator

* Update translation files for multiple languages

* chore: bump version to 1.1.13b2 and update nested plugin for improved sorting with Subquery

* chore: bump version to 1.1.13b2 and update nested plugin for improved sorting with Subquery

* fix: update nested plugin to use Max aggregation to prevent duplication of rows

* feat: replace fitDataStretchGrow with fitDataFillAvailableSpace in Tabulator layouts

* style: update CSS classes for row action buttons to improve styling consistency

* chore: bump version to 1.1.13 in pyproject.toml

---------

Co-authored-by: oko-vac <oko-vac@users.noreply.github.com>

Author: mihalikv

AGENTS.md

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

pyproject.toml

@@ -35,6 +35,7 @@
     TABLE_PARAMS_SELECTED_FILTER_TYPE,
     ADVANCED_FILTER_DATA_NAME,
     IGNORE_LIST_SELECTION,
+    MODIFIER_OBJECT_ID,
 )
 from django_smartbase_admin.services.views import SBAdminViewService
 from django_smartbase_admin.utils import import_with_injection
@@ -176,6 +177,23 @@ def get_template_data(self):
         }
 
         columns, id_column_name = self.get_tabulator_columns_add_id_column_if_missing()
+        row_actions = self.view.get_sbadmin_row_actions_processed(
+            self.threadsafe_request
+        )
+        if row_actions:
+            columns.append(
+                {
+                    "field": "_row_actions",
+                    "title": "",
+                    "headerSort": False,
+                    "frozen": True,
+                    "sbadminKeepDataWidth": True,
+                    "sbadminSystemColumn": True,
+                    "hozAlign": "right",
+                    "formatter": "sbadminRowActionsFormatter",
+                    "cssClass": "row-actions-cell",
+                }
+            )
         tabulator_definition = (
             self.tabulator_definition
             or self.view.get_tabulator_definition(self.threadsafe_request)
@@ -184,8 +202,10 @@ def get_template_data(self):
         tabulator_definition["tableIdColumnName"] = id_column_name
         tabulator_definition["constants"] = constants
 
-        list_actions = self.list_actions or self.view._get_sbadmin_list_actions(
-            self.threadsafe_request
+        list_actions = (
+            self.view.process_list_actions(self.threadsafe_request, self.list_actions)
+            if self.list_actions is not None
+            else self.view.get_sbadmin_list_actions_processed(self.threadsafe_request)
         )
 
         context_data.update(
@@ -207,9 +227,7 @@ def get_template_data(self):
                 "search_field_placeholder": self.view.get_search_field_placeholder(
                     self.threadsafe_request
                 ),
-                "list_actions": self.view.process_actions(
-                    self.threadsafe_request, list_actions
-                ),
+                "list_actions": list_actions,
                 "list_selection_actions": self.view.get_sbadmin_list_selection_actions_grouped(
                     self.threadsafe_request
                 ),
@@ -456,6 +474,9 @@ def get_data(self, page_num=None, page_size=None, additional_filter=None):
                 data=data,
             )
 
+        raw_rows_by_pk = {row[self.get_pk_field().name]: dict(row) for row in data}
+        self.inject_row_actions(data, raw_rows_by_pk=raw_rows_by_pk)
+
         return {
             "last_page": math.ceil(total_count / page_size),
             "data": data,
@@ -490,6 +511,48 @@ def process_final_data(self, final_data: list[dict[str, Any]]) -> None:
                     value = escape(value)
                 row[field_key] = value
 
+    def inject_row_actions(
+        self, final_data: list[dict[str, Any]], raw_rows_by_pk: dict | None = None
+    ) -> None:
+        actions = self.view.get_sbadmin_row_actions_processed(self.threadsafe_request)
+        if not actions:
+            return
+        pk_field = self.get_pk_field().name
+        for row in final_data:
+            obj_id = row.get(pk_field)
+            raw_row = (raw_rows_by_pk or {}).get(obj_id)
+            row["_row_actions"] = [
+                descriptor
+                for action in actions
+                if (
+                    descriptor := self._materialize_row_action(
+                        action, row, obj_id, raw_row=raw_row
+                    )
+                )
+                is not None
+            ]
+
+    def _materialize_row_action(
+        self, action, row: dict[str, Any], obj_id: Any, raw_row=None
+    ) -> dict[str, Any] | None:
+        action_row = raw_row or row
+        if not action.is_enabled(action_row):
+            return None
+
+        url = action.url
+        if url and MODIFIER_OBJECT_ID in url:
+            url = url.replace(MODIFIER_OBJECT_ID, str(obj_id))
+
+        return {
+            "url": url,
+            "title": str(action.get_title(action_row) or ""),
+            "icon": action.get_icon(action_row),
+            "css_class": action.get_css_class(action_row) or "",
+            "open_in_modal": bool(action.open_in_modal),
+            "is_method_action": bool(action.action_id) and not action.open_in_modal,
+            "open_in_new_tab": bool(action.open_in_new_tab),
+        }
+
     def get_json_data(self):
         return self.get_data()
 

src/django_smartbase_admin/actions/admin_action_list.py

@@ -54,7 +54,11 @@
 )
 from django_smartbase_admin.engine.actions import SBAdminCustomAction
 from django_smartbase_admin.services.thread_local import SBAdminThreadLocalService
-from django_smartbase_admin.utils import FormFieldsetMixin, is_modal
+from django_smartbase_admin.utils import (
+    FormFieldsetMixin,
+    is_modal,
+    render_notifications,
+)
 
 parler_enabled = None
 try:
@@ -146,6 +150,7 @@
     OBJECT_ID_PLACEHOLDER,
     TRANSLATIONS_SELECTED_LANGUAGES,
     ROW_CLASS_FIELD,
+    TABLE_RELOAD_DATA_EVENT_NAME,
 )
 from django_smartbase_admin.services.translations import SBAdminTranslationsService
 from django_smartbase_admin.services.views import SBAdminViewService
@@ -1033,6 +1038,16 @@ def get_modal_save_response(cls, request, obj):
         trigger_client_event(response, "hideModal", {"elt": "sb-admin-modal"})
         return response
 
+    def build_action_response(
+        self, request, *, reload_table=True, hide_modal=False
+    ) -> HttpResponse:
+        response = HttpResponse(render_notifications(request))
+        if hide_modal:
+            trigger_client_event(response, "hideModal", {})
+        if reload_table:
+            trigger_client_event(response, TABLE_RELOAD_DATA_EVENT_NAME, {})
+        return response
+
     def response_add(self, request, obj, post_url_continue=None):
         if is_modal(request):
             return self.get_modal_save_response(request, obj)
@@ -1111,6 +1126,11 @@ def get_queryset(self, request=None) -> QuerySet:
     def get_sbadmin_inline_list_actions(self, request) -> list:
         return [*(self.sbadmin_inline_list_actions or [])]
 
+    def get_sbadmin_inline_list_actions_processed(self, request) -> list:
+        return self.process_inline_actions(
+            request, self.get_sbadmin_inline_list_actions(request)
+        )
+
     def get_action_url(self, action, modifier="template") -> str:
         return reverse(
             "sb_admin:sb_admin_base",
@@ -1126,6 +1146,9 @@ def register_autocomplete_views(self, request) -> None:
         form_class = self.get_formset(request, 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
@@ -1170,7 +1193,9 @@ def get_context_data(self, request) -> dict[str, Any]:
                 self.opts.model_name,
             )
         context_data = {
-            "inline_list_actions": self.get_sbadmin_inline_list_actions(request),
+            "inline_list_actions": self.get_sbadmin_inline_list_actions_processed(
+                request
+            ),
             "is_sortable_active": is_sortable_active,
             "add_url": add_url,
         }