Feature/Phone number in fbs handler

CHANGELOG.md

@@ -12,6 +12,7 @@ before starting to add changes. Use example [placed in the end of the page](#exa
 ## [Unreleased]
 
 - Removed modules ldap_auth, logging_alerts, maillog
+- Updated `os2forms_fbs_handler` to include and correct set phone number.
 
 ## [3.21.2] 2025-01-07
 

modules/os2forms_fbs_handler/src/Client/FBS.php → modules/os2forms_fbs_handler/src/Client/Fbs.php

@@ -11,7 +11,7 @@
 /**
  * Minimalistic client to create user with guardians at FBS.
  */
-class FBS {
+class Fbs {
 
   /**
    * FBS session key.
@@ -118,8 +118,14 @@ public function createPatronWithGuardian(Patron $patron, Guardian $guardian) {
       'pincode' => $patron->pincode,
       'preferredPickupBranch' => $patron->preferredPickupBranch,
       'name' => 'Unknown Name',
-      'emailAddresses' => $patron->emailAddresses,
+      'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
       'guardian' => $guardian->toArray(),
+      'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [
+        [
+          'receiveNotification' => TRUE,
+          'phoneNumber' => $patron->phoneNumber,
+        ],
+      ] : [],
     ];
 
     return $this->request($uri, $payload);
@@ -148,12 +154,13 @@ public function getPatron(string $patronId): ?Patron {
         (bool) $json->patron->receiveSms,
         (bool) $json->patron->receivePostalMail,
         $json->patron->notificationProtocols,
-        $json->patron->phoneNumber,
         is_null($json->patron->onHold) ? $json->patron->onHold : (array) $json->patron->onHold,
         $json->patron->preferredLanguage,
         (bool) $json->patron->guardianVisibility,
         $json->patron->defaultInterestPeriod,
         (bool) $json->patron->resident,
+        $json->patron->phoneNumber,
+
         [
           [
             'emailAddress' => $json->patron->emailAddress,
@@ -185,15 +192,15 @@ public function updatePatron(Patron $patron): bool {
     $payload = [
       'patron' => [
         'preferredPickupBranch' => $patron->preferredPickupBranch,
-        'emailAddresses' => $patron->emailAddresses,
+        'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
         'guardianVisibility' => $patron->guardianVisibility,
         'receivePostalMail' => $patron->receiveEmail,
-        'phoneNumbers' => [
+        'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [
           [
             'receiveNotification' => TRUE,
             'phoneNumber' => $patron->phoneNumber,
           ],
-        ],
+        ] : [],
       ],
       'pincodeChange' => [
         'pincode' => $patron->pincode,

modules/os2forms_fbs_handler/src/Client/Model/Patron.php

@@ -19,13 +19,13 @@ public function __construct(
     public readonly ?bool $receiveSms = FALSE,
     public readonly ?bool $receivePostalMail = FALSE,
     public readonly ?array $notificationProtocols = NULL,
-    public readonly ?string $phoneNumber = NULL,
     public readonly ?array $onHold = NULL,
     public readonly ?string $preferredLanguage = NULL,
     public readonly ?bool $guardianVisibility = NULL,
     public readonly ?int $defaultInterestPeriod = NULL,
     public readonly ?bool $resident = NULL,
     // Allow these properties below to be updatable.
+    public ?string $phoneNumber = NULL,
     public ?array $emailAddresses = NULL,
     public ?bool $receiveEmail = NULL,
     public ?string $preferredPickupBranch = NULL,

modules/os2forms_fbs_handler/src/Plugin/AdvancedQueue/JobType/FbsCreateUser.php

@@ -8,7 +8,7 @@
 use Drupal\advancedqueue\Job;
 use Drupal\advancedqueue\JobResult;
 use Drupal\advancedqueue\Plugin\AdvancedQueue\JobType\JobTypeBase;
-use Drupal\os2forms_fbs_handler\Client\FBS;
+use Drupal\os2forms_fbs_handler\Client\Fbs;
 use Drupal\os2forms_fbs_handler\Client\Model\Guardian;
 use Drupal\os2forms_fbs_handler\Client\Model\Patron;
 use Drupal\os2web_audit\Service\Logger;
@@ -84,7 +84,7 @@ public function process(Job $job): JobResult {
       $config = $payload['configuration'];
 
       try {
-        $fbs = new FBS($this->client, $config['endpoint_url'], $config['agency_id'], $config['username'], $config['password']);
+        $fbs = new Fbs($this->client, $config['endpoint_url'], $config['agency_id'], $config['username'], $config['password']);
 
         // Log into FBS and obtain session.
         $fbs->login();
@@ -116,6 +116,9 @@ public function process(Job $job): JobResult {
                 'receiveNotification' => TRUE,
               ],
             ];
+            if (isset($data['barn_tlf'])) {
+              $patron->phoneNumber = $data['barn_tlf'];
+            }
             $patron->receiveEmail = TRUE;
             $patron->pincode = $data['pinkode'];
 
@@ -136,6 +139,9 @@ public function process(Job $job): JobResult {
           $patron->receiveEmail = TRUE;
           $patron->personId = $data['barn_cpr'];
           $patron->pincode = $data['pinkode'];
+          if (isset($data['barn_tlf'])) {
+            $patron->phoneNumber = $data['barn_tlf'];
+          }
 
           $fbs->createPatronWithGuardian($patron, $guardian);
         }

modules/os2forms_fbs_handler/src/Plugin/WebformHandler/FbsWebformHandler.php

@@ -183,6 +183,7 @@ public function postSave(WebformSubmissionInterface $webform_submission, $update
       'afhentningssted',
       'barn_cpr',
       'barn_mail',
+      'barn_tlf',
       'cpr',
       'email',
       'navn',