Merge pull request #1461 from open-zaak/feature/1460-zaak-order-new-p…

…arams Feature/1460 zaak order new params
open-zaak · Sep 19, 2023 · 49b6444 · 49b6444
2 parents 330aaab + f8dd4b8
commit 49b6444
Show file tree

Hide file tree

Showing 4 changed files with 67 additions and 11 deletions.
diff --git a/src/openzaak/components/zaken/api/filters.py b/src/openzaak/components/zaken/api/filters.py
@@ -87,7 +87,14 @@ class ZaakFilter(FilterSet):
         help_text=get_help_text("zaken.OrganisatorischeEenheid", "identificatie"),
     )
     ordering = filters.OrderingFilter(
-        fields=("startdatum", "einddatum", "publicatiedatum", "archiefactiedatum",),
+        fields=(
+            "startdatum",
+            "einddatum",
+            "publicatiedatum",
+            "archiefactiedatum",
+            "registratiedatum",
+            "identificatie",
+        ),
         help_text=_("Het veld waarop de resultaten geordend worden."),
     )
 

diff --git a/src/openzaak/components/zaken/openapi.yaml b/src/openzaak/components/zaken/openapi.yaml
@@ -3082,6 +3082,10 @@ paths:
             - -publicatiedatum
             - archiefactiedatum
             - -archiefactiedatum
+            - registratiedatum
+            - -registratiedatum
+            - identificatie
+            - -identificatie
         style: form
         explode: false
       - name: page
@@ -7828,7 +7832,15 @@ components:
 
             * `archiefactiedatum` - Archiefactiedatum
 
-            * `-archiefactiedatum` - Archiefactiedatum (descending)'
+            * `-archiefactiedatum` - Archiefactiedatum (descending)
+
+            * `registratiedatum` - Registratiedatum
+
+            * `-registratiedatum` - Registratiedatum (descending)
+
+            * `identificatie` - Identificatie
+
+            * `-identificatie` - Identificatie (descending)'
           type: string
           enum:
           - startdatum
@@ -7839,6 +7851,10 @@ components:
           - -publicatiedatum
           - archiefactiedatum
           - -archiefactiedatum
+          - registratiedatum
+          - -registratiedatum
+          - identificatie
+          - -identificatie
     Wijzigingen:
       type: object
       properties:

diff --git a/src/openzaak/components/zaken/swagger2.0.json b/src/openzaak/components/zaken/swagger2.0.json
@@ -4107,7 +4107,11 @@
                                     "publicatiedatum",
                                     "-publicatiedatum",
                                     "archiefactiedatum",
-                                    "-archiefactiedatum"
+                                    "-archiefactiedatum",
+                                    "registratiedatum",
+                                    "-registratiedatum",
+                                    "identificatie",
+                                    "-identificatie"
                                 ]
                             }
                         },
@@ -9528,7 +9532,7 @@
                 },
                 "ordering": {
                     "title": "Ordering",
-                    "description": "Het veld waarop de resultaten geordend worden.\n\nUitleg bij mogelijke waarden:\n\n* `startdatum` - Startdatum\n* `-startdatum` - Startdatum (descending)\n* `einddatum` - Einddatum\n* `-einddatum` - Einddatum (descending)\n* `publicatiedatum` - Publicatiedatum\n* `-publicatiedatum` - Publicatiedatum (descending)\n* `archiefactiedatum` - Archiefactiedatum\n* `-archiefactiedatum` - Archiefactiedatum (descending)",
+                    "description": "Het veld waarop de resultaten geordend worden.\n\nUitleg bij mogelijke waarden:\n\n* `startdatum` - Startdatum\n* `-startdatum` - Startdatum (descending)\n* `einddatum` - Einddatum\n* `-einddatum` - Einddatum (descending)\n* `publicatiedatum` - Publicatiedatum\n* `-publicatiedatum` - Publicatiedatum (descending)\n* `archiefactiedatum` - Archiefactiedatum\n* `-archiefactiedatum` - Archiefactiedatum (descending)\n* `registratiedatum` - Registratiedatum\n* `-registratiedatum` - Registratiedatum (descending)\n* `identificatie` - Identificatie\n* `-identificatie` - Identificatie (descending)",
                     "type": "string",
                     "enum": [
                         "startdatum",
@@ -9538,7 +9542,11 @@
                         "publicatiedatum",
                         "-publicatiedatum",
                         "archiefactiedatum",
-                        "-archiefactiedatum"
+                        "-archiefactiedatum",
+                        "registratiedatum",
+                        "-registratiedatum",
+                        "identificatie",
+                        "-identificatie"
                     ]
                 }
             }

