Fixed bug #12618: Validation issue in sysext lowlevel: Fix some warnings
authorChristian Kuhn <lolli@schwarzbu.ch>
Mon, 23 Nov 2009 15:03:28 +0000 (15:03 +0000)
committerChristian Kuhn <lolli@schwarzbu.ch>
Mon, 23 Nov 2009 15:03:28 +0000 (15:03 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@6497 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
t3lib/class.t3lib_arraybrowser.php

index 65c9561..05e8ba1 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2009-11-23  Christian Kuhn  <lolli@schwarzbu.ch>
+
+       * Fixed bug #12618: Validation issue in sysext lowlevel: Fix some warnings
+
 2009-11-23  Oliver Hader  <oliver@typo3.org>
 
        * Fixed bug #12699: Upgrade backend login to match t3skin (thanks to T3UXW09 participants)
index cb6dbe9..01e71be 100644 (file)
@@ -105,7 +105,7 @@ class t3lib_arrayBrowser    {
                while (list($key,)=each($arr))  {
                        $a++;
                        $depth = $depth_in.$key;
-                       $goto = substr(md5($depth),0,6);
+                       $goto = 'a' . substr(md5($depth), 0, 6);
 
                        $deeper = (is_array($arr[$key]) && ($this->depthKeys[$depth] || $this->expAll)) ? 1 : 0;
                        $PM = 'join';
@@ -120,7 +120,7 @@ class t3lib_arrayBrowser    {
                                $HTML.=$theIcon;
                        } else {
                                $HTML.=
-                                       ($this->expAll ? '' : '<a name="'.$goto.'" href="'.htmlspecialchars('index.php?node['.$depth.']='.($deeper?0:1).'#'.$goto).'">').
+                                       ($this->expAll ? '' : '<a id="' . $goto . '" href="' . htmlspecialchars('index.php?node[' . $depth . ']=' . ($deeper ? 0 : 1) . '#' . $goto) . '">') .
                                        $theIcon.
                                        ($this->expAll ? '' : '</a>');
                        }
@@ -159,7 +159,11 @@ class t3lib_arrayBrowser   {
         * @return      string          Title string, htmlspecialchars()'ed
         */
        function wrapValue($theValue,$depth)    {
-               return '<b>'.htmlspecialchars($theValue).'</b>';
+               $wrappedValue = '';
+               if (strlen($theValue) > 0) {
+                       $wrappedValue = '<b>' . htmlspecialchars($theValue) . '</b>';
+               }
+               return $wrappedValue;
        }
 
        /**
@@ -178,7 +182,7 @@ class t3lib_arrayBrowser    {
                        // If varname is set:
                if ($this->varName && !$this->dontLinkVar) {
                        $variableName = $this->varName.'[\''.str_replace('.','\'][\'',$depth).'\'] = '.(!t3lib_div::testInt($theValue) ? '\''.addslashes($theValue).'\'' : $theValue).'; ';
-                       $label = '<a href="'.htmlspecialchars('index.php?varname='.$variableName.'#varname').'">'.$label.'</a>';
+                       $label = '<a href="index.php?varname=' . urlencode($variableName) . '#varname">' . $label . '</a>';
                }
 
                        // Return: