[#2863] When retrieving eHerkenning-cases, filter on either vestiging…

…snummer or rsin/kvk, but not both
maykinmedia · Nov 7, 2024 · eab2d78 · eab2d78
1 parent a7ce5c9
commit eab2d78
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/src/open_inwoner/cms/products/tests/test_plugin_categories.py b/src/open_inwoner/cms/products/tests/test_plugin_categories.py
@@ -665,7 +665,6 @@ def test_categories_based_on_cases_for_eherkenning_user_with_vestigingsnummer(
                 furl(f"{ZAKEN_ROOT}zaken")
                 .add(
                     {
-                        "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": identifier,
                         "maximaleVertrouwelijkheidaanduiding": VertrouwelijkheidsAanduidingen.beperkt_openbaar,
                         "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": "1234",
                     }

diff --git a/src/open_inwoner/openzaak/clients.py b/src/open_inwoner/openzaak/clients.py
@@ -162,7 +162,6 @@ def fetch_cases_by_kvk_or_rsin(
         config = OpenZaakConfig.get_solo()
 
         params = {
-            "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": kvk_or_rsin,
             "maximaleVertrouwelijkheidaanduiding": config.zaak_max_confidentiality,
         }
 
@@ -172,6 +171,12 @@ def fetch_cases_by_kvk_or_rsin(
                     "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": vestigingsnummer,
                 }
             )
+        else:
+            params.update(
+                {
+                    "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": kvk_or_rsin,
+                }
+            )
 
         if zaak_identificatie:
             params.update({"identificatie": zaak_identificatie})

diff --git a/src/open_inwoner/openzaak/tests/test_cases.py b/src/open_inwoner/openzaak/tests/test_cases.py
@@ -422,7 +422,6 @@ def _setUpMocks(self, m):
                 furl(f"{self.zaken_root}zaken")
                 .add(
                     {
-                        "rol__betrokkeneIdentificatie__nietNatuurlijkPersoon__innNnpId": identifier,
                         "maximaleVertrouwelijkheidaanduiding": VertrouwelijkheidsAanduidingen.beperkt_openbaar,
                         "rol__betrokkeneIdentificatie__vestiging__vestigingsNummer": "1234",
                     }
@@ -840,13 +839,10 @@ def test_list_cases_for_eherkenning_user_with_vestigingsnummer(self, m):
                         else self.eherkenning_user.kvk
                     )
 
-                    self.assertEqual(len(list_zaken_req.qs), 3)
+                    self.assertEqual(len(list_zaken_req.qs), 2)
                     self.assertEqual(
                         list_zaken_req.qs,
                         {
-                            "rol__betrokkeneidentificatie__nietnatuurlijkpersoon__innnnpid": [
-                                identifier
-                            ],
                             "maximalevertrouwelijkheidaanduiding": [
                                 VertrouwelijkheidsAanduidingen.beperkt_openbaar
                             ],