[BUGFIX] Change highlight color on folder change 74/36374/2
authorBenjamin Mack <benni@typo3.org>
Thu, 29 Jan 2015 09:46:44 +0000 (10:46 +0100)
committerTymoteusz Motylewski <t.motylewski@gmail.com>
Thu, 29 Jan 2015 17:36:06 +0000 (18:36 +0100)
Clicking on folder in filelist results in JS error.

When using the filelist main frame
for navigation through the folder structure
there is still some leftover from the old
code, which changes the highlighted
item when clicking on a folder.

The patch solves the issue by properly
addressing the navigation frame and
the JS method.

This part is actually a leftover from pre-4.2
code which was not changed by me in 4.2
with the introduction of the AJAX-based
folder tree with prototype.js.

Resolves: #64572
Releases: master, 6.2
Change-Id: Iff7c7aff07454e428fe78194532ba95df2bd25d0
Reviewed-on: http://review.typo3.org/36342
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
(cherry picked from commit 68e1fdc7c54c7ba426e04b4f19a28c9b89196a93)
Reviewed-on: http://review.typo3.org/36374

typo3/sysext/filelist/Classes/FileList.php

index 6b4aec2..ecdc221 100644 (file)
@@ -518,7 +518,7 @@ class FileList extends \TYPO3\CMS\Backend\RecordList\AbstractRecordList {
         */
        public function linkWrapDir($title, \TYPO3\CMS\Core\Resource\Folder $folderObject) {
                $href = $this->backPath . $this->script . '&id=' . rawurlencode($folderObject->getCombinedIdentifier());
-               $onclick = ' onclick="' . htmlspecialchars(('top.content.nav_frame.hilight_row("file","folder' . GeneralUtility::md5int($folderObject->getCombinedIdentifier()) . '_"+top.fsMod.currentBank)')) . '"';
+               $onclick = ' onclick="' . htmlspecialchars(('top.document.getElementsByName("navigation")[0].contentWindow.Tree.highlightActiveItem("file","folder' . GeneralUtility::md5int($folderObject->getCombinedIdentifier()) . '_"+top.fsMod.currentBank)')) . '"';
                // Sometimes $code contains plain HTML tags. In such a case the string should not be modified!
                if ((string)$title === strip_tags($title)) {
                        return '<a href="' . htmlspecialchars($href) . '"' . $onclick . ' title="' . htmlspecialchars($title) . '">' . GeneralUtility::fixed_lgd_cs($title, $this->fixedL) . '</a>';