[BUGFIX] Removed clickMenu code around header icon in info window 58/31358/2
authorFrans Saris <franssaris@gmail.com>
Thu, 19 Jun 2014 15:58:41 +0000 (17:58 +0200)
committerMarkus Klein <klein.t3@mfc-linz.at>
Sat, 5 Jul 2014 16:56:23 +0000 (18:56 +0200)
The header icon in the ElementInfo screen has a click link added
but this doesn't work and doesn't make sense as shown links would
be in the wrong context.

This patch removes the javasript to open the clickmenu from the icons.

Resolves: #56250
Releases: 6.3, 6.2
Change-Id: I2b907188958f166a129617da36585c07be3a143c
Reviewed-on: https://review.typo3.org/31358
Reviewed-by: Markus Klein
Tested-by: Markus Klein
typo3/sysext/backend/Classes/Controller/ContentElement/ElementInformationController.php
typo3/sysext/backend/Classes/Template/DocumentTemplate.php

index 24af331..53a1428 100644 (file)
@@ -217,13 +217,13 @@ class ElementInformationController {
        protected function renderPageTitle() {
                if ($this->type === 'folder') {
                        $table = $GLOBALS['LANG']->sL('LLL:EXT:lang/locallang_common.xlf:folder');
-                       $title = $this->doc->getResourceHeader($this->folderObject, array(' ', ''));
+                       $title = $this->doc->getResourceHeader($this->folderObject, array(' ', ''), FALSE);
                } elseif ($this->type === 'file') {
                        $table = $GLOBALS['LANG']->sL($GLOBALS['TCA'][$this->table]['ctrl']['title']);
-                       $title = $this->doc->getResourceHeader($this->fileObject, array(' ', ''));
+                       $title = $this->doc->getResourceHeader($this->fileObject, array(' ', ''), FALSE);
                } else {
                        $table = $GLOBALS['LANG']->sL($GLOBALS['TCA'][$this->table]['ctrl']['title']);
-                       $title = $this->doc->getHeader($this->table, $this->row, $this->pageinfo['_thePath'], 1, array(' ', ''));
+                       $title = $this->doc->getHeader($this->table, $this->row, $this->pageinfo['_thePath'], 1, array(' ', ''), FALSE);
                }
 
                return '<h1>' .
index 02b9175..4fd9e8c 100644 (file)
@@ -533,10 +533,11 @@ function jumpToUrl(URL) {
         * @param string $path Alt text
         * @param boolean $noViewPageIcon Set $noViewPageIcon TRUE if you don't want a magnifier-icon for viewing the page in the frontend
         * @param array $tWrap is an array with indexes 0 and 1 each representing HTML-tags (start/end) which will wrap the title
+        * @param bool $enableClickMenu If TRUE, render click menu code around icon image
         * @return string HTML content
         * @todo Define visibility
         */
-       public function getHeader($table, $row, $path, $noViewPageIcon = FALSE, $tWrap = array('', '')) {
+       public function getHeader($table, $row, $path, $noViewPageIcon = FALSE, $tWrap = array('', ''), $enableClickMenu = TRUE) {
                $viewPage = '';
                if (is_array($row) && $row['uid']) {
                        $iconImgTag = IconUtility::getSpriteIconForRecord($table, $row, array('title' => htmlspecialchars($path)));
@@ -549,7 +550,12 @@ function jumpToUrl(URL) {
                        $iconImgTag = IconUtility::getSpriteIcon('apps-pagetree-page-domain', array('title' => htmlspecialchars($path)));
                        $title = $GLOBALS['TYPO3_CONF_VARS']['SYS']['sitename'];
                }
-               return '<span class="typo3-moduleHeader">' . $this->wrapClickMenuOnIcon($iconImgTag, $table, $row['uid']) . $viewPage . $tWrap[0] . htmlspecialchars(GeneralUtility::fixed_lgd_cs($title, 45)) . $tWrap[1] . '</span>';
+
+               if ($enableClickMenu) {
+                       $iconImgTag = $this->wrapClickMenuOnIcon($iconImgTag, $table, $row['uid']);
+               }
+
+               return '<span class="typo3-moduleHeader">' . $iconImgTag . $viewPage . $tWrap[0] . htmlspecialchars(GeneralUtility::fixed_lgd_cs($title, 45)) . $tWrap[1] . '</span>';
        }
 
        /**
@@ -558,13 +564,14 @@ function jumpToUrl(URL) {
         *
         * @param \TYPO3\CMS\Core\Resource\ResourceInterface $resource
         * @param array $tWrap is an array with indexes 0 and 1 each representing HTML-tags (start/end) which will wrap the title
+        * @param bool $enableClickMenu If TRUE, render click menu code around icon image
         * @return string
         */
-       public function getResourceHeader(\TYPO3\CMS\Core\Resource\ResourceInterface $resource, $tWrap = array('', '')) {
+       public function getResourceHeader(\TYPO3\CMS\Core\Resource\ResourceInterface $resource, $tWrap = array('', ''), $enableClickMenu = TRUE) {
                $path = $resource->getStorage()->getName() . $resource->getParentFolder()->getIdentifier();
                $iconImgTag = IconUtility::getSpriteIconForResource($resource, array('title' => htmlspecialchars($path)));
 
-               if ($resource instanceof \TYPO3\CMS\Core\Resource\File) {
+               if ($enableClickMenu && ($resource instanceof \TYPO3\CMS\Core\Resource\File)) {
                        $metaData = $resource->_getMetaData();
                        $iconImgTag = $this->wrapClickMenuOnIcon($iconImgTag, 'sys_file_metadata', $metaData['uid']);
                }