Feature: сортировка по всем колонкам во всех списках документов

Made-with: Cursor

app/documents/views.py

@@ -32,6 +32,49 @@ from .services import next_number
 logger = logging.getLogger(__name__)
 
 
+class SortableListMixin:
+    """Миксин для сортировки списка по GET-параметрам sort и order."""
+    sort_param = "sort"
+    order_param = "order"
+    sort_fields = {}  # ключ (из GET) -> поле order_by, например "date", "-date" не нужен, order добавляется в get_queryset
+    default_sort = None   # например "date"
+    default_order = "desc" # asc или desc
+
+    def get_sort_fields(self):
+        return self.sort_fields
+
+    def get_queryset(self):
+        qs = super().get_queryset()
+        fields = self.get_sort_fields()
+        if not fields:
+            return qs
+        sort_key = self.request.GET.get(self.sort_param)
+        order = (self.request.GET.get(self.order_param) or "").lower()
+        if order not in ("asc", "desc"):
+            order = self.default_order or "asc"
+        if sort_key and sort_key in fields:
+            order_by_field = fields[sort_key]
+            if order == "desc":
+                order_by_field = "-" + order_by_field
+            return qs.order_by(order_by_field)
+        if self.default_sort and self.default_sort in fields:
+            order_by_field = fields[self.default_sort]
+            if (self.default_order or "asc") == "desc":
+                order_by_field = "-" + order_by_field
+            return qs.order_by(order_by_field)
+        return qs
+
+    def get_context_data(self, **kwargs):
+        ctx = super().get_context_data(**kwargs)
+        ctx["sort_key"] = self.request.GET.get(self.sort_param) or self.default_sort
+        ctx["sort_order"] = (self.request.GET.get(self.order_param) or self.default_order or "asc").lower()
+        if ctx["sort_order"] not in ("asc", "desc"):
+            ctx["sort_order"] = "asc"
+        ctx["sort_param"] = self.sort_param
+        ctx["order_param"] = self.order_param
+        return ctx
+
+
 def set_author(form, request):
     """Подставить автора при создании документа: из профиля пользователя (UserProfile.employee) или сотрудник по имени пользователя."""
     if form.instance.pk:
@@ -56,10 +99,21 @@ def _get_author_for_request(request):
 
 
 # --- Заказы клиентов ---
-class CustomerOrderList(LoginRequiredMixin, ListView):
+class CustomerOrderList(LoginRequiredMixin, SortableListMixin, ListView):
     model = CustomerOrder
     template_name = "documents/customer_order_list.html"
     context_object_name = "object_list"
+    default_sort = "date"
+    default_order = "desc"
+    sort_fields = {
+        "date": "date",
+        "number": "number",
+        "status": "status__name",
+        "order_kind": "order_kind__name",
+        "organization": "organization__name",
+        "client": "client__name",
+        "total_amount": "total_amount",
+    }
 
 class CustomerOrderCreate(LoginRequiredMixin, CreateView):
     model = CustomerOrder
@@ -146,10 +200,20 @@ class CustomerOrderDetail(LoginRequiredMixin, DetailView):
 
 
 # --- Заказы поставщику ---
-class SupplierOrderList(LoginRequiredMixin, ListView):
+class SupplierOrderList(LoginRequiredMixin, SortableListMixin, ListView):
     model = SupplierOrder
     template_name = "documents/supplier_order_list.html"
     context_object_name = "object_list"
+    default_sort = "date"
+    default_order = "desc"
+    sort_fields = {
+        "date": "date",
+        "number": "number",
+        "organization": "organization__name",
+        "supplier": "supplier__name",
+        "total_in_currency": "total_in_currency",
+        "total_amount": "total_amount",
+    }
 
 class SupplierOrderCreate(LoginRequiredMixin, CreateView):
     model = SupplierOrder
@@ -217,10 +281,19 @@ class SupplierOrderDelete(LoginRequiredMixin, DeleteView):
 
 
 # --- Поступление денежных средств ---
-class CashInflowList(LoginRequiredMixin, ListView):
+class CashInflowList(LoginRequiredMixin, SortableListMixin, ListView):
     model = CashInflow
     template_name = "documents/cash_inflow_list.html"
     context_object_name = "object_list"
+    default_sort = "date"
+    default_order = "desc"
+    sort_fields = {
+        "date": "date",
+        "number": "number",
+        "recipient": "recipient__name",
+        "amount": "amount",
+        "customer_order": "customer_order__number",
+    }
 
 class CashInflowCreate(LoginRequiredMixin, CreateView):
     model = CashInflow
@@ -275,10 +348,19 @@ class CashInflowDelete(LoginRequiredMixin, DeleteView):
 
 
 # --- Перемещение денежных средств ---
-class CashTransferList(LoginRequiredMixin, ListView):
+class CashTransferList(LoginRequiredMixin, SortableListMixin, ListView):
     model = CashTransfer
     template_name = "documents/cash_transfer_list.html"
     context_object_name = "object_list"
+    default_sort = "date"
+    default_order = "desc"
+    sort_fields = {
+        "date": "date",
+        "number": "number",
+        "sender": "sender__name",
+        "recipient": "recipient__name",
+        "amount": "amount",
+    }
 
 class CashTransferCreate(LoginRequiredMixin, CreateView):
     model = CashTransfer
@@ -333,10 +415,19 @@ class CashTransferDelete(LoginRequiredMixin, DeleteView):
 
 
 # --- Расход денежных средств ---
-class CashExpenseList(LoginRequiredMixin, ListView):
+class CashExpenseList(LoginRequiredMixin, SortableListMixin, ListView):
     model = CashExpense
     template_name = "documents/cash_expense_list.html"
     context_object_name = "object_list"
+    default_sort = "date"
+    default_order = "desc"
+    sort_fields = {
+        "date": "date",
+        "number": "number",
+        "sender": "sender__name",
+        "amount": "amount",
+        "supplier_order": "supplier_order__number",
+    }
 
 class CashExpenseCreate(LoginRequiredMixin, CreateView):
     model = CashExpense