diff --git a/src/openzaak/components/zaken/tests/test_zaken.py b/src/openzaak/components/zaken/tests/test_zaken.py
@@ -620,14 +620,15 @@ def test_sort_datum_ascending(self):
             "einddatum",
             "publicatiedatum",
             "archiefactiedatum",
+            "registratiedatum",
         ]
 
         for param in sorting_params:
             with self.subTest(param=param):
                 Zaak.objects.all().delete()
-                ZaakFactory.create(**{param: "2019-01-01"}, zaaktype=self.zaaktype)
-                ZaakFactory.create(**{param: "2019-03-01"}, zaaktype=self.zaaktype)
-                ZaakFactory.create(**{param: "2019-02-01"}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 1, 1)}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 3, 1)}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 2, 1)}, zaaktype=self.zaaktype)
                 url = reverse("zaak-list")
 
                 response = self.client.get(url, {"ordering": param}, **ZAAK_READ_KWARGS)
@@ -646,14 +647,15 @@ def test_sort_datum_descending(self):
             "einddatum",
             "publicatiedatum",
             "archiefactiedatum",
+            "registratiedatum",
         ]
 
         for param in sorting_params:
             with self.subTest(param=param):
                 Zaak.objects.all().delete()
-                ZaakFactory.create(**{param: "2019-01-01"}, zaaktype=self.zaaktype)
-                ZaakFactory.create(**{param: "2019-03-01"}, zaaktype=self.zaaktype)
-                ZaakFactory.create(**{param: "2019-02-01"}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 1, 1)}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 3, 1)}, zaaktype=self.zaaktype)
+                ZaakFactory.create(**{param: date(2019, 2, 1)}, zaaktype=self.zaaktype)
                 url = reverse("zaak-list")
 
                 response = self.client.get(
@@ -668,6 +670,29 @@ def test_sort_datum_descending(self):
                 self.assertEqual(data[1][param], "2019-02-01")
                 self.assertEqual(data[2][param], "2019-01-01")
 
+    def test_sort_identificatie(self):
+        ZaakFactory.create(identificatie="ZAAK1", zaaktype=self.zaaktype)
+        ZaakFactory.create(identificatie="ZAAK2", zaaktype=self.zaaktype)
+        url = reverse("zaak-list")
+
+        with self.subTest(param="identificatie"):
+            response = self.client.get(
+                url, {"ordering": "identificatie"}, **ZAAK_READ_KWARGS
+            )
+
+            self.assertEqual(response.status_code, status.HTTP_200_OK)
+            self.assertEqual(response.json()["results"][0]["identificatie"], "ZAAK1")
+            self.assertEqual(response.json()["results"][1]["identificatie"], "ZAAK2")
+
+        with self.subTest(param="-identificatie"):
+            response = self.client.get(
+                url, {"ordering": "-identificatie"}, **ZAAK_READ_KWARGS
+            )
+
+            self.assertEqual(response.status_code, status.HTTP_200_OK)
+            self.assertEqual(response.json()["results"][0]["identificatie"], "ZAAK2")
+            self.assertEqual(response.json()["results"][1]["identificatie"], "ZAAK1")
+
 
 class ZakenFilterTests(JWTAuthMixin, APITestCase):
     heeft_alle_autorisaties = True