From 30ca07b2d2479db6be14d251802885aef5dd7e84 Mon Sep 17 00:00:00 2001 From: SilviaAmAm Date: Tue, 11 Jul 2023 10:37:44 +0200 Subject: [PATCH] :ok_hand: [#3215] PR Feedback 1 --- docs/manual/forms/form_fields.rst | 4 ++-- .../authentication/contrib/digid_eherkenning_oidc/plugin.py | 2 +- src/openforms/prefill/__init__.py | 5 +---- src/openforms/prefill/tests/test_prefill_hook.py | 4 ++-- 4 files changed, 6 insertions(+), 9 deletions(-) diff --git a/docs/manual/forms/form_fields.rst b/docs/manual/forms/form_fields.rst index 1d0ccc34de..462449fe44 100644 --- a/docs/manual/forms/form_fields.rst +++ b/docs/manual/forms/form_fields.rst @@ -136,8 +136,8 @@ Prefill ------- * **Plugin**: Welke prefill plugin te gebruiken om data op te halen over de persoon/het bedrijf die ingelogd is. -* **Pluginattribuut**: Welke attribuut van de prefill-backend data te gebruiken om deze component te prefillen. -* **Identifier rol**: In het geval dat de gebruiker met DigiD Machtigen en eHerkenning Bewindvoering heeft ingelogd, moeten de gegevens gerelateerd aan het 'hoofd identifier' worden gebruikt, of die gerelateerd aan de gemachtigde persoon? +* **Pluginattribuut**: Welk attribuut van de prefill-backend data te gebruiken om deze component voorin te vullen. +* **Identifier rol**: In het geval dat de gebruiker met DigiD Machtigen of eHerkenning Bewindvoering heeft ingelogd: van welke rol moeten de gegevens gebruikt worden, die van de 'hoofd identifier' worden gebruikt, of die van de gemachtigde persoon? Globale configuratieopties diff --git a/src/openforms/authentication/contrib/digid_eherkenning_oidc/plugin.py b/src/openforms/authentication/contrib/digid_eherkenning_oidc/plugin.py index 56d6a82070..6efc464a81 100644 --- a/src/openforms/authentication/contrib/digid_eherkenning_oidc/plugin.py +++ b/src/openforms/authentication/contrib/digid_eherkenning_oidc/plugin.py @@ -155,7 +155,7 @@ def add_claims_to_sessions_if_not_cosigning(self, claim, request): if not claim or CO_SIGN_PARAMETER in request.GET: return - config = OpenIDConnectDigiDMachtigenConfig.get_solo() + config = self.config_class.get_solo() machtigen_data = request.session[DIGID_MACHTIGEN_OIDC_AUTH_SESSION_KEY] request.session[FORM_AUTH_SESSION_KEY] = { "plugin": self.identifier, diff --git a/src/openforms/prefill/__init__.py b/src/openforms/prefill/__init__.py index de8b903936..ea9d94d38a 100644 --- a/src/openforms/prefill/__init__.py +++ b/src/openforms/prefill/__init__.py @@ -151,10 +151,7 @@ def prefill_variables(submission: "Submission", register=None) -> None: attribute_name = variable.form_variable.prefill_attribute identifier_role = variable.form_variable.prefill_identifier_role - if not grouped_fields[plugin_id].get(identifier_role): - grouped_fields[plugin_id][identifier_role] = [] - - grouped_fields[plugin_id][identifier_role].append(attribute_name) + grouped_fields[plugin_id].setdefault(identifier_role, []).append(attribute_name) results = _fetch_prefill_values(grouped_fields, submission, register) diff --git a/src/openforms/prefill/tests/test_prefill_hook.py b/src/openforms/prefill/tests/test_prefill_hook.py index 7aa5dd5a50..da946e05bd 100644 --- a/src/openforms/prefill/tests/test_prefill_hook.py +++ b/src/openforms/prefill/tests/test_prefill_hook.py @@ -120,7 +120,7 @@ class PrefillHookTests(TransactionTestCase): "openforms.prefill.contrib.haalcentraal.plugin.HaalCentraalPrefill.get_prefill_values", return_value={"naam.voornamen": "John", "naam.geslachtsnaam": "Dodo"}, ) - def test_fetch_values_with_multiple_bsn(self, m_bsn): + def test_fetch_values_with_multiple_people(self, m_haal_centraal): components = [ { "key": "mainPersonName", @@ -165,7 +165,7 @@ def test_fetch_values_with_multiple_bsn(self, m_bsn): "openforms.prefill.contrib.kvk.plugin.KVK_KVKNumberPrefill.get_prefill_values", return_value={"bezoekadres.postcode": "1111 AA"}, ) - def test_fetch_values_with_kvk_and_bsn(self, m_kvk, m_bsn): + def test_fetch_values_with_legal_entity_and_person(self, m_kvk, m_haal_centraal): components = [ { "key": "companyPostcode",