👌 [#3215] PR Feedback 1

open-formulieren · Jul 11, 2023 · 30ca07b · 30ca07b
1 parent e07b4bf
commit 30ca07b
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 9 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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",