- 🇺🇸United States Chris Burge
@Nadim Hossain - How is this behavior reproducible?
$exploded_value[1]should always be set. If no value is provided by the admin, that should get caught byCustomConfig::validateCustomConfig().
There are actually two problems after updating to php 8.1 -
1. [PHP 8.0] Deprecated function: strcasecmp(): Passing null to parameter #1 ($string1) of type string is deprecated
2. Undefined array key 1
Update to php 8.1 and you will see those error.
In ckeditor_config/src/Plugin/CKEditorPlugin/CustomConfig.php, change from this -
// Loop through config lines and append to editorSettings.
foreach ($config_array as $config_value) {
$exploded_value = explode(" = ", $config_value);
$key = $exploded_value[0];
$value = $exploded_value[1];
to this -
// Loop through config lines and append to editorSettings.
foreach ($config_array as $config_value) {
$exploded_value = explode(" = ", $config_value);
$key = $exploded_value[0];
$value = isset($exploded_value[1]) ? $exploded_value[1] : '';
$value = ($value === 0 || $value === NULL) ? '' : $value;
Needs review
3.0
Code
The issue particularly affects sites running on PHP version 8.1.0 or later.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
@Nadim Hossain - How is this behavior reproducible? $exploded_value[1] should always be set. If no value is provided by the admin, that should get caught by CustomConfig::validateCustomConfig().