Make sure to not return an empty value.

cdujeu · cdujeu · commit 7064ebd2fd4d · 2016-01-13T10:24:36.000+01:00
diff --git a/core/src/core/classes/class.ConfService.php b/core/src/core/classes/class.ConfService.php
@@ -1447,6 +1447,7 @@ public static function getUserPersonalParameter($parameterName, $userIdOrObject,
         if(is_a($userIdOrObject, "AbstractAjxpUser")){
             $value = $userIdOrObject->personalRole->filterParameterValue($pluginId, $parameterName, AJXP_REPO_SCOPE_ALL, $defaultValue);
             self::$usersParametersCache[$cacheId][$userIdOrObject->getId()] = $value;
+            if(empty($value) && !empty($defaultValue)) $value = $defaultValue;
             return $value;
         }
         // Already in memory cache

Original file line number	Diff line number	Diff line change
`@@ -1447,6 +1447,7 @@ public static function getUserPersonalParameter($parameterName, $userIdOrObject,`
`1447`	`1447`	`if(is_a($userIdOrObject, "AbstractAjxpUser")){`
`1448`	`1448`	`$value = $userIdOrObject->personalRole->filterParameterValue($pluginId, $parameterName, AJXP_REPO_SCOPE_ALL, $defaultValue);`
`1449`	`1449`	`self::$usersParametersCache[$cacheId][$userIdOrObject->getId()] = $value;`
	`1450`	`+ if(empty($value) && !empty($defaultValue)) $value = $defaultValue;`
`1450`	`1451`	`return $value;`
`1451`	`1452`	`}`
`1452`	`1453`	`// Already in memory cache`