[TASK] Remove the "@todo define visibility" in ext:backend 67/33267/3
authorWouter Wolters <typo3@wouterwolters.nl>
Sat, 11 Oct 2014 16:12:37 +0000 (18:12 +0200)
committerGeorg Ringer <georg.ringer@gmail.com>
Sun, 12 Oct 2014 18:46:06 +0000 (20:46 +0200)
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: #62165
Related: #61957
Releases: master
Change-Id: Id4723c5c366d629fabb114456eebbed746aa9a53
Reviewed-on: http://review.typo3.org/33267
Reviewed-by: Stefan Froemken <froemken@gmail.com>
Tested-by: Stefan Froemken <froemken@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
22 files changed:
typo3/sysext/backend/Classes/AjaxLoginHandler.php
typo3/sysext/backend/Classes/Controller/FileSystemNavigationFrameController.php
typo3/sysext/backend/Classes/Controller/NewRecordController.php
typo3/sysext/backend/Classes/Controller/Wizard/AddController.php
typo3/sysext/backend/Classes/Controller/Wizard/EditController.php
typo3/sysext/backend/Classes/Controller/Wizard/RteController.php
typo3/sysext/backend/Classes/Form/DataPreprocessor.php
typo3/sysext/backend/Classes/Form/Element/InlineElement.php
typo3/sysext/backend/Classes/Form/Element/ValueSlider.php
typo3/sysext/backend/Classes/History/RecordHistory.php
typo3/sysext/backend/Classes/Module/BaseScriptClass.php
typo3/sysext/backend/Classes/RecordList/AbstractRecordList.php
typo3/sysext/backend/Classes/RecordList/ElementBrowserRecordList.php
typo3/sysext/backend/Classes/Rte/AbstractRte.php
typo3/sysext/backend/Classes/Template/BigDocumentTemplate.php
typo3/sysext/backend/Classes/Tree/Pagetree/IndicatorProviderInterface.php
typo3/sysext/backend/Classes/Tree/View/AbstractTreeView.php
typo3/sysext/backend/Classes/Tree/View/BrowseTreeView.php
typo3/sysext/backend/Classes/Tree/View/FolderTreeView.php
typo3/sysext/backend/Classes/Tree/View/PagePositionMap.php
typo3/sysext/backend/Classes/Tree/View/PageTreeView.php
typo3/sysext/backend/Classes/View/PageTreeView.php

