Details
-
Bug
-
Resolution: Fixed
-
Major
-
COmanage Registry 4.3.1 (Ruddy Rex MR1)
-
None
Description
The code in petition-attributes.inc in app/View/CoPetitions
} elseif(strpos($fieldName, 'EnrolleeCoPerson.EmailAddress') !== false) { |
$email_list = $args['default']; |
if(!empty($email_list) && !empty($vv_envsource_delimiter)) { |
$delimiter = ""; |
if($vv_envsource_delimiter == AuthProviderEnum::Shibboleth) { |
$delimiter = ";"; |
} elseif($vv_envsource_delimiter == AuthProviderEnum::Simplesamlphp) {
|
$delimiter =","; |
} $emails = explode($delimiter, $email_list);
|
if(is_array($emails) && count($emails) > 1) { |
// XXX We are only keeping the first email |
$args['value'] = $emails[0]; |
}
|
}
|
}
|
initializes $delimiter to an empty string. The logic only changes $delimiter to a non-empty string if the configuration of the EnvSource is for Shibboleth SP or SimpleSAML php. If the configuration is Other then $delimiter remains an empty string.
The bug is that explode() cannot take an empty string as the first argument for PHP 8.x. If the first argument is an empty string then explode() throws a ValueError.
So the default now throws a ValueError.
Attachments
Issue Links
- is related to
-
CO-2736 Fix php 8.1 deprecation errors
- In Progress