[BUGFIX] Followup: [CLEANUP] BackendUtility 16/34416/3
authorFrans Saris <franssaris@gmail.com>
Thu, 20 Nov 2014 22:07:09 +0000 (23:07 +0100)
committerMarkus Klein <klein.t3@reelworx.at>
Thu, 20 Nov 2014 22:27:28 +0000 (23:27 +0100)
With the cleanup for BackendUtility the possible return value of
getProcessedValue() changed. As multiple functions depend on the
fact that the result could be NULL this breaks existing functionality.

Make default return value NULL instead of an empty string.

Resolves: #62414
Releases: master
Change-Id: I01292ec514d5440fac2e4ddaa851607bcf2cb86e
Reviewed-on: http://review.typo3.org/34416
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
typo3/sysext/backend/Classes/Utility/BackendUtility.php

index 331d468..e6a1c93 100644 (file)
@@ -2070,7 +2070,7 @@ class BackendUtility {
         * @param bool $noRecordLookup If set, no records will be looked up, UIDs are just shown.
         * @param int $uid Uid of the current record
         * @param bool $forceResult If BackendUtility::getRecordTitle is used to process the value, this parameter is forwarded.
-        * @return string
+        * @return string|NULL
         */
        static public function getProcessedValue($table, $col, $value, $fixed_lgd_chars = 0, $defaultPassthrough = FALSE, $noRecordLookup = FALSE, $uid = 0, $forceResult = TRUE) {
                if ($col === 'uid') {
@@ -2079,7 +2079,7 @@ class BackendUtility {
                }
                // Check if table and field is configured
                if (!is_array($GLOBALS['TCA'][$table]) || !is_array($GLOBALS['TCA'][$table]['columns'][$col])) {
-                       return '';
+                       return NULL;
                }
                // Depending on the fields configuration, make a meaningful output value.
                $theColConf = $GLOBALS['TCA'][$table]['columns'][$col]['config'];