index 731a8fd..ac39f2f 100644 (file)
@@ -101,7 +101,6 @@ class AjaxLoginHandler {
         * @param array $parameters Parameters (not used)
         * @param \TYPO3\CMS\Core\Http\AjaxRequestHandler $ajaxObj The calling parent AJAX object
         * @return void
-        * @todo Define visibility
         */
        public function isTimedOut(array $parameters, \TYPO3\CMS\Core\Http\AjaxRequestHandler $ajaxObj) {
                if (is_object($GLOBALS['BE_USER'])) {
index c8bb64a..cbcbf13 100644 (file)
@@ -55,7 +55,7 @@ class FileSystemNavigationFrameController {
        public $currentSubScript;
 
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $cMR;
 
index b76402e..2fc1f16 100644 (file)
@@ -64,12 +64,12 @@ class NewRecordController {
        public $web_list_modTSconfig;
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $allowedNewTables;
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $deniedNewTables;
 
index 1e1f66c..18cd932 100644 (file)
@@ -28,45 +28,45 @@ class AddController {
        // Internal, dynamic:
        // Content accumulation for the module.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $content;
 
        // If set, the TCEmain class is loaded and used to add the returning ID to the parent record.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $processDataFlag = 0;
 
        // Internal, static:
        // Create new record -pid (pos/neg). If blank, return immediately
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $pid;
 
        // The parent table we are working on.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $table;
 
        // Loaded with the created id of a record when TCEforms (alt_doc.php) returns ...
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $id;
 
        // Internal, static: GPvars
        // Wizard parameters, coming from TCEforms linking to the wizard.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $P;
 
        // Information coming back from alt_doc.php script, telling what the table/id was of the newly created record.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $returnEditConf;
 
index 8fa66d1..c076f72 100644 (file)
@@ -28,13 +28,13 @@ class EditController {
        // Internal, static: GPvars
        // Wizard parameters, coming from TCEforms linking to the wizard.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $P;
 
        // Boolean; if set, the window will be closed by JavaScript
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $doClose;
 
@@ -106,7 +106,6 @@ class EditController {
         * Printing a little JavaScript to close the open window.
         *
         * @return void
-        * @todo Define visibility
         */
        public function closeWindow() {
                echo '<script language="javascript" type="text/javascript">close();</script>';
index 7c4d42c..21d5726 100644 (file)
@@ -189,7 +189,6 @@ class RteController {
         * Outputting the accumulated content to screen
         *
         * @return void
-        * @todo Define visibility
         */
        public function printContent() {
                $this->content .= $this->doc->endPage();
@@ -257,7 +256,6 @@ class RteController {
         * @param string $table Table name
         * @param integer $uid Record uid
         * @return boolean
-        * @todo Define visibility
         * @todo: Refactor to remove duplicate code (see FormsController, TableController)
         */
        public function checkEditAccess($table, $uid) {
index 169330c..0498a1a 100644 (file)
@@ -28,50 +28,50 @@ class DataPreprocessor {
        // External, static:
        // If set, the records requested are locked.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $lockRecords = 0;
 
        // Is set externally if RTE is disabled.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $disableRTE = 0;
 
        // If the pid in the command is 'prev' then $prevPageID is used as pid for the record. This is used to attach new records to other previous records eg. new pages.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $prevPageID = '';
 
        // Can be set with an array of default values for tables. First key is table name, second level keys are field names. Originally this was a GLOBAL array used internally.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $defVals = array();
 
        // If set, the processed data is overlaid the raw record.
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $addRawData = FALSE;
 
        // Internal, dynamic
        // Used to register, which items are already loaded!!
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $regTableItems = array();
 
        // This stores the record data of the loaded records
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $regTableItems_data = array();
 
        // Contains loadModules object, if used. (for reuse internally)
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $loadModules = '';
 
@@ -90,7 +90,6 @@ class DataPreprocessor {
         * @param string $operation If "new", then a record with default data is returned. Further, the $id values are meant to be PID values (or if negative, pointing to a previous record). If NOT new, then the table/ids are just pointing to an existing record!
         * @return void
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function fetchRecord($table, $idList, $operation) {
                if ((string)$idList === 'prev') {
@@ -187,7 +186,6 @@ class DataPreprocessor {
         * @param array $row The row of the current record. If NEW record, then it may be loaded with default values (by eg. fetchRecord()).
         * @return void
         * @see fetchRecord()
-        * @todo Define visibility
         */
        public function renderRecord($table, $id, $pid, $row) {
                $dateTimeFormats = $GLOBALS['TYPO3_DB']->getDateTimeFormats($table);
@@ -232,7 +230,6 @@ class DataPreprocessor {
         * @param integer $tscPID PAGE TSconfig pid
         * @return array Processed record data
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function renderRecordRaw($table, $id, $pid, $row, $TSconfig = '', $tscPID = 0) {
                if (!is_array($TSconfig)) {
@@ -288,7 +285,6 @@ class DataPreprocessor {
         * @param array $row The row array, always of the real record (also for flexforms)
         * @param string $field The field
         * @return string Modified $value
-        * @todo Define visibility
         */
        public function renderRecord_SW($data, $fieldConfig, $TSconfig, $table, $row, $field) {
                switch ((string)$fieldConfig['config']['type']) {
@@ -320,7 +316,6 @@ class DataPreprocessor {
         * @return string The processed input field value ($data)
         * @access private
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function renderRecord_groupProc($data, $fieldConfig, $TSconfig, $table, $row, $field) {
                switch ($fieldConfig['config']['internal_type']) {
@@ -372,7 +367,6 @@ class DataPreprocessor {
         * @return string The processed input field value ($data)
         * @access private
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function renderRecord_selectProc($data, $fieldConfig, $TSconfig, $table, $row, $field) {
                // Initialize:
@@ -433,7 +427,6 @@ class DataPreprocessor {
         * @return string The processed input field value ($data)
         * @access private
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function renderRecord_flexProc($data, $fieldConfig, $TSconfig, $table, $row, $field) {
                // Convert the XML data to PHP array:
@@ -470,7 +463,6 @@ class DataPreprocessor {
         * @param integer $pid PID value
         * @return array The processed version of $totalRecordContent
         * @access private
-        * @todo Define visibility
         */
        public function renderRecord_typesProc($totalRecordContent, $types_fieldConfig, $tscPID, $table, $pid) {
                foreach ($types_fieldConfig as $vconf) {
@@ -503,7 +495,6 @@ class DataPreprocessor {
         * @return string The processed input field value ($data)
         * @access private
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function renderRecord_inlineProc($data, $fieldConfig, $TSconfig, $table, $row, $field) {
                // Initialize:
@@ -543,7 +534,6 @@ class DataPreprocessor {
         * @return array Modified $dataPart array.
         * @access private
         * @see \TYPO3\CMS\Core\DataHandling\DataHandler::checkValue_flex_procInData(), renderRecord_flexProc_procInData_travDS()
-        * @todo Define visibility
         */
        public function renderRecord_flexProc_procInData($dataPart, $dataStructArray, $pParams) {
                if (is_array($dataPart)) {
@@ -567,7 +557,6 @@ class DataPreprocessor {
         * @param array $pParams Various parameters pass-through.
         * @return void
         * @see \TYPO3\CMS\Core\DataHandling\DataHandler::checkValue_flex_procInData(), renderRecord_flexProc_procInData_travDS()
-        * @todo Define visibility
         */
        public function renderRecord_flexProc_procInData_travDS(&$dataValues, $DSelements, $pParams) {
                if (is_array($DSelements)) {
@@ -624,7 +613,6 @@ class DataPreprocessor {
         * @return array Modified $dataAcc array
         * @access private
         * @see renderRecord_selectProc()
-        * @todo Define visibility
         */
        public function selectAddSpecial($dataAcc, $elements, $specialKey) {
                // Special select types:
@@ -745,7 +733,6 @@ class DataPreprocessor {
         * @return array Modified $dataAcc array
         * @access private
         * @see renderRecord_selectProc()
-        * @todo Define visibility
         */
        public function selectAddForeign($dataAcc, $elements, $fieldConfig, $field, $TSconfig, $row, $table) {
                // Init:
@@ -802,7 +789,6 @@ class DataPreprocessor {
         * @param string $table Current table name. passed on to \TYPO3\CMS\Core\Database\RelationHandler
         * @return array An array with ids of the records from the input elements array.
         * @access private
-        * @todo Define visibility
         */
        public function getDataIdList($elements, $fieldConfig, $row, $table) {
                // Use given uid (might be the uid of a workspace record)
@@ -830,7 +816,6 @@ class DataPreprocessor {
         * @return array The modified input $selItems array
         * @access private
         * @see renderRecord_selectProc()
-        * @todo Define visibility
         */
        public function procesItemArray($selItems, $config, $fieldTSConfig, $table, $row, $field) {
                $selItems = $this->addItems($selItems, $fieldTSConfig['addItems.']);
@@ -848,7 +833,6 @@ class DataPreprocessor {
         * @return array The modified input $items array
         * @access private
         * @see procesItemArray()
-        * @todo Define visibility
         */
        public function addItems($items, $iArray) {
                if (is_array($iArray)) {
@@ -871,7 +855,6 @@ class DataPreprocessor {
         * @return array The modified input $items array
         * @access private
         * @see procesItemArray()
-        * @todo Define visibility
         */
        public function procItems($items, $itemsProcFuncTSconfig, $config, $table, $row, $field) {
                $params = array();
@@ -897,7 +880,6 @@ class DataPreprocessor {
         * @param integer $id The id of the record
         * @param integer $pid The pid of the record
         * @return void
-        * @todo Define visibility
         */
        public function lockRecord($table, $id, $pid = 0) {
                if ($this->lockRecords) {
@@ -915,7 +897,6 @@ class DataPreprocessor {
         * @return void
         * @access private
         * @see renderRecord()
-        * @todo Define visibility
         */
        public function regItem($table, $id, $field, $content) {
 
@@ -927,7 +908,6 @@ class DataPreprocessor {
         * @param string Language label key
         * @return string Localized label value.
         * @access private
-        * @todo Define visibility
         */
        public function sL($in) {
                return $GLOBALS['LANG']->sL($in);
index ebccab9..4b5491c 100644 (file)
@@ -38,82 +38,85 @@ class InlineElement {
         * Reference to the calling TCEforms instance
         *
         * @var \TYPO3\CMS\Backend\Form\FormEngine
-        * @todo Define visibility
         */
        public $fObj;
 
        // Reference to $fObj->backPath
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $backPath;
 
        // Indicates if a field is rendered upon an AJAX call
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $isAjaxCall = FALSE;
 
        // The structure/hierarchy where working in, e.g. cascading inline tables
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $inlineStructure = array();
 
        // The first call of an inline type appeared on this page (pid of record)
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $inlineFirstPid;
 
        // Keys: form, object -> hold the name/id for each of them
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $inlineNames = array();
 
        // Inline data array used for JSON output
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $inlineData = array();
 
        // Expanded/collapsed states for the current BE user
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $inlineView = array();
 
        // Count the number of inline types used
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $inlineCount = 0;
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $inlineStyles = array();
 
        // How the $this->fObj->prependFormFieldNames should be set ('data' is default)
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $prependNaming = 'data';
 
        // Reference to $this->fObj->prependFormFieldNames
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $prependFormFieldNames;
 
        // Reference to $this->fObj->prependCmdFieldNames
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $prependCmdFieldNames;
 
-       // Array containing instances of hook classes called once for IRRE objects
+       /**
+        * Array containing instances of hook classes called once for IRRE objects
+        *
+        * @var array
+        */
        protected $hookObjects = array();
 
        /**
@@ -121,7 +124,6 @@ class InlineElement {
         *
         * @param \TYPO3\CMS\Backend\Form\FormEngine $tceForms Reference to an TCEforms instance
         * @return void
-        * @todo Define visibility
         */
        public function init(&$tceForms) {
                $this->fObj = $tceForms;
@@ -165,7 +167,6 @@ class InlineElement {
         * @param array $row The record data array where the value(s) for the field can be found
         * @param array $PA An array with additional configuration options.
         * @return string The HTML code for the TCEform field
-        * @todo Define visibility
         */
        public function getSingleField_typeInline($table, $field, $row, &$PA) {
                // Check the TCA configuration - if FALSE is returned, something was wrong
@@ -357,7 +358,6 @@ class InlineElement {
         * @param array $rec The table record of the child/embedded table (normaly post-processed by \TYPO3\CMS\Backend\Form\DataPreprocessor)
         * @param array $config Content of $PA['fieldConf']['config']
         * @return string The HTML code for this "foreign record
-        * @todo Define visibility
         */
        public function renderForeignRecord($parentUid, $rec, $config = array()) {
                $foreign_table = $config['foreign_table'];
@@ -497,7 +497,6 @@ class InlineElement {
         * @param array $config content of $PA['fieldConf']['config']
         * @param boolean $isVirtualRecord
         * @return string The HTML code of the header
-        * @todo Define visibility
         */
        public function renderForeignRecordHeader($parentUid, $foreign_table, $rec, $config, $isVirtualRecord = FALSE) {
                // Init:
@@ -626,7 +625,6 @@ class InlineElement {
         * @param array $rec The current record of that foreign_table
         * @param array $config (modified) TCA configuration of the field
         * @return string The HTML code with the control-icons
-        * @todo Define visibility
         */
        public function renderForeignRecordHeaderControl($parentUid, $foreign_table, $rec, $config = array(), $isVirtualRecord = FALSE) {
                // Initialize:
@@ -750,7 +748,6 @@ class InlineElement {
         * @param string $appendFormFieldNames The [<table>][<uid>] of the parent record (the intermediate table)
         * @param array $config content of $PA['fieldConf']['config']
         * @return string A HTML string with <table> tag around.
-        * @todo Define visibility
         */
        public function renderCombinationTable(&$rec, $appendFormFieldNames, $config = array()) {
                $foreign_table = $config['foreign_table'];
@@ -793,7 +790,6 @@ class InlineElement {
         * @param array $conf TCA configuration of the parent(!) field
         * @param array $uniqueIds The uids that have already been used and should be unique
         * @return string A HTML <select> box with all possible records
-        * @todo Define visibility
         */
        public function renderPossibleRecordsSelector($selItems, $conf, $uniqueIds = array()) {
                $foreign_table = $conf['foreign_table'];
@@ -817,7 +813,6 @@ class InlineElement {
         * @param array $PA An array with additional configuration options
         * @param array $uniqueIds The uids that have already been used and should be unique
         * @return string A HTML <select> box with all possible records
-        * @todo Define visibility
         */
        public function renderPossibleRecordsSelectorTypeSelect($selItems, $conf, &$PA, $uniqueIds = array()) {
                $foreign_table = $conf['foreign_table'];
@@ -874,7 +869,6 @@ class InlineElement {
         * @param array $conf TCA configuration of the parent(!) field
         * @param array $PA An array with additional configuration options
         * @return string A HTML link that opens an element browser in a new window
-        * @todo Define visibility
         */
        public function renderPossibleRecordsSelectorTypeGroupDB($conf, &$PA) {
                $foreign_table = $conf['foreign_table'];
@@ -1010,7 +1004,6 @@ class InlineElement {
         *
         * @param string $objectId The container id of the object - elements inside will be sortable
         * @return void
-        * @todo Define visibility
         */
        public function addJavaScriptSortable($objectId) {
                $this->fObj->additionalJS_post[] = '
@@ -1177,7 +1170,6 @@ class InlineElement {
         * @param string $domObjectId The calling object in hierarchy, that requested a new record.
         * @param string $foreignUid If set, the new record should be inserted after that one.
         * @return array An array to be used for JSON
-        * @todo Define visibility
         */
        public function createNewRecord($domObjectId, $foreignUid = 0) {
                // The current table - for this table we should add/import records
@@ -1317,7 +1309,6 @@ class InlineElement {
         *
         * @param string $domObjectId The calling object in hierarchy, that requested a new record.
         * @return array An array to be used for JSON
-        * @todo Define visibility
         */
        public function getRecordDetails($domObjectId) {
                // The current table - for this table we should add/import records
@@ -1417,7 +1408,6 @@ class InlineElement {
         * @param string $expand Whether this record is expanded.
         * @param string $collapse Whether this record is collapsed.
         * @return void
-        * @todo Define visibility
         */
        public function setExpandedCollapsedState($domObjectId, $expand, $collapse) {
                // Parse the DOM identifier (string), add the levels to the structure stack (array), but don't load TCA config
@@ -1464,7 +1454,6 @@ class InlineElement {
         * @param array $PA An array with additional configuration options.
         * @param array $config (Redundant) content of $PA['fieldConf']['config'] (for convenience)
         * @return array The records related to the parent item as associative array.
-        * @todo Define visibility
         */
        public function getRelatedRecords($table, $field, $row, &$PA, $config) {
                $language = 0;
@@ -1616,7 +1605,6 @@ class InlineElement {
         * @param array $conf An array with additional configuration options.
         * @param string $checkForConfField For which field in the foreign_table the possible records should be fetched
         * @return mixed Array of possible record items; FALSE if type is "group/db", then everything could be "possible
-        * @todo Define visibility
         */
        public function getPossibleRecords($table, $field, $row, $conf, $checkForConfField = 'foreign_selector') {
                // ctrl configuration from TCA:
@@ -1676,7 +1664,6 @@ class InlineElement {
         * @param array $conf The TCA field configuration of the inline field to be rendered
         * @param boolean $splitValue For usage with group/db, values come like "tx_table_123|Title%20abc", but we need "tx_table" and "123
         * @return array The uids, that have been used already and should be used unique
-        * @todo Define visibility
         */
        public function getUniqueIds($records, $conf = array(), $splitValue = FALSE) {
                $uniqueIds = array();
@@ -1730,7 +1717,6 @@ class InlineElement {
         * @param string $uid The uid of the record to fetch, or the pid if a new record should be created
         * @param string $cmd The command to perform, empty or 'new'
         * @return array A record row from the database post-processed by \TYPO3\CMS\Backend\Form\DataPreprocessor
-        * @todo Define visibility
         */
        public function getRecord($pid, $table, $uid, $cmd = '') {
                // Fetch workspace version of a record (if any):
@@ -1761,7 +1747,6 @@ class InlineElement {
         * @param integer $pid The pid of the page the record should be stored (only relevant for NEW records)
         * @param string $table The table to fetch data from (= foreign_table)
         * @return array A record row from the database post-processed by \TYPO3\CMS\Backend\Form\DataPreprocessor
-        * @todo Define visibility
         */
        public function getNewRecord($pid, $table) {
                $rec = $this->getRecord($pid, $table, $pid, 'new');
@@ -1785,7 +1770,6 @@ class InlineElement {
         * @param array $config The TCA-configuration of the inline field
         * @param array $parameters The full parameter array (PA)
         * @return void
-        * @todo Define visibility
         */
        public function pushStructure($table, $uid, $field = '', $config = array(), array $parameters = array()) {
                $structure = array(
@@ -1814,7 +1798,6 @@ class InlineElement {
         * Remove the item on top of the structure stack and return it.
         *
         * @return array The top item of the structure stack - array(<table>,<uid>,<field>,<config>)
-        * @todo Define visibility
         */
        public function popStructure() {
                $popItem = NULL;
@@ -1833,7 +1816,6 @@ class InlineElement {
         * inline structure stack.
         *
         * @return void
-        * @todo Define visibility
         */
        public function updateStructureNames() {
                $current = $this->getStructureLevel(-1);
@@ -1854,7 +1836,6 @@ class InlineElement {
         * @param array $levelData Array of a level of the structure stack (containing the keys table, uid and field)
         * @param string $disposal How the structure name is used (e.g. as <div id="..."> or <input name="..." />)
         * @return string The name/id of that level, to be used for HTML output
-        * @todo Define visibility
         */
        public function getStructureItemName($levelData, $disposal = self::Disposal_AttributeId) {
                $name = NULL;
@@ -1893,7 +1874,6 @@ class InlineElement {
         *
         * @param integer $level Which level to return
         * @return array The item of the stack at the requested level
-        * @todo Define visibility
         */
        public function getStructureLevel($level) {
                $level = $this->calculateStructureLevel($level);
@@ -1930,8 +1910,7 @@ class InlineElement {
         * An identifier looks like "<table>-<uid>-<field>".
         *
         * @param integer $structureDepth How much levels to output, beginning from the top of the stack
-        * @return      string          The path of identifiers
-        * @todo Define visibility
+        * @return string The path of identifiers
         */
        public function getStructurePath($structureDepth = -1) {
                $structureLevels = array();
@@ -1956,7 +1935,6 @@ class InlineElement {
         * @param string $domObjectId The DOM object-id
         * @param boolean $loadConfig Load the TCA configuration for that level (default: TRUE)
         * @return void
-        * @todo Define visibility
         */
        public function parseStructureString($string, $loadConfig = TRUE) {
                $unstable = array();
@@ -2020,7 +1998,6 @@ class InlineElement {
         * @param string $field The field name which this element is supposed to edit
         * @param array $row The record data array of the parent
         * @return boolean If critical configuration errors were found, FALSE is returned
-        * @todo Define visibility
         */
        public function checkConfiguration(&$config) {
                $foreign_table = $config['foreign_table'];
@@ -2064,7 +2041,6 @@ class InlineElement {
         * @param string $table The table to check access for
         * @param string $theUid The record uid of the table
         * @return boolean Returns TRUE is the user has access, or FALSE if not
-        * @todo Define visibility
         */
        public function checkAccess($cmd, $table, $theUid) {
                // Checking if the user has permissions? (Only working as a precaution, because the final permission check is always down in TCE. But it's good to notify the user on beforehand...)
@@ -2135,7 +2111,6 @@ class InlineElement {
         * @param array $compare Keys and values to compare to the ['config'] part of the top level of the stack
         * @return boolean Whether the comparison was successful
         * @see arrayCompareComplex
-        * @todo Define visibility
         */
        public function compareStructureConfiguration($compare) {
                $level = $this->getStructureLevel(-1);
@@ -2148,7 +2123,6 @@ class InlineElement {
         *
         * @param string $string A transferData reference string, containing the uid
         * @return string The normalized uid
-        * @todo Define visibility
         */
        public function normalizeUid($string) {
                $parts = explode('|', $string);
@@ -2162,7 +2136,6 @@ class InlineElement {
         * @param array $styleAttrs Attributes for the style argument in the table tag
         * @param array $tableAttrs Attributes for the table tag (like width, border, etc.)
         * @return string The wrapped HTML code
-        * @todo Define visibility
         */
        public function wrapFormsSection($section, $styleAttrs = array(), $tableAttrs = array()) {
                $style = '';
@@ -2194,7 +2167,6 @@ class InlineElement {
         * @param string $table The table to check
         * @param string $field The field on this table to check
         * @return boolean Is inline child and field is responsible for the label
-        * @todo Define visibility
         */
        public function isInlineChildAndLabelField($table, $field) {
                $level = $this->getStructureLevel(-1);
@@ -2211,7 +2183,6 @@ class InlineElement {
         * (count($this->inlineStructure['stable'])
         *
         * @return integer The depth of the structure stack
-        * @todo Define visibility
         */
        public function getStructureDepth() {
                return count($this->inlineStructure['stable']);
@@ -2248,7 +2219,6 @@ class InlineElement {
         * @param array $searchArray The array with keys and values to search for
         * @param string $type Use '%AND' or '%OR' for comparison
         * @return boolean The result of the comparison
-        * @todo Define visibility
         */
        public function arrayCompareComplex($subjectArray, $searchArray, $type = '') {
                $localMatches = 0;
@@ -2305,7 +2275,6 @@ class InlineElement {
         *
         * @param mixed $object The object to be checked
         * @return boolean Returns TRUE, if the object is an associative array
-        * @todo Define visibility
         */
        public function isAssociativeArray($object) {
                return is_array($object) && count($object) && array_keys($object) !== range(0, sizeof($object) - 1) ? TRUE : FALSE;
@@ -2318,7 +2287,6 @@ class InlineElement {
         * @param array $haystack The array the element should be removed from.
         * @param mixed $strict Search elements strictly.
         * @return array The array $haystack without the $needle
-        * @todo Define visibility
         */
        public function removeFromArray($needle, $haystack, $strict = NULL) {
                $pos = array_search($needle, $haystack, $strict);
@@ -2335,7 +2303,6 @@ class InlineElement {
         *
         * @param array $possibleRecords The possibleRecords array (for select fields)
         * @return mixed A flat array with key=uid, value=label; if $possibleRecords isn't an array, FALSE is returned.
-        * @todo Define visibility
         */
        public function getPossibleRecordsFlat($possibleRecords) {
                $flat = FALSE;
@@ -2353,7 +2320,6 @@ class InlineElement {
         *
         * @param array $conf TCA configuration of the parent(!) field
         * @return array Associative array with the keys 'PA' and 'type', both are FALSE if the selector was not valid.
-        * @todo Define visibility
         */
        public function getPossibleRecordsSelectorConfig($conf, $field = '') {
                $foreign_table = $conf['foreign_table'];
@@ -2394,7 +2360,6 @@ class InlineElement {
         *
         * @param array $config TCE configuration of the selector
         * @return mixed The type of the selector, 'select' or 'groupdb' - FALSE not valid
-        * @todo Define visibility
         */
        public function getPossibleRecordsSelectorType($config) {
                $type = FALSE;
@@ -2415,7 +2380,6 @@ class InlineElement {
         * @param array $row The record row from the database
         * @param array $config TCA configuration of the field
         * @return boolean Determines whether the field should be skipped.
-        * @todo Define visibility
         */
        public function skipField($table, $field, $row, $config) {
                $skipThisField = FALSE;
@@ -2465,7 +2429,6 @@ class InlineElement {
         * @param string $table Name of the child table
         * @param integer $uid uid of the the child record
         * @return boolean TRUE=expand, FALSE=collapse
-        * @todo Define visibility
         */
        public function getExpandedCollapsedState($table, $uid) {
                if (isset($this->inlineView[$table]) && is_array($this->inlineView[$table])) {
@@ -2482,7 +2445,6 @@ class InlineElement {
         * @param array $uc The uc array to be processed and saved (by reference)
         * @param \TYPO3\CMS\Core\DataHandling\DataHandler $tce Instance of TCEmain that saved data before
         * @return void
-        * @todo Define visibility
         */
        public function updateInlineView(&$uc, $tce) {
                if (isset($uc['inlineView']) && is_array($uc['inlineView'])) {
@@ -2521,7 +2483,6 @@ class InlineElement {
         * Returns the the margin in pixels, that is used for each new inline level.
         *
         * @return integer A pixel value for the margin of each new inline level.
-        * @todo Define visibility
         */
        public function getLevelMargin() {
                $margin = ($this->inlineStyles['margin-right'] + 1) * 2;
index 9f2c3d5..3ccdddf 100644 (file)
@@ -29,7 +29,6 @@ class ValueSlider {
         * @param array $params
         * @param \TYPO3\CMS\Backend\Form\FormEngine $pObj
         * @return string
-        * @todo Define visibility
         */
        public function renderWizard(&$params, &$pObj) {
                $pObj->loadJavascriptLib('sysext/backend/Resources/Public/JavaScript/ValueSlider.js');
index 97c5994..28263ee 100644 (file)
@@ -35,54 +35,54 @@ class RecordHistory {
        // External, static:
        // Maximum number of sys_history steps to show.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $maxSteps = 20;
 
        // display diff or not (0-no diff, 1-inline)
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $showDiff = 1;
 
        // on a pages table - show sub elements as well.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $showSubElements = 1;
 
        // show inserts and deletes as well
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $showInsertDelete = 1;
 
        // Internal, GPvars
        // Element reference, syntax [tablename]:[uid]
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $element;
 
        // syslog ID which is not shown anymore
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $lastSyslogId;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $returnUrl;
 
        // Internal
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $changeLog;
 
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $showMarked = FALSE;
 
@@ -98,8 +98,6 @@ class RecordHistory {
 
        /**
         * Constructor for the class
-        *
-        * @todo Define visibility
         */
        public function __construct() {
                // GPvars:
@@ -116,7 +114,6 @@ class RecordHistory {
         * It detects incoming variables like element reference, history element uid etc. and renders the correct screen.
         *
         * @return HTML content for the module
-        * @todo Define visibility
         */
        public function main() {
                $content = '';
@@ -165,7 +162,6 @@ class RecordHistory {
         *
         * @param integer $uid Uid of sys_history entry
         * @return void
-        * @todo Define visibility
         */
        public function toggleHighlight($uid) {
                $uid = (int)$uid;
@@ -180,7 +176,6 @@ class RecordHistory {
         * @param array $diff Diff array to rollback
         * @return void
         * @access private
-        * @todo Define visibility
         */
        public function performRollback($diff) {
                if (!$this->rollbackFields) {
@@ -295,7 +290,6 @@ class RecordHistory {
         * Displays settings
         *
         * @return string HTML code to modify settings
-        * @todo Define visibility
         */
        public function displaySettings() {
                // Get current selection from UC, merge data, write it back to UC
@@ -367,7 +361,6 @@ class RecordHistory {
         * Shows the full change log
         *
         * @return string HTML for list, wrapped in a table.
-        * @todo Define visibility
         */
        public function displayHistory() {
                $lines = array();
@@ -478,7 +471,6 @@ class RecordHistory {
         *
         * @param array $diff Difference array
         * @return string HTML output
-        * @todo Define visibility
         */
        public function displayMultipleDiff($diff) {
                $content = '';
@@ -525,7 +517,6 @@ class RecordHistory {
         * @param integer $rollbackUid If set to UID of record, display rollback links
         * @return string HTML table
         * @access private
-        * @todo Define visibility
         */
        public function renderDiff($entry, $table, $rollbackUid = 0) {
                $lines = array();
@@ -564,7 +555,6 @@ class RecordHistory {
         * Creates a diff between the current version of the records and the selected version
         *
         * @return array Diff for many elements, 0 if no changelog is found
-        * @todo Define visibility
         */
        public function createMultipleDiff() {
                $insertsDeletes = array();
@@ -626,7 +616,6 @@ class RecordHistory {
         * Creates change log including sub-elements, filling $this->changeLog
         *
         * @return integer
-        * @todo Define visibility
         */
        public function createChangeLog() {
                $elParts = explode(':', $this->element);
@@ -665,7 +654,6 @@ class RecordHistory {
         * @param string $table DB table name
         * @param integer $uid UID of record
         * @return array history data of the record
-        * @todo Define visibility
         */
        public function getHistoryData($table, $uid) {
                // If table is found in $GLOBALS['TCA']:
@@ -745,7 +733,6 @@ class RecordHistory {
         * @param string $table
         * @param string $uid
         * @return string
-        * @todo Define visibility
         */
        public function generateTitle($table, $uid) {
                $out = $table . ':' . $uid;
@@ -763,7 +750,6 @@ class RecordHistory {
         * @param string $alt Optional, alternative label and title tag of image
         * @param integer $type Optional, type of rollback: 0 - ALL; 1 - element; 2 - field
         * @return string HTML output
-        * @todo Define visibility
         */
        public function createRollbackLink($key, $alt = '', $type = 0) {
                return $this->linkPage('<img ' . IconUtility::skinImg('', ('gfx/revert_' . $type . '.gif'), 'width="33" height="33"') . ' alt="' . $alt . '" title="' . $alt . '" align="middle" />', array('rollbackFields' => $key));
@@ -778,7 +764,6 @@ class RecordHistory {
         * @param string $title Possible title.
         * @return string Link.
         * @access private
-        * @todo Define visibility
         */
        public function linkPage($str, $inparams = array(), $anchor = '', $title = '') {
                // Setting default values based on GET parameters:
@@ -799,7 +784,6 @@ class RecordHistory {
         * @param array $dataArray The data array
         * @return array The modified data array
         * @access private
-        * @todo Define visibility
         */
        public function removeFilefields($table, $dataArray) {
                if ($GLOBALS['TCA'][$table]) {
@@ -818,7 +802,6 @@ class RecordHistory {
         * @param string $table Table of input element
         * @param integer $uid UID of record
         * @return integer converted UID of record
-        * @todo Define visibility
         */
        public function resolveElement($table, $uid) {
                if (isset($GLOBALS['TCA'][$table])) {
@@ -833,7 +816,6 @@ class RecordHistory {
         * Resolve sh_uid (used from log)
         *
         * @return void
-        * @todo Define visibility
         */
        public function resolveShUid() {
                if ($this->getArgument('sh_uid')) {
index ea94267..012b9f2 100644 (file)
@@ -223,7 +223,6 @@ class BaseScriptClass {
         *
         * @return void
         * @see init(), $MOD_MENU, $MOD_SETTINGS, \TYPO3\CMS\Backend\Utility\BackendUtility::getModuleData(), mergeExternalItems()
-        * @todo Define visibility
         */
        public function menuConfig() {
                // Page/be_user TSconfig settings and blinding of menu-items
@@ -242,7 +241,6 @@ class BaseScriptClass {
         * @return array Modified array part.
         * @access private
         * @see \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::insertModuleFunction(), menuConfig()
-        * @todo Define visibility
         */
        public function mergeExternalItems($modName, $menuKey, $menuArr) {
                $mergeArray = $GLOBALS['TBE_MODULES_EXT'][$modName]['MOD_MENU'][$menuKey];
@@ -265,7 +263,6 @@ class BaseScriptClass {
         * @return void
         * @see getExternalItemConfig(), $include_once, init()
         * @deprecated since 6.2. Instead of this include_once array, extensions should use auto-loading
-        * @todo Define visibility
         */
        public function handleExternalFunctionValue($MM_key = 'function', $MS_value = NULL) {
                if ($MS_value === NULL) {
@@ -286,7 +283,6 @@ class BaseScriptClass {
         * @param string $value Optionally the value-key to fetch from the array that would otherwise have been returned if this value was not set. Look source...
         * @return mixed The value from the TBE_MODULES_EXT array.
         * @see handleExternalFunctionValue()
-        * @todo Define visibility
         */
        public function getExternalItemConfig($modName, $menuKey, $value = '') {
                return (string)$value !== '' ? $GLOBALS['TBE_MODULES_EXT'][$modName]['MOD_MENU'][$menuKey][$value] : $GLOBALS['TBE_MODULES_EXT'][$modName]['MOD_MENU'][$menuKey];
@@ -301,7 +297,6 @@ class BaseScriptClass {
         *
         * @return void
         * @see handleExternalFunctionValue(), \TYPO3\CMS\Core\Utility\ExtensionManagementUtility::insertModuleFunction(), $extObj
-        * @todo Define visibility
         */
        public function checkExtObj() {
                if (is_array($this->extClassConf) && $this->extClassConf['name']) {
@@ -316,7 +311,6 @@ class BaseScriptClass {
         * Calls the checkExtObj function in sub module if present.
         *
         * @return void
-        * @todo Define visibility
         */
        public function checkSubExtObj() {
                if (is_object($this->extObj)) {
@@ -331,7 +325,6 @@ class BaseScriptClass {
         * $this->pObj->doc->JScode = $this->pObj->doc->wrapScriptTags(' ...
         *
         * @return void
-        * @todo Define visibility
         */
        public function extObjHeader() {
                if (is_callable(array($this->extObj, 'head'))) {
@@ -343,7 +336,6 @@ class BaseScriptClass {
         * Calls the 'main' function inside the "Function menu module" if present
         *
         * @return void
-        * @todo Define visibility
         */
        public function extObjContent() {
                $this->extObj->pObj = $this;
index 9474d7e..ade8cbb 100644 (file)
@@ -34,117 +34,118 @@ abstract class AbstractRecordList {
        // Used in this class:
        // default Max items shown
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $iLimit = 10;
 
        // OBSOLETE - NOT USED ANYMORE. leftMargin
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $leftMargin = 0;
 
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $showIcon = 1;
 
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $no_noWrap = 0;
 
        // If set this is <td> CSS-classname for odd columns in addElement. Used with db_layout / pages section
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $oddColumnsCssClass = '';
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $backPath = '';
 
        // Decides the columns shown. Filled with values that refers to the keys of the data-array. $this->fieldArray[0] is the title column.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $fieldArray = array();
 
        // Keys are fieldnames and values are td-parameters to add in addElement(), please use $addElement_tdCSSClass for CSS-classes;
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $addElement_tdParams = array();
 
        // Keys are fieldnames and values are td-css-classes to add in addElement();
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $addElement_tdCssClass = array();
 
        // Not used in this class - but maybe extension classes...
        // Max length of strings
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $fixedL = 30;
 
        /**
         * Script URL
+        *
+        * @var string
         */
        public $thisScript = '';
 
        // Set to zero, if you don't want a left-margin with addElement function
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $setLMargin = 1;
 
        // Counter increased for each element. Used to index elements for the JavaScript-code that transfers to the clipboard
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $counter = 0;
 
        // This could be set to the total number of items. Used by the fwd_rew_navigation...
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $totalItems = '';
 
        // Internal (used in this class.)
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $firstElementNumber = 0;
 
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $eCounter = 0;
 
        // String with accumulated HTML content
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $HTMLcode = '';
 
        // Contains page translation languages
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $pageOverlays = array();
 
        // Contains sys language icons and titles
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $languageIconTitles = array();
 
        /**
         * @var \TYPO3\CMS\Backend\Configuration\TranslationConfigurationProvider
-        * @todo Define visibility
         */
        public $translateTools;
 
@@ -187,7 +188,6 @@ abstract class AbstractRecordList {
         * @param integer OBSOLETE - NOT USED ANYMORE. $lMargin is the leftMargin (integer)
         * @param string $altLine Is the HTML <img>-tag for an alternative 'gfx/ol/line.gif'-icon (used in the top)
         * @return string HTML content for the table row
-        * @todo Define visibility
         */
        public function addElement($h, $icon, $data, $trParams = '', $lMargin = '', $altLine = '') {
                $noWrap = $this->no_noWrap ? '' : ' nowrap="nowrap"';
@@ -265,7 +265,6 @@ abstract class AbstractRecordList {
         * Dummy function, used to write the top of a table listing.
         *
         * @return void
-        * @todo Define visibility
         */
        public function writeTop() {
 
@@ -275,7 +274,6 @@ abstract class AbstractRecordList {
         * Finishes the list with the "stopper"-gif, adding the HTML code for that item to the internal ->HTMLcode string
         *
         * @return void
-        * @todo Define visibility
         * @deprecated since 6.2, will be removed 2 versions later - Function not needed anymore
         */
        public function writeBottom() {
@@ -287,7 +285,6 @@ abstract class AbstractRecordList {
         *
         * @param string $table Table name
         * @return array array([boolean], [HTML]) where [boolean] is 1 for reverse element, [HTML] is the table-row code for the element
-        * @todo Define visibility
         */
        public function fwd_rwd_nav($table = '') {
                $code = '';
@@ -320,7 +317,6 @@ abstract class AbstractRecordList {
         * @param string $table Table name
         * @return string
         * @access private
-        * @todo Define visibility
         */
        public function fwd_rwd_HTML($type, $pointer, $table = '') {
                $content = '';
@@ -343,7 +339,6 @@ abstract class AbstractRecordList {
         *
         * @param string $altId Alternative id value. Enter blank string for the current id ($this->id)
         * @return string URL
-        * @todo Define visibility
         */
        public function listURL($altId = '') {
                return $this->getThisScript() . 'id=' . ($altId !== '' ? $altId : $this->id);
@@ -353,7 +348,6 @@ abstract class AbstractRecordList {
         * Returning JavaScript for ClipBoard functionality.
         *
         * @return string
-        * @todo Define visibility
         */
        public function CBfunctions() {
                return '
@@ -392,7 +386,6 @@ abstract class AbstractRecordList {
         * Initializes page languages and icons
         *
         * @return void
-        * @todo Define visibility
         */
        public function initializeLanguages() {
                // Look up page overlays:
@@ -406,7 +399,6 @@ abstract class AbstractRecordList {
         * @param integer $sys_language_uid Sys language uid
         * @param boolean $addAsAdditionalText If set to true, only the flag is returned
         * @return string Language icon
-        * @todo Define visibility
         */
        public function languageFlag($sys_language_uid, $addAsAdditionalText = TRUE) {
                $out = '';
index 6039d22..e3ecc03 100644 (file)
@@ -44,7 +44,6 @@ class ElementBrowserRecordList extends \TYPO3\CMS\Recordlist\RecordList\Database
         * Initializes the script path
         *
         * @return void
-        * @todo Define visibility
         */
        public function __construct() {
                parent::__construct();
@@ -58,7 +57,6 @@ class ElementBrowserRecordList extends \TYPO3\CMS\Recordlist\RecordList\Database
         * @param string $table If this is "-1" then $this->table is used, otherwise the value of the input variable.
         * @param string $exclList Commalist of fields NOT to pass as parameters (currently "sortField" and "sortRev")
         * @return string Query-string for URL
-        * @todo Define visibility
         */
        public function listURL($altId = '', $table = -1, $exclList = '') {
                return $this->getThisScript() . 'id=' . ($altId !== '' ? $altId : $this->id) . '&table=' . rawurlencode(($table == -1 ? $this->table : $table)) . ($this->thumbs ? '&imagemode=' . $this->thumbs : '') . ($this->searchString ? '&search_field=' . rawurlencode($this->searchString) : '') . ($this->searchLevels ? '&search_levels=' . rawurlencode($this->searchLevels) : '') . ((!$exclList || !GeneralUtility::inList($exclList, 'sortField')) && $this->sortField ? '&sortField=' . rawurlencode($this->sortField) : '') . ((!$exclList || !GeneralUtility::inList($exclList, 'sortRev')) && $this->sortRev ? '&sortRev=' . rawurlencode($this->sortRev) : '') . $this->ext_addP();
@@ -68,7 +66,6 @@ class ElementBrowserRecordList extends \TYPO3\CMS\Recordlist\RecordList\Database
         * Returns additional, local GET parameters to include in the links of the record list.
         *
         * @return string
-        * @todo Define visibility
         */
        public function ext_addP() {
                $str = '&act=' . $GLOBALS['SOBE']->browser->act . '&mode=' . $GLOBALS['SOBE']->browser->mode . '&expandPage=' . $GLOBALS['SOBE']->browser->expandPage . '&bparams=' . rawurlencode($GLOBALS['SOBE']->browser->bparams);
@@ -83,7 +80,6 @@ class ElementBrowserRecordList extends \TYPO3\CMS\Recordlist\RecordList\Database
         * @param string $code Title string
         * @param array $row Records array (from table name)
         * @return string
-        * @todo Define visibility
         */
        public function linkWrapItems($table, $uid, $code, $row) {
                if (!$code) {
@@ -151,7 +147,6 @@ class ElementBrowserRecordList extends \TYPO3\CMS\Recordlist\RecordList\Database
         *
         * @return void
         * @see fieldSelectBox
-        * @todo Define visibility
         */
        public function generateList() {
                $this->allFields = TRUE;
index 13b415a..233aad1 100644 (file)
@@ -28,14 +28,14 @@ class AbstractRte {
        // Internal, dynamic:
        // Error messages regarding non-availability is collected here.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $errorLog = array();
 
        // Internal, static:
        // Set this to the extension key of the RTE so it can identify itself.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $ID = '';
 
@@ -51,7 +51,6 @@ class AbstractRte {
         * If there are reasons why the RTE cannot be displayed you simply enter them as text in ->errorLog
         *
         * @return boolean TRUE if this RTE object offers an RTE in the current browser environment
-        * @todo Define visibility
         */
        public function isAvailable() {
                return TRUE;
@@ -72,7 +71,6 @@ class AbstractRte {
         * @param string $RTErelPath Relative path for images/links in RTE; this is used when the RTE edits content from static files where the path of such media has to be transformed forth and back!
         * @param integer $thePidValue PID value of record (true parent page id)
         * @return string HTML code for RTE!
-        * @todo Define visibility
         */
        public function drawRTE(&$pObj, $table, $field, $row, $PA, $specConf, $thisConfig, $RTEtypeVal, $RTErelPath, $thePidValue) {
                // Transform value:
@@ -101,7 +99,6 @@ class AbstractRte {
         * @param string $RTErelPath Relative path for images/links in RTE; this is used when the RTE edits content from static files where the path of such media has to be transformed forth and back!
         * @param integer $pid PID value of record (true parent page id)
         * @return string Transformed content
-        * @todo Define visibility
         */
        public function transformContent($dirRTE, $value, $table, $field, $row, $specConf, $thisConfig, $RTErelPath, $pid) {
                if ($specConf['rte_transform']) {
@@ -129,7 +126,6 @@ class AbstractRte {
         *
         * @param string $fieldName Field name of the RTE field.
         * @return string <input> field of type "hidden" with a flag telling the TCEmain that this fields content should be transformed back to database state.
-        * @todo Define visibility
         */
        public function triggerField($fieldName) {
                $triggerFieldName = preg_replace('/\\[([^]]+)\\]$/', '[_TRANSFORM_\\1]', $fieldName);
index 6acc259..c722540 100644 (file)
@@ -25,7 +25,7 @@ namespace TYPO3\CMS\Backend\Template;
 class BigDocumentTemplate extends \TYPO3\CMS\Backend\Template\DocumentTemplate {
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $divClass = 'typo3-bigDoc';
 
index edca51f..3f43f63 100644 (file)
@@ -18,13 +18,11 @@ namespace TYPO3\CMS\Backend\Tree\Pagetree;
  *
  * @author Susanne Moog <typo3@susanne-moog.de>
  */
-interface IndicatorProviderInterface
-{
+interface IndicatorProviderInterface {
        /**
         * Returns the indicator html code
         *
         * @return string
-        * @todo Define visibility
         */
        public function getIndicator();
 
index 4b3f033..7400d9e 100644 (file)
@@ -29,44 +29,44 @@ abstract class AbstractTreeView {
        // EXTERNAL, static:
        // If set, the first element in the tree is always expanded.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $expandFirst = 0;
 
        // If set, then ALL items will be expanded, regardless of stored settings.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $expandAll = 0;
 
        // Holds the current script to reload to.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $thisScript = '';
 
        // Which HTML attribute to use: alt/title. See init().
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $titleAttrib = 'title';
 
        // If TRUE, no context menu is rendered on icons. If set to "titlelink" the
        // icon is linked as the title is.
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $ext_IconMode = FALSE;
 
        // If set, the id of the mounts will be added to the internal ids array
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $addSelfId = 0;
 
        // Used if the tree is made of records (not folders for ex.)
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $title = 'no title';
 
@@ -74,7 +74,7 @@ abstract class AbstractTreeView {
        // This cannot be enabled by default because it will destroy many applications
        // where another title attribute is in fact applied later.
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $showDefaultTitleAttribute = FALSE;
 
@@ -82,7 +82,7 @@ abstract class AbstractTreeView {
        // highlighted in yellow. This might be too expensive in terms
        // of processing power.
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $highlightPagesWithVersions = TRUE;
 
@@ -91,7 +91,6 @@ abstract class AbstractTreeView {
         * Done by default in init()
         *
         * @var \TYPO3\CMS\Core\Authentication\BackendUserAuthentication
-        * @todo Define visibility
         */
        public $BE_USER = '';
 
@@ -102,7 +101,7 @@ abstract class AbstractTreeView {
         * values are the ID of the root element (COULD be zero or anything else.
         * For pages that would be the uid of the page, zero for the pagetree root.)
         *
-        * @todo Define visibility
+        * @var string
         */
        public $MOUNTS = '';
 
@@ -110,14 +109,14 @@ abstract class AbstractTreeView {
         * Database table to get the tree data from.
         * Leave blank if data comes from an array.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $table = '';
 
        /**
         * Defines the field of $table which is the parent id field (like pid for table pages).
         *
-        * @todo Define visibility
+        * @var string
         */
        public $parentField = 'pid';
 
@@ -126,7 +125,7 @@ abstract class AbstractTreeView {
         * Only makes sense when $this->table is set.
         *
         * @see init()
-        * @todo Define visibility
+        * @var string
         */
        public $clause = '';
 
@@ -135,7 +134,7 @@ abstract class AbstractTreeView {
         * Only makes sense when $this->table is set.
         *
         * @see init()
-        * @todo Define visibility
+        * @var string
         */
        public $orderByFields = '';
 
@@ -144,7 +143,7 @@ abstract class AbstractTreeView {
         * Make SURE that these fields names listed herein are actually possible to select from $this->table (if that variable is set to a TCA table name)
         *
         * @see addField()
-        * @todo Define visibility
+        * @var array
         */
        public $fieldArray = array('uid', 'title');
 
@@ -152,7 +151,7 @@ abstract class AbstractTreeView {
         * List of other fields which are ALLOWED to set (here, based on the "pages" table!)
         *
         * @see addField()
-        * @todo Define visibility
+        * @var array
         */
        public $defaultList = 'uid,pid,tstamp,sorting,deleted,perms_userid,perms_groupid,perms_user,perms_group,perms_everybody,crdate,cruser_id';
 
@@ -163,7 +162,7 @@ abstract class AbstractTreeView {
         * MUST NOT contain underscore chars.
         * etc.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $treeName = '';
 
@@ -173,28 +172,28 @@ abstract class AbstractTreeView {
         * Needs to be unique if multiple trees are on one HTML page.
         *
         * @see printTree()
-        * @todo Define visibility
+        * @var string
         */
        public $domIdPrefix = 'row';
 
        /**
         * Back path for icons
         *
-        * @todo Define visibility
+        * @var string
         */
        public $backPath;
 
        /**
         * Icon file path.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $iconPath = '';
 
        /**
         * Icon file name for item icons.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $iconName = 'default.gif';
 
@@ -202,14 +201,14 @@ abstract class AbstractTreeView {
         * If TRUE, HTML code is also accumulated in ->tree array during rendering of the tree.
         * If 2, then also the icon prefix code (depthData) is stored
         *
-        * @todo Define visibility
+        * @var int
         */
        public $makeHTML = 1;
 
        /**
         * If TRUE, records as selected will be stored internally in the ->recs array
         *
-        * @todo Define visibility
+        * @var int
         */
        public $setRecs = 0;
 
@@ -217,7 +216,7 @@ abstract class AbstractTreeView {
         * Sets the associative array key which identifies a new sublevel if arrays are used for trees.
         * This value has formerly been "subLevel" and "--sublevel--"
         *
-        * @todo Define visibility
+        * @var string
         */
        public $subLevelID = '_SUB_LEVEL';
 
@@ -227,70 +226,70 @@ abstract class AbstractTreeView {
        // For record trees:
        // one-dim array of the uid's selected.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $ids = array();
 
        // The hierarchy of element uids
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $ids_hierarchy = array();
 
        // The hierarchy of versioned element uids
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $orig_ids_hierarchy = array();
 
        // Temporary, internal array
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $buffer_idH = array();
 
        // For FOLDER trees:
        // Special UIDs for folders (integer-hashes of paths)
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $specUIDmap = array();
 
        // For arrays:
        // Holds the input data array
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $data = FALSE;
 
        // Holds an index with references to the data array.
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $dataLookup = FALSE;
 
        // For both types
        // Tree is accumulated in this variable
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $tree = array();
 
        // Holds (session stored) information about which items in the tree are unfolded and which are not.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $stored = array();
 
        // Points to the current mountpoint key
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $bank = 0;
 
        // Accumulates the displayed records.
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $recs = array();
 
@@ -319,7 +318,6 @@ abstract class AbstractTreeView {
         * @param string Record WHERE clause
         * @param string Record ORDER BY field
         * @return void
-        * @todo Define visibility
         */
        public function init($clause = '', $orderByFields = '') {
                // Setting BE_USER by default
@@ -351,7 +349,6 @@ abstract class AbstractTreeView {
         *
         * @param string $treeName Default is the table name. Underscores are stripped.
         * @return void
-        * @todo Define visibility
         */
        public function setTreeName($treeName = '') {
                $this->treeName = $treeName ?: $this->treeName;
@@ -365,7 +362,6 @@ abstract class AbstractTreeView {
         * @param string $field Field name to
         * @param boolean $noCheck If set, the fieldname will be set no matter what. Otherwise the field name must either be found as key in $GLOBALS['TCA'][$table]['columns'] or in the list ->defaultList
         * @return void
-        * @todo Define visibility
         */
        public function addField($field, $noCheck = 0) {
                if ($noCheck || is_array($GLOBALS['TCA'][$this->table]['columns'][$field]) || GeneralUtility::inList($this->defaultList, $field)) {
@@ -377,7 +373,6 @@ abstract class AbstractTreeView {
         * Resets the tree, recs, ids, ids_hierarchy and orig_ids_hierarchy internal variables. Use it if you need it.
         *
         * @return void
-        * @todo Define visibility
         */
        public function reset() {
                $this->tree = array();
@@ -397,7 +392,6 @@ abstract class AbstractTreeView {
         * Is based on the mounts found in the internal array ->MOUNTS (set in the constructor)
         *
         * @return string HTML code for the browsable tree
-        * @todo Define visibility
         */
        public function getBrowsableTree() {
                // Get stored tree structure AND updating it if needed according to incoming PM GET var.
@@ -453,7 +447,6 @@ abstract class AbstractTreeView {
         *
         * @param array $treeArr "tree-array" - if blank string, the internal ->tree array is used.
         * @return string The HTML code for the tree
-        * @todo Define visibility
         */
        public function printTree($treeArr = '') {
                $titleLen = (int)$this->BE_USER->uc['titleLen'];
@@ -501,7 +494,6 @@ abstract class AbstractTreeView {
         * @return string Image tag with the plus/minus icon.
         * @access private
         * @see \TYPO3\CMS\Backend\Tree\View\PageTreeView::PMicon()
-        * @todo Define visibility
         */
        public function PMicon($row, $a, $c, $nextCount, $exp) {
                $PM = $nextCount ? ($exp ? 'minus' : 'plus') : 'join';
@@ -523,7 +515,6 @@ abstract class AbstractTreeView {
         * @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 = '') {
                if ($this->thisScript) {
@@ -546,7 +537,6 @@ abstract class AbstractTreeView {
         * @param integer $bank Bank pointer (which mount point number)
         * @return string
         * @access private
-        * @todo Define visibility
         */
        public function wrapTitle($title, $row, $bank = 0) {
                $aOnClick = 'return jumpTo(\'' . $this->getJumpToParam($row) . '\',this,\'' . $this->domIdPrefix . $this->getId($row) . '\',' . $bank . ');';
@@ -560,7 +550,6 @@ abstract class AbstractTreeView {
         * @param array $row The row for the current element
         * @return string The processed icon input value.
         * @access private
-        * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
                return $icon;
@@ -572,7 +561,6 @@ abstract class AbstractTreeView {
         * @param string $icon Icon image tag
         * @param string $attr Attributes to add, eg. ' border="0"'
         * @return string Image tag, modified with $attr attributes added.
-        * @todo Define visibility
         */
        public function addTagAttributes($icon, $attr) {
                return preg_replace('/ ?\\/?>$/', '', $icon) . ' ' . $attr . ' />';
@@ -585,7 +573,6 @@ abstract class AbstractTreeView {
         * @param array $row record row with "php_tree_stop" field
         * @return string Modified string
         * @access private
-        * @todo Define visibility
         */
        public function wrapStop($str, $row) {
                if ($row['php_tree_stop']) {
@@ -608,7 +595,6 @@ abstract class AbstractTreeView {
         * @return boolean
         * @access private
         * @see \TYPO3\CMS\Backend\Tree\View\PageTreeView::expandNext()
-        * @todo Define visibility
         */
        public function expandNext($id) {
                return $this->stored[$this->bank][$id] || $this->expandAll ? 1 : 0;
@@ -619,7 +605,6 @@ abstract class AbstractTreeView {
         *
         * @return void
         * @access private
-        * @todo Define visibility
         */
        public function initializePositionSaving() {
                // Get stored tree structure:
@@ -649,7 +634,6 @@ abstract class AbstractTreeView {
         *
         * @return void
         * @access private
-        * @todo Define visibility
         */
        public function savePosition() {
                $this->BE_USER->uc['browseTrees'][$this->treeName] = serialize($this->stored);
@@ -666,7 +650,6 @@ abstract class AbstractTreeView {
         *
         * @param array $rec Record for root.
         * @return string Icon image tag.
-        * @todo Define visibility
         */
        public function getRootIcon($rec) {
                return $this->wrapIcon(IconUtility::getSpriteIcon('apps-pagetree-root'), $rec);
@@ -678,7 +661,6 @@ abstract class AbstractTreeView {
         *
         * @param array $row Item row.
         * @return string Image tag.
-        * @todo Define visibility
         */
        public function getIcon($row) {
                if ($this->iconPath && $this->iconName) {
@@ -699,7 +681,6 @@ abstract class AbstractTreeView {
         * @param array $row The input row array (where the key "title" is used for the title)
         * @param integer $titleLen Title length (30)
         * @return string The title.
-        * @todo Define visibility
         */
        public function getTitleStr($row, $titleLen = 30) {
                if ($this->ext_showNavTitle && strlen(trim($row['nav_title'])) > 0) {
@@ -720,7 +701,6 @@ abstract class AbstractTreeView {
         * @param array $row The input row array (where the key "title" is used for the title)
         * @return string The attribute value (is htmlspecialchared() already)
         * @see wrapIcon()
-        * @todo Define visibility
         */
        public function getTitleAttrib($row) {
                return htmlspecialchars($row['title']);
@@ -731,7 +711,6 @@ abstract class AbstractTreeView {
         *
         * @param array $row Record array
         * @return integer The "uid" field value.
-        * @todo Define visibility
         */
        public function getId($row) {
                return $row['uid'];
@@ -742,7 +721,6 @@ abstract class AbstractTreeView {
         *
         * @param array $row The record array.
         * @return string The jump-url parameter.
-        * @todo Define visibility
         */
        public function getJumpToParam($row) {
                return $this->getId($row);
@@ -762,7 +740,6 @@ abstract class AbstractTreeView {
         * @param string $blankLineCode ? (internal)
         * @param string $subCSSclass CSS class to use for <td> sub-elements
         * @return integer The count of items on the level
-        * @todo Define visibility
         */
        public function getTree($uid, $depth = 999, $depthData = '', $blankLineCode = '', $subCSSclass = '') {
                // Buffer for id hierarchy is reset:
@@ -849,7 +826,6 @@ abstract class AbstractTreeView {
         * @param integer $uid Id to count subitems for
         * @return integer
         * @access private
-        * @todo Define visibility
         */
        public function getCount($uid) {
                if (is_array($this->data)) {
@@ -865,7 +841,6 @@ abstract class AbstractTreeView {
         *
         * @param integer $uid uid, <= 0 (normally, this does not matter)
         * @return array Array with title/uid keys with values of $this->title/0 (zero)
-        * @todo Define visibility
         */
        public function getRootRecord($uid) {
                return array('title' => $this->title, 'uid' => 0);
@@ -878,7 +853,6 @@ abstract class AbstractTreeView {
         *
         * @param integer $uid UID to look up
         * @return array The record
-        * @todo Define visibility
         */
        public function getRecord($uid) {
                if (is_array($this->data)) {
@@ -897,7 +871,6 @@ abstract class AbstractTreeView {
         * @param string $subCSSclass Class for sub-elements.
         * @return mixed Data handle (Tables: An sql-resource, arrays: A parentId integer. -1 is returned if there were NO subLevel.)
         * @access private
-        * @todo Define visibility
         */
        public function getDataInit($parentId, $subCSSclass = '') {
                if (is_array($this->data)) {
@@ -920,7 +893,6 @@ abstract class AbstractTreeView {
         * @return integer number of items
         * @access private
         * @see getDataInit()
-        * @todo Define visibility
         */
        public function getDataCount(&$res) {
                if (is_array($this->data)) {
@@ -939,7 +911,6 @@ abstract class AbstractTreeView {
         * @return array item data array OR FALSE if end of elements.
         * @access private
         * @see getDataInit()
-        * @todo Define visibility
         */
        public function getDataNext(&$res, $subCSSclass = '') {
                if (is_array($this->data)) {
@@ -982,7 +953,6 @@ abstract class AbstractTreeView {
         * @param mixed $res Data handle
         * @return void
         * @access private
-        * @todo Define visibility
         */
        public function getDataFree(&$res) {
                if (!is_array($this->data)) {
@@ -1001,7 +971,6 @@ abstract class AbstractTreeView {
         * @param boolean $traverse Internal, for recursion.
         * @param integer $pid Internal, for recursion.
         * @return void
-        * @todo Define visibility
         */
        public function setDataFromArray(&$dataArr, $traverse = FALSE, $pid = 0) {
                if (!$traverse) {
@@ -1027,7 +996,6 @@ abstract class AbstractTreeView {
         * @param array $treeArr Content for $this->data
         * @param array $treeLookupArr Content for $this->dataLookup
         * @return void
-        * @todo Define visibility
         */
        public function setDataFromTreeArray(&$treeArr, &$treeLookupArr) {
                $this->data = &$treeArr;
index c62a8aa..1270afc 100644 (file)
@@ -31,7 +31,6 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @param string $clause Additional clause for selecting pages.
         * @param string $orderByFields record ORDER BY field
         * @return void
-        * @todo Define visibility
         */
        public function init($clause = '', $orderByFields = '') {
                // This will hide records from display - it has nothing todo with user rights!!
@@ -64,7 +63,6 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         *
         * @param array $row The table row.
         * @return string
-        * @todo Define visibility
         */
        public function getTitleAttrib($row) {
                return BackendUtility::titleAttribForPages($row, '1=1 ' . $this->clause, 0);
@@ -77,7 +75,6 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @param array $row The row for the current element
         * @return string The processed icon input value.
         * @access private
-        * @todo Define visibility
         */
        public function wrapIcon($icon, $row) {
                // Add title attribute to input icon tag
@@ -99,7 +96,6 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @param array $row The input row array (where the key "title" is used for the title)
         * @param integer $titleLen Title length (30)
         * @return string The title.
-        * @todo Define visibility
         */
        public function getTitleStr($row, $titleLen = 30) {
                $title = parent::getTitleStr($row, $titleLen);
@@ -119,7 +115,6 @@ class BrowseTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @param array $row Record row with "php_tree_stop" field
         * @return string Modified string
         * @access private
-        * @todo Define visibility
         */
        public function wrapStop($str, $row) {
                if ($row['php_tree_stop']) {
index a1c63f9..be81e0e 100644 (file)
@@ -202,7 +202,6 @@ class FolderTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         *
         * @param \TYPO3\CMS\Core\Resource\Folder $folderObject The folder to be used
         * @return      string The attribute value (is htmlspecialchared() already)
-        * @todo Define visibility
         */
        public function getTitleAttrib(\TYPO3\CMS\Core\Resource\Folder $folderObject) {
                return htmlspecialchars($folderObject->getName());
@@ -515,7 +514,6 @@ class FolderTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         *
         * @return      void
         * @access private
-        * @todo Define visibility
         */
        public function initializePositionSaving() {
                // Get stored tree structure:
index 5363570..9fec04c 100644 (file)
@@ -29,75 +29,75 @@ class PagePositionMap {
 
        // EXTERNAL, static:
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $moveOrCopy = 'move';
 
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $dontPrintPageInsertIcons = 0;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $backPath = '';
 
        // How deep the position page tree will go.
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $depth = 2;
 
        // Can be set to the sys_language uid to select content elements for.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $cur_sys_language;
 
        // INTERNAL, dynamic:
        // Request uri
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $R_URI = '';
 
        // Element id.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $elUid = '';
 
        // tt_content element uid to move.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $moveUid = '';
 
        // Caching arrays:
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $getModConfigCache = array();
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $checkNewPageCache = array();
 
        // Label keys:
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $l_insertNewPageHere = 'insertNewPageHere';
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $l_insertNewRecordHere = 'insertNewRecordHere';
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $modConfigStr = 'mod.web_list.newPageWiz';
 
@@ -132,7 +132,6 @@ class PagePositionMap {
         * @param string $perms_clause Page selection permission clause.
         * @param string $R_URI Current REQUEST_URI
         * @return string HTML code for the tree.
-        * @todo Define visibility
         */
        public function positionTree($id, $pageinfo, $perms_clause, $R_URI) {
                $code = '';
@@ -216,7 +215,6 @@ class PagePositionMap {
         *
         * @param string $prefix Insert record image prefix.
         * @return string <script> section
-        * @todo Define visibility
         */
        public function JSimgFunc($prefix = '') {
                $code = $GLOBALS['TBE_TEMPLATE']->wrapScriptTags('
@@ -247,7 +245,6 @@ class PagePositionMap {
         * @param array $dat Infomation 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) {
                if ($dat['row']['uid'] == $id) {
@@ -265,7 +262,6 @@ class 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) {
                $TSconfigProp = $this->getModConfig($newPagePID);
@@ -283,7 +279,6 @@ class PagePositionMap {
         * Get label, htmlspecialchars()'ed
         *
         * @return string The localized label for "insert new page here
-        * @todo Define visibility
         */
        public function insertlabel() {
                return $GLOBALS['LANG']->getLL($this->l_insertNewPageHere, 1);
@@ -295,7 +290,6 @@ class PagePositionMap {
         * @param string $str Page title.
         * @param array $rec Page record (?)
         * @return string Wrapped title.
-        * @todo Define visibility
         */
        public function linkPageTitle($str, $rec) {
                return $str;
@@ -307,7 +301,6 @@ class PagePositionMap {
         *
         * @param integer $pid Page id for which to test.
         * @return boolean
-        * @todo Define visibility
         */
        public function checkNewPageInPid($pid) {
                if (!isset($this->checkNewPageCache[$pid])) {
@@ -323,7 +316,6 @@ class PagePositionMap {
         * @param integer $pid Page id for which to get the module configuration.
         * @return array The properties of teh module configuration for the page id.
         * @see onClickEvent()
-        * @todo Define visibility
         */
        public function getModConfig($pid) {
                if (!isset($this->getModConfigCache[$pid])) {
@@ -339,7 +331,6 @@ class PagePositionMap {
         * @param string $codes Keywords for which lines to insert.
         * @param boolean $allBlank If TRUE all lines are just blank clear.gifs
         * @return string HTML content.
-        * @todo Define visibility
         */
        public function insertQuadLines($codes, $allBlank = FALSE) {
                $codeA = GeneralUtility::trimExplode(',', $codes . ',line', TRUE);
@@ -368,7 +359,6 @@ class PagePositionMap {
         * @param boolean $showHidden If not set, then hidden/starttime/endtime records are filtered out.
         * @param string $R_URI Request URI
         * @return string HTML
-        * @todo Define visibility
         */
        public function printContentElementColumns($pid, $moveUid, $colPosList, $showHidden, $R_URI) {
                $this->R_URI = $R_URI;
@@ -398,7 +388,6 @@ class PagePositionMap {
         * @param array $colPosArray Column position array
         * @param integer $pid The id of the page
         * @return string HTML
-        * @todo Define visibility
         */
        public function printRecordMap($lines, $colPosArray, $pid = 0) {
                $row1 = '';
@@ -482,7 +471,6 @@ class PagePositionMap {
         * @param string $vv Column info.
         * @return string
         * @see printRecordMap()
-        * @todo Define visibility
         */
        public function wrapColumnHeader($str, $vv) {
                return $str;
@@ -498,7 +486,6 @@ class PagePositionMap {
         * @param integer $moveUid Move uid
         * @param integer $pid PID value.
         * @return string
-        * @todo Define visibility
         */
        public function insertPositionIcon($row, $vv, $kk, $moveUid, $pid) {
                if (is_array($row) && !empty($row['uid'])) {
@@ -522,7 +509,6 @@ class PagePositionMap {
         * @param integer $pid PID value.
         * @param integer $sys_lang System language (not used currently)
         * @return string
-        * @todo Define visibility
         */
        public function onClickInsertRecord($row, $vv, $moveUid, $pid, $sys_lang = 0) {
                $table = 'tt_content';
@@ -542,7 +528,6 @@ class PagePositionMap {
         * @param string $str HTML content
         * @param array $row Record array.
         * @return string HTML content
-        * @todo Define visibility
         */
        public function wrapRecordHeader($str, $row) {
                return $str;
@@ -553,7 +538,6 @@ class PagePositionMap {
         *
         * @param array $row Record row.
         * @return string HTML
-        * @todo Define visibility
         */
        public function getRecordHeader($row) {
                $line = IconUtility::getSpriteIconForRecord('tt_content', $row, array('title' => htmlspecialchars(BackendUtility::getRecordIconAltText($row, 'tt_content'))));
@@ -567,7 +551,6 @@ class 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) {
                return '<a href="' . htmlspecialchars(GeneralUtility::linkThisScript(array('uid' => (int)$row['uid'], 'moveUid' => ''))) . '">' . $str . '</a>';
index a573b77..eea9c90 100644 (file)
@@ -22,7 +22,7 @@ namespace TYPO3\CMS\Backend\Tree\View;
 class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $fieldArray = array(
                'uid',
@@ -35,12 +35,12 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
        );
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $defaultList = 'uid,pid,tstamp,sorting,deleted,perms_userid,perms_groupid,perms_user,perms_group,perms_everybody,crdate,cruser_id';
 
        /**
-        * @todo Define visibility
+        * @var int
         */
        public $setRecs = 0;
 
@@ -51,7 +51,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @param string $clause Part of where query which will filter out non-readable pages.
         * @param string $orderByFields Record ORDER BY field
         * @return void
-        * @todo Define visibility
         */
        public function init($clause = '', $orderByFields = '') {
                parent::init(' AND deleted=0 ' . $clause, 'sorting');
@@ -73,7 +72,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         *
         * @param integer $id ID (uid) to test for (see extending classes where this is checked against session data)
         * @return boolean
-        * @todo Define visibility
         */
        public function expandNext($id) {
                return 1;
@@ -91,7 +89,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         * @return string Image tag with the plus/minus icon.
         * @access private
         * @see AbstarctTreeView::PMicon()
-        * @todo Define visibility
         */
        public function PMicon($row, $a, $c, $nextCount, $exp) {
                $PM = 'join';
@@ -106,7 +103,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\AbstractTreeView {
         *
         * @return void
         * @access private
-        * @todo Define visibility
         */
        public function initializePositionSaving() {
                $this->stored = array();
index 237f903..1ed619b 100644 (file)
@@ -26,22 +26,22 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
 
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $ext_showPageId;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $ext_IconMode;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $ext_separateNotinmenuPages;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $ext_alphasortNotinmenuPages;
 
@@ -105,7 +105,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @param array $row Record row with "php_tree_stop" field
         * @return string Modified string
         * @access private
-        * @todo Define visibility
         */
        public function wrapStop($str, $row) {
                if ($row['php_tree_stop']) {
@@ -122,7 +121,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @param integer $bank Bank pointer (which mount point number)
         * @return string
         * @access private
-        * @todo Define visibility
         */
        public function wrapTitle($title, $row, $bank = 0) {
                // Hook for overriding the page title
@@ -145,7 +143,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         *
         * @param array $treeArr "tree-array" - if blank string, the internal ->tree array is used.
         * @return string The HTML code for the tree
-        * @todo Define visibility
         */
        public function printTree($treeArr = '') {
                $titleLen = (int)$this->BE_USER->uc['titleLen'];
@@ -253,7 +250,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @return string Image tag with the plus/minus icon.
         * @access private
         * @see \TYPO3\CMS\Backend\Tree\View\PageTreeView::PMicon()
-        * @todo Define visibility
         */
        public function PMicon($row, $a, $c, $nextCount, $exp) {
                $PM = $nextCount ? ($exp ? 'minus' : 'plus') : 'join';
@@ -273,7 +269,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @param string $cmd Command for 'PM' get var
         * @return boolean $isExpand Link-wrapped input string
         * @access private
-        * @todo Define visibility
         */
        public function PMiconATagWrap($icon, $cmd, $isExpand = TRUE) {
                if ($this->thisScript) {
@@ -290,7 +285,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * Is based on the mounts found in the internal array ->MOUNTS (set in the constructor)
         *
         * @return string HTML code for the browsable tree
-        * @todo Define visibility
         */
        public function getBrowsableTree() {
                // Get stored tree structure AND updating it if needed according to incoming PM GET var.
@@ -351,7 +345,6 @@ class PageTreeView extends \TYPO3\CMS\Backend\Tree\View\BrowseTreeView {
         * @param string $blankLineCode ? (internal)
         * @param string $subCSSclass
         * @return integer The count of items on the level
-        * @todo Define visibility
         */
        public function getTree($uid, $depth = 999, $blankLineCode = '', $subCSSclass = '') {
                // Buffer for id hierarchy is reset: