Added changes from pr 187

Author: jekuaitk

CHANGELOG.md

@@ -19,6 +19,8 @@ before starting to add changes. Use example [placed in the end of the page](#exa
   - Added condition to maestro notification submission handler
 - [PR-192](https://github.com/OS2Forms/os2forms/pull/192)
   - Fix bug in MitidChildrenSelectAjaxBehaviour.php
+- [PR-187](https://github.com/OS2Forms/os2forms/pull/187)
+  - Avoid double-saving submissions when handling name and address protection.
 
 ## Patch release selvbetjening 4.6.0
 

modules/os2forms_nemid/os2forms_nemid.module

@@ -469,5 +469,4 @@ function os2forms_nemid_submission_set_address_protected(array $form, FormStateI
   $data = $webformSubmission->getData();
   $data['os2forms_nemid_elements_nemid_address_protected'] = TRUE;
   $webformSubmission->setData($data);
-  $webformSubmission->save();
 }

modules/os2forms_nemid/src/Plugin/WebformElement/NemidAddress.php

@@ -47,9 +47,18 @@ public function alterForm(array &$element, array &$form, FormStateInterface $for
     // Only manipulate element on submission create form.
     if (!$webformSubmission->isCompleted()) {
       if ($cprLookupResult && $cprLookupResult->isNameAddressProtected()) {
-        $element['#info_message'] = 'adresse beskyttelse';
         NestedArray::setValue($form['elements'], $element['#webform_parents'], $element);
-        $form['actions']['submit']['#submit'][] = 'os2forms_nemid_submission_set_address_protected';
+
+        // It is important the 'os2forms_nemid_submission_set_address_protected'
+        // submit action is executed before the 'save' action. Otherwise,
+        // submissions are both created and completed, resulting in unexpected
+        // behavior, e.g. handlers being run twice.
+        if (isset($form['actions']['submit']['#submit']) && is_array($form['actions']['submit']['#submit'])) {
+          array_unshift($form['actions']['submit']['#submit'], 'os2forms_nemid_submission_set_address_protected');
+        }
+        else {
+          $form['actions']['submit']['#submit'][] = 'os2forms_nemid_submission_set_address_protected';
+        }
       }
     }
     else {