fix: Set blank fields if family_name, given_name or upm_classif_codes missing for institutional accounts

Author: pablofl01

OpenID.php

@@ -338,9 +338,9 @@ private function _user_from_claim(array $claim): WP_User
             'user_email' => $claim[$this->user_mapping['user_email']] ?? $claim['email'],
             'meta_input' => [
                 'openid_id' => $claim['sub'],
-                'upm_classif_codes' => implode(', ', $claim['upmClassifCode']) ?? '',
-                'given_name' => $claim['given_name'],
-                'family_name' => $claim['family_name'],
+                'upm_classif_codes' => isset($claim['upmClassifCode']) ? implode(', ', $claim['upmClassifCode']) : '',
+                'given_name' => $claim['given_name'] ?? '',
+                'family_name' => $claim['family_name'] ?? '',
             ],
         ];
 
@@ -368,9 +368,9 @@ private function _user_from_claim(array $claim): WP_User
     if ($user) {
         // Update user meta
         update_user_meta($user->ID, 'openid_id', $claim['sub']);
-        update_user_meta($user->ID, 'upm_classif_codes', implode(', ', $claim['upmClassifCode']) ?? '');
-        update_user_meta($user->ID, 'given_name', $claim['given_name']);
-        update_user_meta($user->ID, 'family_name', $claim['family_name']);
+        update_user_meta($user->ID, 'upm_classif_codes', isset($claim['upmClassifCode']) ? implode(', ', $claim['upmClassifCode']) : '');
+        update_user_meta($user->ID, 'given_name', $claim['given_name'] ?? '');
+        update_user_meta($user->ID, 'family_name', $claim['family_name'] ?? '');
 
         // Update additional fields if they exist in the claim
         foreach ($this->user_mapping as $key => $value) {
@@ -395,9 +395,9 @@ private function _user_from_claim(array $claim): WP_User
     if ($user) {
         // Update user meta
         update_user_meta($user->ID, 'openid_id', $claim['sub']);
-        update_user_meta($user->ID, 'upm_classif_codes', implode(', ', $claim['upmClassifCode']) ?? '');
-        update_user_meta($user->ID, 'given_name', $claim['given_name']);
-        update_user_meta($user->ID, 'family_name', $claim['family_name']);
+        update_user_meta($user->ID, 'upm_classif_codes', isset($claim['upmClassifCode']) ? implode(', ', $claim['upmClassifCode']) : '');
+        update_user_meta($user->ID, 'given_name', $claim['given_name'] ?? '');
+        update_user_meta($user->ID, 'family_name', $claim['family_name'] ?? '');
 
         // Update additional fields if they exist in the claim
         foreach ($this->user_mapping as $key => $value) {
@@ -423,9 +423,9 @@ private function _user_from_claim(array $claim): WP_User
         'role' => $this->default_role,
         'meta_input' => [
             'openid_id' => $claim['sub'],
-            'upm_classif_codes' => implode(', ', $claim['upmClassifCode']) ?? '',
-            'given_name' => $claim['given_name'],
-            'family_name' => $claim['family_name'],
+            'upm_classif_codes' => isset($claim['upmClassifCode']) ? implode(', ', $claim['upmClassifCode']) : '',
+            'given_name' => $claim['given_name'] ?? '',
+            'family_name' => $claim['family_name'] ?? '',
         ],
     ];