[CLEANUP] Use proper Clickmenu API 74/20174/5
authorFrancois Suter <francois@typo3.org>
Fri, 26 Apr 2013 09:00:47 +0000 (11:00 +0200)
committerBenjamin Mack <benni@typo3.org>
Tue, 2 Jul 2013 12:05:06 +0000 (14:05 +0200)
The proper way to call up the CSM is with Clickmenu.show().
Many places in the Core still use the deprecated
showClickmenu() function. This patch replaces them all.

Resolves: #20326
Releases: 6.2
Change-Id: Ia2d72b97ae9bf09026d9923051b2c5aec617db5d
Reviewed-on: https://review.typo3.org/20174
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Alexander Opitz
Tested-by: Alexander Opitz
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack
typo3/sysext/backend/Classes/Template/DocumentTemplate.php
typo3/sysext/backend/Resources/Public/JavaScript/clickmenu.js
typo3/sysext/beuser/Resources/Private/Partials/BackendUser/IndexListRow.html
typo3/sysext/beuser/Resources/Private/Templates/BackendUser/Compare.html
typo3/sysext/beuser/Resources/Private/Templates/BackendUser/Index.html
typo3/sysext/core/Classes/Utility/File/ExtendedFileUtility.php
typo3/sysext/sys_note/Resources/Private/Templates/Note/List.html
typo3/sysext/tstemplate_info/Classes/Controller/TypoScriptTemplateInformationModuleFunctionController.php

