Skip to content

Commit a1a9dae

Browse files
committed
Simplified validation
1 parent abc2ee5 commit a1a9dae

File tree

1 file changed

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

1 file changed

+4
-4
lines changed

modules/os2forms_fbs_handler/src/Client/Fbs.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,9 +118,9 @@ public function createPatronWithGuardian(Patron $patron, Guardian $guardian) {
118118
'pincode' => $patron->pincode,
119119
'preferredPickupBranch' => $patron->preferredPickupBranch,
120120
'name' => 'Unknown Name',
121-
'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
121+
'emailAddresses' => $patron->emailAddresses ?? [],
122122
'guardian' => $guardian->toArray(),
123-
'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [
123+
'phoneNumbers' => $patron->phoneNumber ? [
124124
[
125125
'receiveNotification' => TRUE,
126126
'phoneNumber' => $patron->phoneNumber,
@@ -192,10 +192,10 @@ public function updatePatron(Patron $patron): bool {
192192
$payload = [
193193
'patron' => [
194194
'preferredPickupBranch' => $patron->preferredPickupBranch,
195-
'emailAddresses' => isset($patron->emailAddresses) && $patron->emailAddresses ? $patron->emailAddresses : [],
195+
'emailAddresses' => $patron->emailAddresses ?? [],
196196
'guardianVisibility' => $patron->guardianVisibility,
197197
'receivePostalMail' => $patron->receiveEmail,
198-
'phoneNumbers' => isset($patron->phoneNumber) && $patron->phoneNumber ? [
198+
'phoneNumbers' => $patron->phoneNumber ? [
199199
[
200200
'receiveNotification' => TRUE,
201201
'phoneNumber' => $patron->phoneNumber,

0 commit comments

Comments
 (0)