[BUGFIX] Fix invalid HTML in ElementBrowser 72/41172/2
authorMarkus Klein <markus.klein@typo3.org>
Mon, 13 Jul 2015 18:55:44 +0000 (20:55 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Tue, 14 Jul 2015 11:13:20 +0000 (13:13 +0200)
An old implementations expected icons to be img-tags, which
is not the case anymore. The regex for replacing parts of the
tag do not work with the span tags anymore and therefore all
calls to the function are dropped.

Resolves: #68086
Releases: master, 6.2
Change-Id: I073e6e2d640ac5f53ae13ec564e6f8903b65e8b8
Reviewed-on: http://review.typo3.org/41172
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/class.browse_links.php
typo3/db_new.php
typo3/move_el.php
typo3/sysext/backend/Classes/Tree/View/BrowseTreeView.php
typo3/sysext/backend/Classes/Tree/View/FolderTreeView.php

index b4e4c1e..5316289 100644 (file)
@@ -127,11 +127,10 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
         * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
-               $content = $this->addTagAttributes($icon, ' title="id=' . $row['uid'] . '"');
                if ($this->ext_showPageId) {
                if ($this->ext_showPageId) {
-                       $content .= '[' . $row['uid'] . ']&nbsp;';
+                       $icon .= '[' . $row['uid'] . ']&nbsp;';
                }
                }
-               return $content;
+               return $icon;
        }
 
 }
        }
 
 }
index 0b73550..0cc6586 100644 (file)
@@ -30,18 +30,6 @@ require __DIR__ . '/init.php';
 class newRecordLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
 
        /**
 class newRecordLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
 
        /**
-        * Inserting uid-information in title-text for an icon
-        *
-        * @param string $icon Icon image
-        * @param array $row Item row
-        * @return string Wrapping icon image.
-        * @todo Define visibility
-        */
-       public function wrapIcon($icon, $row) {
-               return $this->addTagAttributes($icon, ' title="id=' . htmlspecialchars($row['uid']) . '"');
-       }
-
-       /**
         * Determines whether to expand a branch or not.
         * Here the branch is expanded if the current id matches the global id for the listing/new
         *
         * Determines whether to expand a branch or not.
         * Here the branch is expanded if the current id matches the global id for the listing/new
         *
index ca8defb..2213c56 100644 (file)
@@ -27,18 +27,6 @@ require __DIR__ . '/init.php';
  */
 class moveElementLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
 
  */
 class moveElementLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
 
-       /**
-        * Inserting uid-information in title-text for an icon
-        *
-        * @param string $icon Icon image
-        * @param array $row Item row
-        * @return string Wrapping icon image.
-        * @todo Define visibility
-        */
-       public function wrapIcon($icon, $row) {
-               return $this->addTagAttributes($icon, ' title="id=' . htmlspecialchars($row['uid']) . '"');
-       }
-
 }
 
 /**
 }
 
 /**
index c62a8aa..87ce686 100644 (file)
@@ -80,14 +80,13 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
         * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
-               // Add title attribute to input icon tag
-               $theIcon = $this->addTagAttributes($icon, $this->titleAttrib ? $this->titleAttrib . '="' . $this->getTitleAttrib($row) . '"' : '');
                // Wrap icon in click-menu link.
                // Wrap icon in click-menu link.
+               $theIcon = '';
                if (!$this->ext_IconMode) {
                if (!$this->ext_IconMode) {
-                       $theIcon = $GLOBALS['TBE_TEMPLATE']->wrapClickMenuOnIcon($theIcon, $this->treeName, $this->getId($row), 0);
+                       $theIcon = $GLOBALS['TBE_TEMPLATE']->wrapClickMenuOnIcon($icon, $this->treeName, $this->getId($row), 0);
                } elseif ($this->ext_IconMode === 'titlelink') {
                        $aOnClick = 'return jumpTo(\'' . $this->getJumpToParam($row) . '\',this,\'' . $this->domIdPrefix . $this->getId($row) . '\',' . $this->bank . ');';
                } elseif ($this->ext_IconMode === 'titlelink') {
                        $aOnClick = 'return jumpTo(\'' . $this->getJumpToParam($row) . '\',this,\'' . $this->domIdPrefix . $this->getId($row) . '\',' . $this->bank . ');';
-                       $theIcon = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $theIcon . '</a>';
+                       $theIcon = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $icon . '</a>';
                }
                return $theIcon;
        }
                }
                return $theIcon;
        }
index 729c731..2215d75 100644 (file)
@@ -133,16 +133,16 @@ class FolderTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         */
        public function wrapIcon($icon, \TYPO3\CMS\Core\Resource\Folder $folderObject) {
                // Add title attribute to input icon tag
         */
        public function wrapIcon($icon, \TYPO3\CMS\Core\Resource\Folder $folderObject) {
                // Add title attribute to input icon tag
-               $theFolderIcon = $this->addTagAttributes($icon, $this->titleAttrib ? $this->titleAttrib . '="' . $this->getTitleAttrib($folderObject) . '"' : '');
+               $theFolderIcon = '';
                // Wrap icon in click-menu link.
                if (!$this->ext_IconMode) {
                        // Check storage access to wrap with click menu
                        if (!$folderObject instanceof \TYPO3\CMS\Core\Resource\InaccessibleFolder) {
                // Wrap icon in click-menu link.
                if (!$this->ext_IconMode) {
                        // Check storage access to wrap with click menu
                        if (!$folderObject instanceof \TYPO3\CMS\Core\Resource\InaccessibleFolder) {
-                               $theFolderIcon = $GLOBALS['TBE_TEMPLATE']->wrapClickMenuOnIcon($theFolderIcon, $folderObject->getCombinedIdentifier(), '', 0);
+                               $theFolderIcon = $GLOBALS['TBE_TEMPLATE']->wrapClickMenuOnIcon($icon, $folderObject->getCombinedIdentifier(), '', 0);
                        }
                } elseif ($this->ext_IconMode === 'titlelink') {
                        $aOnClick = 'return jumpTo(\'' . $this->getJumpToParam($folderObject) . '\',this,\'' . $this->domIdPrefix . $this->getId($folderObject) . '\',' . $this->bank . ');';
                        }
                } elseif ($this->ext_IconMode === 'titlelink') {
                        $aOnClick = 'return jumpTo(\'' . $this->getJumpToParam($folderObject) . '\',this,\'' . $this->domIdPrefix . $this->getId($folderObject) . '\',' . $this->bank . ');';
-                       $theFolderIcon = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $theFolderIcon . '</a>';
+                       $theFolderIcon = '<a href="#" onclick="' . htmlspecialchars($aOnClick) . '">' . $icon . '</a>';
                }
                return $theFolderIcon;
        }
                }
                return $theFolderIcon;
        }