Commit f371f1dc authored by Georg Ringer's avatar Georg Ringer Committed by Wouter Wolters
Browse files

[TASK] Remove the "@todo define visibility"

The "@todo Define visibility" is not very helpful because:

 * Nobody knows what breaks if changing the visibility
 * The code is flooded with todos which makes it hard
   to work with in an IDE

During ACME it was decided to remove the @todo again.

Resolves: #61957
Releases: master
Change-Id: I0caa9756bba326dc132b6a2dabe79218c53a32c8
Reviewed-on: http://review.typo3.org/33044

Reviewed-by: default avatarMarkus Klein <klein.t3@reelworx.at>
Tested-by: default avatarMarkus Klein <klein.t3@reelworx.at>
Reviewed-by: Wouter Wolters's avatarWouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters's avatarWouter Wolters <typo3@wouterwolters.nl>
parent 0d8a8853
......@@ -16,8 +16,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
/**
* Constructor. Just calling init()
*
* @todo Define visibility
*/
public function __construct() {
$this->determineScriptUrl();
......@@ -32,7 +30,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
* @param array $v The record
* @param boolean $ext_pArrPages (Ignore)
* @return string Wrapping title string.
* @todo Define visibility
*/
public function wrapTitle($title, $v, $ext_pArrPages = '') {
if ($this->ext_isLinkable($v['doktype'], $v['uid'])) {
......@@ -48,7 +45,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
*
* @param array $treeArr Tree array
* @return string HTML output.
* @todo Define visibility
*/
public function printTree($treeArr = '') {
$titleLen = (int)$GLOBALS['BE_USER']->uc['titleLen'];
......@@ -91,7 +87,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
* @param integer $doktype Doktype value to test
* @param integer $uid uid to test.
* @return boolean
* @todo Define visibility
*/
public function ext_isLinkable($doktype, $uid) {
if ($uid && $doktype < 199) {
......@@ -106,7 +101,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
* @param string $cmd Command for 'PM' get var
* @param boolean $bMark If set, the link will have a anchor point (=$bMark) and a name attribute (=$bMark)
* @return string Link-wrapped input string
* @todo Define visibility
*/
public function PM_ATagWrap($icon, $cmd, $bMark = '') {
$name = '';
......@@ -124,7 +118,6 @@ class localPageTree extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
* @param string $icon The image tag for the icon
* @param array $row The row for the current element
* @return string The processed icon input value.
* @todo Define visibility
*/
public function wrapIcon($icon, $row) {
$content = $this->addTagAttributes($icon, ' title="id=' . $row['uid'] . '"');
......@@ -149,7 +142,6 @@ class TBE_PageTree extends localPageTree {
* @param integer $doktype Doktype value to test
* @param integer $uid uid to test.
* @return boolean
* @todo Define visibility
*/
public function ext_isLinkable($doktype, $uid) {
return TRUE;
......@@ -162,7 +154,6 @@ class TBE_PageTree extends localPageTree {
* @param array $v The record
* @param boolean $ext_pArrPages If set, pages clicked will return immediately, otherwise reload page.
* @return string Wrapping title string.
* @todo Define visibility
*/
public function wrapTitle($title, $v, $ext_pArrPages) {
if ($ext_pArrPages) {
......@@ -186,14 +177,12 @@ class TBE_PageTree extends localPageTree {
class localFolderTree extends \TYPO3\CMS\Backend\Tree\View\FolderTreeView {
/**
* @todo Define visibility
* @var int
*/
public $ext_IconMode = 1;
/**
* Initializes the script path
*
* @todo Define visibility
*/
public function __construct() {
$this->determineScriptUrl();
......@@ -206,7 +195,6 @@ class localFolderTree extends \TYPO3\CMS\Backend\Tree\View\FolderTreeView {
* @param string $title Title, ready for output.
* @param \TYPO3\CMS\Core\Resource\Folder $folderObject The "record
* @return string Wrapping title string.
* @todo Define visibility
*/
public function wrapTitle($title, \TYPO3\CMS\Core\Resource\Folder $folderObject) {
if ($this->ext_isLinkable($folderObject)) {
......@@ -222,7 +210,6 @@ class localFolderTree extends \TYPO3\CMS\Backend\Tree\View\FolderTreeView {
*
* @param \TYPO3\CMS\Core\Resource\Folder $folderObject Object with information about the folder element. Contains keys like title, uid, path, _title
* @return boolean TRUE is returned if the path is found in the web-part of the server and is NOT a recycler or temp folder
* @todo Define visibility
*/
public function ext_isLinkable(\TYPO3\CMS\Core\Resource\Folder $folderObject) {
if (strstr($folderObject->getIdentifier(), '_recycler_') || strstr($folderObject->getIdentifier(), '_temp_')) {
......@@ -240,7 +227,6 @@ class localFolderTree extends \TYPO3\CMS\Backend\Tree\View\FolderTreeView {
* @param boolean $bMark If set, the link will have a anchor point (=$bMark) and a name attribute (=$bMark)
* @return string Link-wrapped input string
* @access private
* @todo Define visibility
*/
public function PM_ATagWrap($icon, $cmd, $bMark = '') {
$name = $anchor = '';
......@@ -261,9 +247,10 @@ class localFolderTree extends \TYPO3\CMS\Backend\Tree\View\FolderTreeView {
*/
class TBE_FolderTree extends localFolderTree {
// If file-drag mode is set, temp and recycler folders are filtered out.
/**
* @todo Define visibility
* If file-drag mode is set, temp and recycler folders are filtered out.
*
* @var int
*/
public $ext_noTempRecyclerDirs = 0;
......@@ -272,7 +259,6 @@ class TBE_FolderTree extends localFolderTree {
*
* @param \TYPO3\CMS\Core\Resource\Folder $folderObject object with information about the folder element. Contains keys like title, uid, path, _title
* @return boolean TRUE is returned if the path is NOT a recycler or temp folder AND if ->ext_noTempRecyclerDirs is not set.
* @todo Define visibility
*/
public function ext_isLinkable($folderObject) {
if ($this->ext_noTempRecyclerDirs && (substr($folderObject->getIdentifier(), -7) == '_temp_/' || substr($folderObject->getIdentifier(), -11) == '_recycler_/')) {
......@@ -288,7 +274,6 @@ class TBE_FolderTree extends localFolderTree {
* @param string $title Title, ready for output.
* @param \TYPO3\CMS\Core\Resource\Folder $folderObject The folderObject 'record'
* @return string Wrapping title string.
* @todo Define visibility
*/
public function wrapTitle($title, $folderObject) {
if ($this->ext_isLinkable($folderObject)) {
......
......@@ -35,7 +35,6 @@ class newRecordLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
* @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']) . '"');
......@@ -47,7 +46,6 @@ class newRecordLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView {
*
* @param integer $id The ID (page id) of the element
* @return boolean Returns TRUE if the IDs matches
* @todo Define visibility
*/
public function expandNext($id) {
return $id == $GLOBALS['SOBE']->id ? 1 : 0;
......
......@@ -33,7 +33,6 @@ class moveElementLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView
* @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']) . '"');
......@@ -49,7 +48,7 @@ class moveElementLocalPageTree extends \TYPO3\CMS\Backend\Tree\View\PageTreeView
class ext_posMap_pages extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
/**
* @todo Define visibility
* @var string
*/
public $l_insertNewPageHere = 'movePageToHere';
......@@ -66,7 +65,6 @@ class ext_posMap_pages extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
* @param integer $pid The pid.
* @param integer $newPagePID New page id.
* @return string Onclick attribute content
* @todo Define visibility
*/
public function onClickEvent($pid, $newPagePID) {
return 'window.location.href=\'tce_db.php?cmd[pages][' . $GLOBALS['SOBE']->moveUid . '][' . $this->moveOrCopy . ']=' . $pid . '&redirect=' . rawurlencode($this->R_URI) . '&prErr=1&uPT=1&vC=' . $GLOBALS['BE_USER']->veriCode() . \TYPO3\CMS\Backend\Utility\BackendUtility::getUrlToken('tceAction') . '\';return false;';
......@@ -78,7 +76,6 @@ class ext_posMap_pages extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
* @param string $str Page title.
* @param array $rec Page record (?)
* @return string Wrapped title.
* @todo Define visibility
*/
public function linkPageTitle($str, $rec) {
$url = \TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript(array('uid' => (int)$rec['uid'], 'moveUid' => $GLOBALS['SOBE']->moveUid));
......@@ -92,7 +89,6 @@ class ext_posMap_pages extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
* @param array $dat Information array with record array inside.
* @param integer $id The current id.
* @return string The title string.
* @todo Define visibility
*/
public function boldTitle($t_code, $dat, $id) {
return parent::boldTitle($t_code, $dat, $GLOBALS['SOBE']->moveUid);
......@@ -108,7 +104,7 @@ class ext_posMap_pages extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
class ext_posMap_tt_content extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
/**
* @todo Define visibility
* @var int
*/
public $dontPrintPageInsertIcons = 1;
......@@ -125,7 +121,6 @@ class ext_posMap_tt_content extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap
* @param string $str Page title.
* @param array $rec Page record (?)
* @return string Wrapped title.
* @todo Define visibility
*/
public function linkPageTitle($str, $rec) {
$url = \TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript(array('uid' => (int)$rec['uid'], 'moveUid' => $GLOBALS['SOBE']->moveUid));
......@@ -138,7 +133,6 @@ class ext_posMap_tt_content extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap
* @param string $str The title value.
* @param array $row The record row.
* @return string Wrapped title string.
* @todo Define visibility
*/
public function wrapRecordTitle($str, $row) {
if ($GLOBALS['SOBE']->moveUid == $row['uid']) {
......
......@@ -30,110 +30,61 @@ class ClickMenu {
// Internal, static: GPvar:
// Defines if the click menu is first level or second. Second means the click menu is triggered from another menu.
/**
* @todo Define visibility
*/
public $cmLevel = 0;
// Clipboard array (submitted by eg. pressing the paste button)
/**
* @todo Define visibility
*/
public $CB;
// Internal, static:
// Backpath for scripts/images.
/**
* @todo Define visibility
*/
public $backPath = '';
// BackPath place holder: We need different backPath set whether the clickmenu is written back to a frame which is not in typo3/ dir or if the clickmenu is shown in the top frame (no backpath)
/**
* @todo Define visibility
*/
public $PH_backPath = '###BACK_PATH###';
// If set, the calling document should be in the listframe of a frameset.
/**
* @todo Define visibility
*/
public $listFrame = 0;
// If set, the menu is about database records, not files. (set if part 2 [1] of the item-var is NOT blank)
/**
* @todo Define visibility
*/
public $isDBmenu = 0;
// If TRUE, the "content" frame is always used for reference (when condensed mode is enabled)
/**
* @todo Define visibility
*/
public $alwaysContentFrame = 0;
// Stores the parts of the input $item string, splitted by "|":
// [0] = table/file, [1] = uid/blank, [2] = flag: If set, listFrame,
// If "2" then "content frame" is forced [3] = ("+" prefix = disable
// all by default, enable these. Default is to disable) Items key list
/**
* @todo Define visibility
*/
public $iParts = array();
// Contains list of keywords of items to disable in the menu
/**
* @todo Define visibility
*/
public $disabledItems = array();
// If TRUE, Show icons on the left.
/**
* @todo Define visibility
*/
public $leftIcons = 0;
// Array of classes to be used for user processing of the menu content. This is for the API of adding items to the menu from outside.
/**
* @todo Define visibility
*/
public $extClassArray = array();
// Enable/disable ajax behavior
/**
* @todo Define visibility
*/
public $ajax = 0;
// Internal, dynamic:
// Counter for elements in the menu. Used to number the name / id of the mouse-over icon.
/**
* @todo Define visibility
*/
public $elCount = 0;
// Set, when edit icon is drawn.
/**
* @todo Define visibility
*/
public $editPageIconSet = 0;
// Set to TRUE, if editing of the element is OK.
/**
* @todo Define visibility
*/
public $editOK = 0;
/**
* @todo Define visibility
*/
public $rec = array();
/**
* Initialize click menu
*
* @return string The clickmenu HTML content
* @todo Define visibility
*/
public function init() {
// Setting GPvars:
......@@ -195,7 +146,6 @@ class ClickMenu {
* @param string $table Table name
* @param integer $uid UID for the current record.
* @return string HTML content
* @todo Define visibility
*/
public function printDBClickMenu($table, $uid) {
$uid = (int)$uid;
......@@ -395,7 +345,6 @@ class ClickMenu {
* @param string $table Table name
* @param integer $uid UID for the current record.
* @return string HTML content
* @todo Define visibility
*/
public function printNewDBLevel($table, $uid) {
$uid = (int)$uid;
......@@ -428,7 +377,6 @@ class ClickMenu {
*
* @param array $menuItems Array for manipulation.
* @return array Processed $menuItems array
* @todo Define visibility
*/
public function externalProcessingOfDBMenuItems($menuItems) {
return $menuItems;
......@@ -441,7 +389,6 @@ class ClickMenu {
* @param string $table Table name
* @param integer $uid UID for the current record.
* @return array Processed $menuItems array
* @todo Define visibility
*/
public function processingByExtClassArray($menuItems, $table, $uid) {
if (is_array($this->extClassArray)) {
......@@ -461,7 +408,6 @@ class ClickMenu {
* @param boolean $hideCM If set, the "hideCM()" will be called
* @param string $overrideLoc If set, gives alternative location to load in (for example top frame or somewhere else)
* @return string JavaScript for an onClick event.
* @todo Define visibility
*/
public function urlRefForCM($url, $retUrl = '', $hideCM = 1, $overrideLoc = '') {
$loc = 'top.content.list_frame';
......@@ -477,7 +423,6 @@ class ClickMenu {
* @param string $type Type: "copy" or "cut
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_copycut($table, $uid, $type) {
if ($this->clipObj->current == 'normal') {
......@@ -501,7 +446,6 @@ class ClickMenu {
* @return array Item array, element in $menuItems
* @see \TYPO3\CMS\Backend\Clipboard\Clipboard::pasteUrl()
* @internal
* @todo Define visibility
*/
public function DB_paste($table, $uid, $type, $elInfo) {
$editOnClick = '';
......@@ -522,7 +466,6 @@ class ClickMenu {
* @param integer $uid UID for the current record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_info($table, $uid) {
return $this->linkItem($this->label('info'), $this->excludeIcon(IconUtility::getSpriteIcon('actions-document-info')), 'top.launchView(\'' . $table . '\', \'' . $uid . '\'); return hideCM();');
......@@ -535,7 +478,6 @@ class ClickMenu {
* @param integer $uid UID for the current record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_history($table, $uid) {
$url = BackendUtility::getModuleUrl('record_history', array('element' => $table . ':' . $uid));
......@@ -550,7 +492,6 @@ class ClickMenu {
* @param array $rec The "pages" record with "perms_*" fields inside.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_perms($table, $uid, $rec) {
if (!ExtensionManagementUtility::isLoaded('perm')) {
......@@ -568,7 +509,6 @@ class ClickMenu {
* @param array $rec Record of the element (needs "pid" field if not pages-record)
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_db_list($table, $uid, $rec) {
$urlParams = array();
......@@ -586,7 +526,6 @@ class ClickMenu {
* @param array $rec Record. Needed for tt-content elements which will have the sys_language_uid sent
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_moveWizard($table, $uid, $rec) {
// Hardcoded field for tt_content elements.
......@@ -602,7 +541,6 @@ class ClickMenu {
* @param array $rec Record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_newWizard($table, $uid, $rec) {
// If mod.web_list.newContentWiz.overrideWithExtension is set, use that extension's create new content wizard instead:
......@@ -620,7 +558,6 @@ class ClickMenu {
* @param integer $uid UID for the current record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_editAccess($table, $uid) {
$addParam = '&columnsOnly=' . rawurlencode((implode(',', $GLOBALS['TCA'][$table]['ctrl']['enablecolumns']) . ($table == 'pages' ? ',extendToSubpages' : '')));
......@@ -634,7 +571,6 @@ class ClickMenu {
* @param integer $uid page uid to edit (PID)
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_editPageProperties($uid) {
$url = 'alt_doc.php?edit[pages][' . $uid . ']=edit';
......@@ -648,7 +584,6 @@ class ClickMenu {
* @param integer $uid UID for the current record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_edit($table, $uid) {
// If another module was specified, replace the default Page module with the new one
......@@ -680,7 +615,6 @@ class ClickMenu {
* @param integer $uid UID for the current record.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_new($table, $uid) {
$editOnClick = '';
......@@ -697,7 +631,6 @@ class ClickMenu {
* @param array $elInfo Label for including in the confirmation message, EXT:lang/locallang_core.xlf:mess.delete
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_delete($table, $uid, $elInfo) {
$editOnClick = '';
......@@ -718,7 +651,6 @@ class ClickMenu {
* @param string $anchor Anchor, if any
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_view($id, $anchor = '') {
return $this->linkItem($this->label('view'), $this->excludeIcon(IconUtility::getSpriteIcon('actions-document-view')), BackendUtility::viewOnClick($id, $this->PH_backPath, BackendUtility::BEgetRootLine($id), $anchor) . 'return hideCM();');
......@@ -730,7 +662,6 @@ class ClickMenu {
* @param integer $page_id Page uid (PID)
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_tempMountPoint($page_id) {
return $this->linkItem($this->label('tempMountPoint'), $this->excludeIcon(IconUtility::getSpriteIcon('apps-pagetree-page-mountpoint')), 'if (top.content.nav_frame) {
......@@ -761,7 +692,6 @@ class ClickMenu {
* @param string $hideField Name of the hide field
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function DB_hideUnhide($table, $rec, $hideField) {
return $this->DB_changeFlag($table, $rec, $hideField, $this->label(($rec[$hideField] ? 'un' : '') . 'hide'), 'hide');
......@@ -777,7 +707,6 @@ class ClickMenu {
* @param string $name Name of the item used for icons and labels
* @param string $iconRelPath Icon path relative to typo3/ folder
* @return array Item array, element in $menuItems
* @todo Define visibility
*/
public function DB_changeFlag($table, $rec, $flagField, $title, $name, $iconRelPath = 'gfx/') {
$uid = $rec['_ORIG_uid'] ?: $rec['uid'];
......@@ -798,7 +727,6 @@ class ClickMenu {
* @param string $combinedIdentifier The combined identifier
* @return string HTML content
* @see \TYPO3\CMS\Core\Resource\ResourceFactory::retrieveFileOrFolderObject()
* @todo Define visibility
*/
public function printFileClickMenu($combinedIdentifier) {
$menuItems = array();
......@@ -924,7 +852,6 @@ class ClickMenu {
*
* @param array $menuItems Array for manipulation.
* @return array Processed $menuItems array
* @todo Define visibility
*/
public function externalProcessingOfFileMenuItems($menuItems) {
return $menuItems;
......@@ -940,7 +867,6 @@ class ClickMenu {
* @param boolean $noReturnUrl If set, the return URL parameter will not be set in the link
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function FILE_launch($path, $moduleName, $type, $image, $noReturnUrl = FALSE) {
$loc = 'top.content.list_frame';
......@@ -966,7 +892,6 @@ class ClickMenu {
* @param string $path Path to the file/directory (target)
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function FILE_upload($path) {
$script = 'file_upload';
......@@ -982,7 +907,6 @@ class ClickMenu {
* @param string $type Type: "copy" or "cut
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function FILE_copycut($path, $type) {
// Pseudo table name for use in the clipboard.
......@@ -1004,7 +928,6 @@ class ClickMenu {
* @param string $path Path to the file/directory (target)
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function FILE_delete($path) {
$editOnClick = '';
......@@ -1026,7 +949,6 @@ class ClickMenu {
* @param array $elInfo Various values for the labels.
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function FILE_paste($path, $target, $elInfo) {
$editOnClick = '';
......@@ -1062,7 +984,6 @@ class ClickMenu {
* @param integer $srcId UID for the current record.
* @param integer $dstId Destination ID
* @return string HTML content
* @todo Define visibility
*/
public function printDragDropClickMenu($table, $srcId, $dstId) {
$menuItems = array();
......@@ -1098,7 +1019,6 @@ class ClickMenu {
*
* @param array $menuItems Array for manipulation.
* @return array Processed $menuItems array
* @todo Define visibility
*/
public function externalProcessingOfDragDropMenuItems($menuItems) {
return $menuItems;
......@@ -1113,7 +1033,6 @@ class ClickMenu {
* @param string $into Parameter code: either "into" or "after
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function dragDrop_copymovepage($srcUid, $dstUid, $action, $into) {
$negativeSign = $into == 'into' ? '' : '-';
......@@ -1131,7 +1050,6 @@ class ClickMenu {
* @param string $action Action code: either "move" or "copy
* @return array Item array, element in $menuItems
* @internal
* @todo Define visibility
*/
public function dragDrop_copymovefolder($srcPath, $dstPath, $action) {
$editOnClick = '';
......@@ -1151,7 +1069,6 @@ class ClickMenu {
* @param array $menuItems Array
* @param string $item HTML code for the element which was clicked - shown in the end of the horizontal menu in topframe after the close-button.
* @return string HTML code
* @todo Define visibility
*/
public function printItems($menuItems, $item) {
$out = '';
......@@ -1170,7 +1087,6 @@ class ClickMenu {
*
* @param array $menuItems The $menuItems array to print
* @return string The JavaScript section which will print the content of the CM to the div-layer in the target frame.
* @todo Define visibility
*/
public function printLayerJScode($menuItems) {
$script = '';
......@@ -1212,7 +1128,6 @@ class ClickMenu {
*
* @param string $str HTML content to wrap in table.
* @return string
* @todo Define visibility
*/
public function wrapColorTableCM($str) {
return '<div class="typo3-CSM-wrapperCM">
......@@ -1225,7 +1140,6 @@ class ClickMenu {
*
* @param array $menuItems Array
* @return array Array of menu items for top frame.
* @todo Define visibility
*/
public function menuItemsForTopFrame($menuItems) {
$out = array();
......@@ -1248,7 +1162,6 @@ class ClickMenu {
*
* @param array $menuItems Array
* @return array array for implosion in the CM div-layers table.
* @todo Define visibility
*/
public function menuItemsForClickMenu($menuItems) {
$out = array();
......@@ -1286,7 +1199,6 @@ class ClickMenu {
* @param array $newMenuItems Menu items array to insert
* @param string $position Position command string. Has this syntax: [cmd]:[menu entry key],[cmd].... cmd can be "after", "before" or "top" (or blank/"bottom" which is default). If "after"/"before" then menu items will be inserted after/before the existing entry with [menu entry key] if found. "after-spacer" and "before-spacer" do the same, but inserts before or after an item and a spacer. If not found, the bottom of list. If "top" the items are inserted in the top of the list.
* @return array Menu items array, processed.
* @todo Define visibility
*/
public function addMenuItems($menuItems, $newMenuItems, $position = '') {
if (is_array($newMenuItems)) {
......@@ -1363,7 +1275,6 @@ class ClickMenu {
* @param boolean $onlyCM ==1 and the element will NOT appear in clickmenus in the topframe (unless clickmenu is totally unavailable)! ==2 and the item will NEVER appear in top frame. (This is mostly for "less important" options since the top frame is not capable of holding so many elements horizontally)
* @param boolean $dontHide If set, the clickmenu layer will not hide itself onclick - used for secondary menus to appear...
* @return array $menuItem entry with 6 numerical entries: [0] is the HTML for display of the element with link and icon an mouseover etc., [1]-[5] is simply the input params passed through!
* @todo Define visibility
*/
public function linkItem($str, $icon, $onClick, $onlyCM = 0, $dontHide = 0) {
$this->elCount++;
......@@ -1388,7 +1299,6 @@ class ClickMenu {
*
* @param string $iconCode The icon-image tag
* @return string The icon-image tag prefixed with space char IF the icon should be printed at all due to user settings
* @todo Define visibility
*/
public function excludeIcon($iconCode) {
return $GLOBALS['BE_USER']->uc['noMenuMode'] && $GLOBALS['BE_USER']->uc['noMenuMode'] !== 'icons' ? '' : ' ' . $iconCode;
......@@ -1399,7 +1309,6 @@ class ClickMenu {
*