Skip to content

Commit 715c5c4

Browse files
committed
Conditional inclusion of patron phone number, since field is not required
1 parent f303621 commit 715c5c4

File tree

1 file changed

+2
-2
lines changed
  • modules/os2forms_fbs_handler/src/Client

1 file changed

+2
-2
lines changed

modules/os2forms_fbs_handler/src/Client/Fbs.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ public function createPatronWithGuardian(Patron $patron, Guardian $guardian) {
118118
'pincode' => $patron->pincode,
119119
'preferredPickupBranch' => $patron->preferredPickupBranch,
120120
'name' => 'Unknown Name',
121-
'emailAddresses' => $patron->emailAddresses,
121+
'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
122122
'guardian' => $guardian->toArray(),
123123
'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [
124124
[
@@ -192,7 +192,7 @@ public function updatePatron(Patron $patron): bool {
192192
$payload = [
193193
'patron' => [
194194
'preferredPickupBranch' => $patron->preferredPickupBranch,
195-
'emailAddresses' => $patron->emailAddresses,
195+
'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
196196
'guardianVisibility' => $patron->guardianVisibility,
197197
'receivePostalMail' => $patron->receiveEmail,
198198
'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [

0 commit comments

Comments
 (0)