[BUGFIX] Backend users: Exception in element browser with PHP 7 08/46008/2
authorMichael Oehlhof <typo3@oehlhof.de>
Sat, 16 Jan 2016 11:43:38 +0000 (12:43 +0100)
committerMorton Jonuschat <m.jonuschat@mojocode.de>
Sat, 16 Jan 2016 16:46:51 +0000 (17:46 +0100)
Make the Declaration of
TYPO3\CMS\Recordlist\Tree\View\ElementBrowserPageTreeView::wrapTitle()
compatible with
TYPO3\CMS\Backend\Tree\View\ElementBrowserPageTreeView::wrapTitle()

Resolves: #72767
Releases: master, 7.6
Change-Id: Ic855be1b3e8020a28a7c44c21709d75a3a04e3b5
Reviewed-on: https://review.typo3.org/46000
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
(cherry picked from commit d3983a1e3022f422dc009d6061ba529d04112ee3)
Reviewed-on: https://review.typo3.org/46008
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
typo3/sysext/recordlist/Classes/Tree/View/ElementBrowserPageTreeView.php

index e236e26..7f6ad5a 100644 (file)
@@ -39,22 +39,22 @@ class ElementBrowserPageTreeView extends \TYPO3\CMS\Backend\Tree\View\ElementBro
      * Wrapping the title in a link, if applicable.
      *
      * @param string $title Title, ready for output.
-     * @param array $row The record
+     * @param array $v The record
      * @param bool $ext_pArrPages If set, pages clicked will return immediately, otherwise reload page.
      * @return string Wrapping title string.
      */
-    public function wrapTitle($title, $row, $ext_pArrPages)
+    public function wrapTitle($title, $v, $ext_pArrPages = false)
     {
-        if ($ext_pArrPages && $row['uid']) {
+        if ($ext_pArrPages && $v['uid']) {
             $iconFactory = GeneralUtility::makeInstance(IconFactory::class);
-            $ficon = $iconFactory->getIconForRecord('pages', $row, Icon::SIZE_SMALL)->render();
-            $out = '<span data-uid="' . htmlspecialchars($row['uid']) . '" data-table="pages" data-title="' . htmlspecialchars($row['title']) . '" data-icon="' . htmlspecialchars($ficon) . '">';
+            $ficon = $iconFactory->getIconForRecord('pages', $v, Icon::SIZE_SMALL)->render();
+            $out = '<span data-uid="' . htmlspecialchars($v['uid']) . '" data-table="pages" data-title="' . htmlspecialchars($v['title']) . '" data-icon="' . htmlspecialchars($ficon) . '">';
             $out .= '<a href="#" data-close="1">' . $title . '</a>';
             $out .= '</span>';
             return $out;
         }
 
-        $parameters = GeneralUtility::implodeArrayForUrl('', $this->linkParameterProvider->getUrlParameters(['pid' => $row['uid']]));
+        $parameters = GeneralUtility::implodeArrayForUrl('', $this->linkParameterProvider->getUrlParameters(['pid' => $v['uid']]));
         return '<a href="#" onclick="return jumpToUrl(' . htmlspecialchars(GeneralUtility::quoteJSvalue($this->getThisScript() . ltrim($parameters, '&'))) . ');">' . $title . '</a>';
     }
 }