From bf20611fd43a609dd1d049c6b3ef820e93d4976d Mon Sep 17 00:00:00 2001 From: Noah Hilverling Date: Mon, 13 Feb 2017 16:52:52 +0100 Subject: [PATCH] ConfigForm: Fix that empty values are not handled correctly refs #2751 --- application/forms/ConfigForm.php | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/application/forms/ConfigForm.php b/application/forms/ConfigForm.php index 1c79600b5..8ee00884d 100644 --- a/application/forms/ConfigForm.php +++ b/application/forms/ConfigForm.php @@ -58,7 +58,7 @@ class ConfigForm extends Form { $sections = array(); foreach ($this->getValues() as $sectionAndPropertyName => $value) { - if (empty($value)) { + if ($value === '' || (is_array($value) && empty($value))) { $value = null; // Causes the config writer to unset it } @@ -137,8 +137,6 @@ class ConfigForm extends Form */ public static function transformEmptyValuesToNull(array $values) { - return array_map(function ($v) { - return empty($v) ? null : $v; - }, $values); + return array_map(function ($v) { return ($v === '' || (is_array($v) && empty($v))) ? null : $v; }, $values); } }