index e4c419f..2b8e5c6 100644 (file)
@@ -460,7 +460,7 @@ class DocumentTemplate {
         */
        public function wrapClickMenuOnIcon($str, $table, $uid = 0, $listFr = TRUE, $addParams = '', $enDisItems = '', $returnOnClick = FALSE) {
                $backPath = rawurlencode($this->backPath) . '|' . GeneralUtility::shortMD5(($this->backPath . '|' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['encryptionKey']));
-               $onClick = 'showClickmenu("' . $table . '","' . ($uid !== 0 ? $uid : '') . '","' . strval($listFr) . '","' . str_replace('+', '%2B', $enDisItems) . '","' . str_replace('&', '&amp;', addcslashes($backPath, '"')) . '","' . str_replace('&', '&amp;', addcslashes($addParams, '"')) . '");return false;';
+               $onClick = 'Clickmenu.show("' . $table . '","' . ($uid !== 0 ? $uid : '') . '","' . strval($listFr) . '","' . str_replace('+', '%2B', $enDisItems) . '","' . str_replace('&', '&amp;', addcslashes($backPath, '"')) . '","' . str_replace('&', '&amp;', addcslashes($addParams, '"')) . '");return false;';
                return $returnOnClick ? $onClick : '<a href="#" onclick="' . htmlspecialchars($onClick) . '" oncontextmenu="' . htmlspecialchars($onClick) . '">' . $str . '</a>';
        }
 
index bcf9fd5..7cf4b8c 100644 (file)
@@ -18,7 +18,7 @@
  */
 
 /**
- * new clickmenu code to make an AJAX call and render the 
+ * new clickmenu code to make an AJAX call and render the
  * AJAX result in a layer next to the mouse cursor
  */
 var Clickmenu = {
@@ -54,8 +54,8 @@ var Clickmenu = {
         *
         * @param       params  parameters added to the URL
         * @return      nothing
-        */ 
-       callURL: function(params) {     
+        */
+       callURL: function(params) {
                if (this.ajax && Ajax.getTransport()) { // run with AJAX
                        params += '&ajax=1';
                        var call = new Ajax.Request(this.clickURL, {
@@ -225,11 +225,6 @@ Event.observe(document, 'mousemove', Clickmenu.calcMousePosEvent.bindAsEventList
 // @deprecated: Deprecated functions since 4.2, here for compatibility, remove in 4.4+
 // ## BEGIN ##
 
-// Still used in Core: \TYPO3\CMS\Backend\Template\DocumentTemplate::wrapClickMenuOnIcon()
-function showClickmenu(table, uid, listFr, enDisItems, backPath, addParams) {
-       Clickmenu.show(table, uid, listFr, enDisItems, backPath, addParams);
-}
-
 // Still used in Core: typo3/alt_clickmenu.php::linkItem()
 function showClickmenu_raw(url) {
        var parts = url.split('?');
@@ -254,7 +249,7 @@ function hideSpecific(level) {
        if (level === 0 || level === 1) {
                Clickmenu.hide('contentMenu'+level);
        }
-} 
+}
 function showHideSelectorBoxes(action) {
        toggleSelectorBoxes(action);
 }
index 75222bd..26147fb 100755 (executable)
@@ -2,7 +2,7 @@
 
 <tr class="db_list_normal">
        <td class="col-icon">
-               <a href="#" onClick="showClickmenu('be_users', '{backendUser.uid}', '1', '', '', ''); return false;" title="id={backendUser.uid}">
+               <a href="#" onClick="Clickmenu.show('be_users', '{backendUser.uid}', '1', '', '', ''); return false;" title="id={backendUser.uid}">
                        <bu:spriteIconForRecord table="be_users" object="{backendUser}" />
                </a>
        </td>
index 86c460f..94f5089 100755 (executable)
@@ -14,7 +14,7 @@
                        <td class="label"></td>
                        <f:for each="{compareUserList}" as="compareUser">
                                <td>
-                                       <a href="#" onClick="showClickmenu('be_users', '{compareUser.uid}', '1', '', '', ''); return false;" title="id={compareUser.uid}">
+                                       <a href="#" onClick="Clickmenu.show('be_users', '{compareUser.uid}', '1', '', '', ''); return false;" title="id={compareUser.uid}">
                                                <bu:spriteIconForRecord table="be_users" object="{compareUser}" />
                                        </a>
                                        {compareUser.userName}
                        <f:for each="{compareUserList}" as="compareUser">
                                <td>
                                        <f:for each="{compareUser.BackendUserGroups}" as="backendUserGroup">
-                                               <a href="#" onClick="showClickmenu('be_groups', '{backendUserGroup.uid}', '1', '', '', ''); return false;" title="id={backendUserGroup.uid}">
+                                               <a href="#" onClick="Clickmenu.show('be_groups', '{backendUserGroup.uid}', '1', '', '', ''); return false;" title="id={backendUserGroup.uid}">
                                                        <bu:spriteIconForRecord table="be_users" object="{backendUserGroup}" />
                                                </a>
                                                {backendUserGroup.title}<br />
index dcbd0ee..81524e3 100755 (executable)
@@ -21,7 +21,7 @@
                                                        arguments="{uid: compareUser.uid}"
                                                ><bu:SpriteManagerIcon iconName="actions-selection-delete" /></f:link.action>
 
-                                               <a href="#" onClick="showClickmenu('be_users', '{compareUser.uid}', '1', '', '', ''); return false;" title="id={compareUser.uid}">
+                                               <a href="#" onClick="Clickmenu.show('be_users', '{compareUser.uid}', '1', '', '', ''); return false;" title="id={compareUser.uid}">
                                                        <bu:spriteIconForRecord table="be_users" object="{compareUser}" />
                                                </a>
 
index ab2fb8e..3ba6c73 100644 (file)
@@ -414,7 +414,7 @@ class ExtendedFileUtility extends \TYPO3\CMS\Core\Utility\File\BasicFileUtility
                                                $row = $this->transformFileReferenceToRecordReference($fileReferenceRow);
                                                $shortcutRecord = \TYPO3\CMS\Backend\Utility\BackendUtility::getRecord($row['tablename'], $row['recuid']);
                                                $icon = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIconForRecord($row['tablename'], $shortcutRecord);
-                                               $onClick = 'showClickmenu("' . $row['tablename'] . '", "' . $row['recuid'] . '", "1", "+info,history,edit", "|", "");return false;';
+                                               $onClick = 'Clickmenu.show("' . $row['tablename'] . '", "' . $row['recuid'] . '", "1", "+info,history,edit", "|", "");return false;';
                                                $shortcutContent[] = '<a href="#" oncontextmenu="' . htmlspecialchars($onClick) . '" onclick="' . htmlspecialchars($onClick) . '">' . $icon . '</a>' . htmlspecialchars((\TYPO3\CMS\Backend\Utility\BackendUtility::getRecordTitle($row['tablename'], $shortcutRecord) . '  [' . \TYPO3\CMS\Backend\Utility\BackendUtility::getRecordPath($shortcutRecord['pid'], '', 80) . ']'));
                                        }
                                }
index ebcbf12..2307aa6 100644 (file)
@@ -9,7 +9,7 @@
                        <f:for each="{notes}" as="note">
                                <div class="single-note category-{note.category}">
                                        <div class="header">
-                                               <span class="typeicon" oncontextmenu="showClickmenu('sys_note', '{note.uid}', '1', '', '', ''); return false;" onclick="showClickmenu('sys_note', '{note.uid}', '1', '','', ''); return false;"></span>
+                                               <span class="typeicon" oncontextmenu="Clickmenu.show('sys_note', '{note.uid}', '1', '', '', ''); return false;" onclick="Clickmenu.show('sys_note', '{note.uid}', '1', '','', ''); return false;"></span>
                                                <span><f:translate key="author" /></span>
                                                <f:if condition="{note.author.realName}">
                                                        <f:then>{note.author.realName}</f:then>
index d4e5bac..e682ecc 100644 (file)
@@ -311,8 +311,8 @@ class TypoScriptTemplateInformationModuleFunctionController extends \TYPO3\CMS\B
                                        }
                                }
                        }
-                       $content = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIconForRecord('sys_template', $tplRow, array('oncontextmenu' => "showClickmenu(
-'sys_template', '" . $tplRow['uid'] . "', '1', '', '', ''); return false;", 'onclick' => "showClickmenu('sys_template', '" . $tplRow['uid'] . "', '1', '','', ''); return false;")) . '<strong>' . htmlspecialchars($tplRow['title']) . '</strong>' . htmlspecialchars((trim($tplRow['sitetitle']) ? ' (' . $tplRow['sitetitle'] . ')' : ''));
+                       $content = \TYPO3\CMS\Backend\Utility\IconUtility::getSpriteIconForRecord('sys_template', $tplRow, array('oncontextmenu' => "Clickmenu.show(
+'sys_template', '" . $tplRow['uid'] . "', '1', '', '', ''); return false;", 'onclick' => "Clickmenu.show('sys_template', '" . $tplRow['uid'] . "', '1', '','', ''); return false;")) . '<strong>' . htmlspecialchars($tplRow['title']) . '</strong>' . htmlspecialchars((trim($tplRow['sitetitle']) ? ' (' . $tplRow['sitetitle'] . ')' : ''));
                        $theOutput .= $this->pObj->doc->section($GLOBALS['LANG']->getLL('templateInformation'), $content, 0, 1);
                        if ($manyTemplatesMenu) {
                                $theOutput .= $this->pObj->doc->section('', $manyTemplatesMenu);
@@ -417,4 +417,4 @@ class TypoScriptTemplateInformationModuleFunctionController extends \TYPO3\CMS\B
 
 }
 
-?>
\ No newline at end of file
+?>