[TASK] Remove the "@todo define visibility" in various extensions 74/33274/3
authorWouter Wolters <typo3@wouterwolters.nl>
Sun, 12 Oct 2014 14:02:35 +0000 (16:02 +0200)
committerMarkus Klein <klein.t3@reelworx.at>
Mon, 13 Oct 2014 14:19:46 +0000 (16:19 +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: #62177
Related: #61957
Releases: master
Change-Id: I5f9b30c73f8e78549912f1d41d482d779f803143
Reviewed-on: http://review.typo3.org/33274
Reviewed-by: Stefan Froemken <froemken@gmail.com>
Tested-by: Stefan Froemken <froemken@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
15 files changed:
typo3/sysext/cms/layout/db_layout.php
typo3/sysext/cms/layout/db_new_content_el.php
typo3/sysext/cshmanual/Classes/Controller/HelpModuleController.php
typo3/sysext/cshmanual/mod/index.php
typo3/sysext/css_styled_content/Classes/Controller/CssStyledContentController.php
typo3/sysext/dbal/Classes/RecordList/DatabaseRecordList.php
typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
typo3/sysext/saltedpasswords/Classes/Evaluation/Evaluator.php
typo3/sysext/saltedpasswords/Classes/SaltedPasswordService.php
typo3/sysext/sv/Classes/AbstractAuthenticationService.php
typo3/sysext/sv/Classes/AuthenticationService.php
typo3/sysext/sys_action/Classes/ActionTask.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateConstantEditorModuleFunctionController.php
typo3/sysext/version/Classes/ClickMenu/VersionClickMenu.php
typo3/sysext/version/Classes/Utility/WorkspacesUtility.php

index a6fc108..25478a1 100644 (file)
@@ -37,12 +37,12 @@ $BE_USER->modAccess($MCONF, 1);
 class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
 
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $dontPrintPageInsertIcons = 1;
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $l_insertNewRecordHere = 'newContentElement';
 
@@ -52,7 +52,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param string $str The title value.
         * @param array $row The record row.
         * @return string Wrapped title string.
-        * @todo Define visibility
         */
        public function wrapRecordTitle($str, $row) {
                $aOnClick = 'jumpToUrl(' . \TYPO3\CMS\Core\Utility\GeneralUtility::quoteJSvalue($GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('tt_content:' . $row['uid'])))) . ');return false;';
@@ -66,7 +65,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param string $vv Column info.
         * @return string
         * @see printRecordMap()
-        * @todo Define visibility
         */
        public function wrapColumnHeader($str, $vv) {
                $aOnClick = 'jumpToUrl(' . \TYPO3\CMS\Core\Utility\GeneralUtility::quoteJSvalue($GLOBALS['SOBE']->local_linkThisScript(array('edit_record' => ('_EDIT_COL:' . $vv)))) . ');return false;';
@@ -82,7 +80,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param integer $pid PID value.
         * @param integer $sys_lang System language
         * @return string
-        * @todo Define visibility
         */
        public function onClickInsertRecord($row, $vv, $moveUid, $pid, $sys_lang = 0) {
                if (is_array($row)) {
@@ -99,7 +96,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param string $str HTML content
         * @param array $row Record array.
         * @return string HTML content
-        * @todo Define visibility
         */
        public function wrapRecordHeader($str, $row) {
                if ($row['uid'] == $this->moveUid) {
index bfd80b5..fa83c95 100644 (file)
@@ -39,7 +39,7 @@ $LOCAL_LANG = $LOCAL_LANG_orig;
 class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
 
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $dontPrintPageInsertIcons = 1;
 
@@ -49,7 +49,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param string $str The title value.
         * @param array $row The record row.
         * @return string Wrapped title string.
-        * @todo Define visibility
         */
        public function wrapRecordTitle($str, $row) {
                return $str;
@@ -64,7 +63,6 @@ class ext_posMap extends \TYPO3\CMS\Backend\Tree\View\PagePositionMap {
         * @param integer $pid PID value.
         * @param integer $sys_lang System language
         * @return string
-        * @todo Define visibility
         */
        public function onClickInsertRecord($row, $vv, $moveUid, $pid, $sys_lang = 0) {
                $table = 'tt_content';
index 7aa82fe..5b94159 100644 (file)
@@ -26,7 +26,7 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 class HelpModuleController {
 
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $allowedHTML = '<strong><em><b><i>';
 
@@ -34,21 +34,21 @@ class HelpModuleController {
         * For these vars, see init()
         * If set access to fields and tables is checked. Should be done for TRUE database tables.
         *
-        * @todo Define visibility
+        * @var bool
         */
        public $limitAccess;
 
        /**
         * The "table" key
         *
-        * @todo Define visibility
+        * @var string
         */
        public $table;
 
        /**
         * The "field" key
         *
-        * @todo Define visibility
+        * @var string
         */
        public $field;
 
@@ -64,35 +64,35 @@ class HelpModuleController {
         * Internal, static: GPvar
         * Table/Field id
         *
-        * @todo Define visibility
+        * @var string
         */
        public $tfID;
 
        /**
         * Back (previous tfID)
         *
-        * @todo Define visibility
+        * @var string
         */
        public $back;
 
        /**
         * If set, then in TOC mode the FULL manual will be printed as well!
         *
-        * @todo Define visibility
+        * @var bool
         */
        public $renderALL;
 
        /**
         * Content accumulation
         *
-        * @todo Define visibility
+        * @var string
         */
        public $content;
 
        /**
         * Glossary words
         *
-        * @todo Define visibility
+        * @var array
         */
        public $glossaryWords;
 
@@ -107,7 +107,6 @@ class HelpModuleController {
         * Initialize the class for various input etc.
         *
         * @return void
-        * @todo Define visibility
         */
        public function init() {
                $this->moduleUrl = BackendUtility::getModuleUrl('help_cshmanual');
@@ -156,7 +155,6 @@ class HelpModuleController {
         * Main function, rendering the display
         *
         * @return void
-        * @todo Define visibility
         */
        public function main() {
                if ($this->field == '*') {
@@ -186,7 +184,6 @@ class HelpModuleController {
         * Outputting the accumulated content to screen
         *
         * @return void
-        * @todo Define visibility
         */
        public function printContent() {
                echo $this->content;
@@ -200,7 +197,6 @@ class HelpModuleController {
         * Creates Table Of Contents and possibly "Full Manual" mode if selected.
         *
         * @return string HTML content
-        * @todo Define visibility
         */
        public function render_TOC() {
                // Initialize:
@@ -293,7 +289,6 @@ class HelpModuleController {
         * @param array $tocArray TOC array; Here TOC index elements are created. Passed by reference!
         * @param array $CSHkeys CSH keys array. Every item rendered will be unset in this array so finally we can see what CSH keys are not processed yet. Passed by reference!
         * @return void
-        * @todo Define visibility
         */
        public function render_TOC_el($table, $tocCat, &$outputSections, &$tocArray, &$CSHkeys) {
                // Render full manual right here!
@@ -324,7 +319,6 @@ class HelpModuleController {
         *
         * @param array $tocArray ToC Array.
         * @return string HTML bullet list for index.
-        * @todo Define visibility
         */
        public function render_TOC_makeTocList($tocArray) {
                // The Various manual sections:
@@ -360,7 +354,6 @@ class HelpModuleController {
         * @param string $key Full CSH key (may be different from table name)
         * @param string $table CSH key / table name
         * @return string HTML output
-        * @todo Define visibility
         */
        public function render_Table($key, $table = NULL) {
                $output = '';
@@ -411,7 +404,6 @@ class HelpModuleController {
         * @param string $key CSH key / table name
         * @param string $field Sub key / field name
         * @return string HTML output
-        * @todo Define visibility
         */
        public function render_Single($key, $field) {
                $output = '';
@@ -439,7 +431,6 @@ class HelpModuleController {
         * @param string $value See-also input codes
         * @param string $anchorTable If $anchorTable is set to a tablename, then references to this table will be made as anchors, not URLs.
         * @return string See-also links HTML
-        * @todo Define visibility
         */
        public function make_seeAlso($value, $anchorTable = '') {
                // Split references by comma or linebreak
@@ -482,7 +473,6 @@ class HelpModuleController {
         * @param string $images Image file reference (list of)
         * @param string $descr Description string (divided for each image by line break)
         * @return string Image HTML codes
-        * @todo Define visibility
         */
        public function printImage($images, $descr) {
                $code = '';
@@ -519,7 +509,6 @@ class HelpModuleController {
         * @param string $str Header text
         * @param integer $type Header type (1, 0)
         * @return string The HTML for the header.
-        * @todo Define visibility
         */
        public function headerLine($str, $type = 0) {
                switch ($type) {
@@ -540,7 +529,6 @@ class HelpModuleController {
         *
         * @param string $str Content to format.
         * @return string Formatted content.
-        * @todo Define visibility
         */
        public function prepareContent($str) {
                return '<p>' . nl2br(trim(strip_tags($str, $this->allowedHTML))) . '</p>
@@ -555,7 +543,6 @@ class HelpModuleController {
         * @param string $field Sub key / field name
         * @param boolean $anchors If anchors is to be shown.
         * @return string HTML content
-        * @todo Define visibility
         */
        public function printItem($key, $field, $anchors = FALSE) {
                $out = '';
@@ -574,7 +561,6 @@ class HelpModuleController {
         * @param string $key CSH key / table name
         * @param string $field Sub key / field name
         * @return array Table and field labels in a numeric array
-        * @todo Define visibility
         */
        public function getTableFieldNames($key, $field) {
                $GLOBALS['LANG']->loadSingleTableDescription($key);
@@ -610,7 +596,6 @@ class HelpModuleController {
         * @param string $mergeToken Token to merge the two strings with
         * @return string Labels joined with merge token
         * @see getTableFieldNames()
-        * @todo Define visibility
         */
        public function getTableFieldLabel($key, $field = '', $mergeToken = ': ') {
                // Get table / field parts:
@@ -629,7 +614,6 @@ class HelpModuleController {
         * Glossary is cached in cache_hash cache and so will be updated only when cache is cleared.
         *
         * @return void
-        * @todo Define visibility
         */
        public function createGlossaryIndex() {
                // Create hash string and try to retrieve glossary array:
@@ -680,7 +664,6 @@ class HelpModuleController {
         *
         * @param string $code Input HTML code
         * @return string Output HTML code
-        * @todo Define visibility
         */
        public function substituteGlossaryWords($code) {
                $htmlParser = GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Html\\HtmlParser');
@@ -696,7 +679,6 @@ class HelpModuleController {
         *
         * @param string $code Input HTML string
         * @return string HTML with substituted words in.
-        * @todo Define visibility
         */
        public function substituteGlossaryWords_htmlcleaner_callback($code) {
                if (is_array($this->substWords) && count($this->substWords) && strlen(trim($code))) {
index 282e4a1..f3f0347 100644 (file)
@@ -26,7 +26,6 @@ class local_t3lib_parsehtml extends \TYPO3\CMS\Core\Html\HtmlParser {
         * @param mixed $conf Not used, ignore.
         * @return string The processed value.
         * @access private
-        * @todo Define visibility
         */
        public function processContent($value, $dir, $conf) {
                $value = $this->pObj->substituteGlossaryWords_htmlcleaner_callback($value);
index 0225176..be01fc6 100644 (file)
@@ -28,26 +28,26 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
        /**
         * Same as class name
         *
-        * @todo Define visibility
+        * @var string
         */
        public $prefixId = 'tx_cssstyledcontent_pi1';
 
        /**
         * Path to this script relative to the extension dir.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $scriptRelPath = 'pi1/class.tx_cssstyledcontent_pi1.php';
 
        /**
         * The extension key
         *
-        * @todo Define visibility
+        * @var string
         */
        public $extKey = 'css_styled_content';
 
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $conf = array();
 
@@ -1178,7 +1178,6 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
         * @param array $conf TypoScript configuration array
         * @param integer $type The "layout" type
         * @return array Array with attributes inside.
-        * @todo Define visibility
         */
        public function getTableAttributes($conf, $type) {
                // Initializing:
@@ -1259,7 +1258,6 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
         *
         * @param string $functionName Name of the function you want to call / hook key
         * @return object Hook object, if any. Otherwise NULL.
-        * @todo Define visibility
         */
        public function hookRequest($functionName) {
                global $TYPO3_CONF_VARS;
index d5ce34a..a5465dd 100644 (file)
@@ -13,12 +13,7 @@ namespace TYPO3\CMS\Dbal\RecordList;
  *
  * The TYPO3 project - inspiring people to share!
  */
-/**
- * Include file extending localRecordList for DBAL compatibility
- *
- * @author     Kasper Skårhøj <kasperYYYY@typo3.com>
- * @author     Karsten Dambekalns <k.dambekalns@fishfarm.de>
- */
+
 /**
  * Child class for rendering of Web > List (not the final class)
  *
@@ -35,7 +30,6 @@ class DatabaseRecordList extends \TYPO3\CMS\Recordlist\RecordList\DatabaseRecord
         *
         * @param       string          Table, in which the fields are being searched.
         * @return      string          Returns part of WHERE-clause for searching, if applicable.
-        * @todo Define visibility
         */
        public function makeSearchString($table) {
                // Make query, only if table is valid and a search string is actually defined:
index 5c67976..734ad41 100644 (file)
@@ -26,14 +26,14 @@ class FrontendLoginController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
        /**
         * Same as class name
         *
-        * @todo Define visibility
+        * @var string
         */
        public $prefixId = 'tx_felogin_pi1';
 
        /**
         * Path to this script relative to the extension dir.
         *
-        * @todo Define visibility
+        * @var string
         * @TODO This is still set to the "old" class location since the locallang.xlf file in the same dir is loaded by pi_loadLL
         */
        public $scriptRelPath = 'pi1/class.tx_felogin_pi1.php';
@@ -41,7 +41,7 @@ class FrontendLoginController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
        /**
         * The extension key.
         *
-        * @todo Define visibility
+        * @var string
         */
        public $extKey = 'felogin';
 
index d083f42..f380a66 100644 (file)
@@ -36,7 +36,6 @@ class Evaluator {
         * hashing will be done on server-side.
         *
         * @return string JavaScript code for evaluating the
-        * @todo Define visibility
         */
        public function returnFieldJS() {
                return 'return value;';
@@ -49,7 +48,6 @@ class Evaluator {
         * @param string $is_in Is-In String
         * @param integer $set Determines if the field can be set (value correct) or not, e.g. if input is required but the value is empty, then $set should be set to FALSE. (PASSED BY REFERENCE!)
         * @return The new value of the field
-        * @todo Define visibility
         */
        public function evaluateFieldValue($value, $is_in, &$set) {
                $isEnabled = $this->mode ? \TYPO3\CMS\Saltedpasswords\Utility\SaltedPasswordsUtility::isUsageEnabled($this->mode) : \TYPO3\CMS\Saltedpasswords\Utility\SaltedPasswordsUtility::isUsageEnabled();
index 87fa8e3..9c90f3f 100644 (file)
@@ -100,7 +100,6 @@ class SaltedPasswordService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
         * @param array $loginData Login data array
         * @param string $passwordCompareStrategy Password compare strategy
         * @return boolean TRUE if login data matched
-        * @todo Define visibility
         */
        public function compareUident(array $user, array $loginData, $passwordCompareStrategy = '') {
                $validPasswd = FALSE;
@@ -239,7 +238,6 @@ class SaltedPasswordService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
         * @param string $message Message to output
         * @return void
         * @see \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog()
-        * @todo Define visibility
         */
        public function writeLogMessage($message) {
                if (func_num_args() > 1) {
index c5b7705..2c23515 100644 (file)
@@ -25,49 +25,48 @@ class AbstractAuthenticationService extends \TYPO3\CMS\Core\Service\AbstractServ
         * User object
         *
         * @var \TYPO3\CMS\Core\Authentication\AbstractUserAuthentication
-        * @todo Define visibility
         */
        public $pObj;
 
        // Subtype of the service which is used to call the service.
        /**
-        * @todo Define visibility
+        * @var string
         */
        public $mode;
 
        // Submitted login form data
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $login = array();
 
        // Various data
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $authInfo = array();
 
        // User db table definition
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $db_user = array();
 
        // Usergroups db table definition
        /**
-        * @todo Define visibility
+        * @var array
         */
        public $db_groups = array();
 
        // If the writelog() functions is called if a login-attempt has be tried without success
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $writeAttemptLog = FALSE;
 
        // If the \TYPO3\CMS\Core\Utility\GeneralUtility::devLog() function should be used
        /**
-        * @todo Define visibility
+        * @var bool
         */
        public $writeDevLog = FALSE;
 
@@ -79,7 +78,6 @@ class AbstractAuthenticationService extends \TYPO3\CMS\Core\Service\AbstractServ
         * @param array $authInfo Information array. Holds submitted form data etc.
         * @param object $pObj Parent object
         * @return void
-        * @todo Define visibility
         */
        public function initAuth($mode, $loginData, $authInfo, $pObj) {
                $this->pObj = $pObj;
@@ -100,7 +98,6 @@ class AbstractAuthenticationService extends \TYPO3\CMS\Core\Service\AbstractServ
         * @param array $loginData Login data array
         * @param string $passwordCompareStrategy Password compare strategy
         * @return boolean TRUE if login data matched
-        * @todo Define visibility
         */
        public function compareUident(array $user, array $loginData, $passwordCompareStrategy = '') {
                if ($this->authInfo['loginType'] === 'BE') {
@@ -129,7 +126,6 @@ class AbstractAuthenticationService extends \TYPO3\CMS\Core\Service\AbstractServ
         * @param integer $recuid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
         * @param integer $recpid Special field used by tce_main.php. These ($tablename, $recuid, $recpid) holds the reference to the record which the log-entry is about. (Was used in attic status.php to update the interface.)
         * @return void
-        * @todo Define visibility
         */
        public function writelog($type, $action, $error, $details_nr, $details, $data, $tablename = '', $recuid = '', $recpid = '') {
                if ($this->writeAttemptLog) {
@@ -149,7 +145,6 @@ class AbstractAuthenticationService extends \TYPO3\CMS\Core\Service\AbstractServ
         * @param string $extraWhere Additional WHERE clause: " AND ...
         * @param array $dbUserSetup User db table definition: $this->db_user
         * @return mixed User array or FALSE
-        * @todo Define visibility
         */
        public function fetchUserRecord($username, $extraWhere = '', $dbUserSetup = '') {
                $dbUser = is_array($dbUserSetup) ? $dbUserSetup : $this->db_user;
index e736c41..2c1d4dc 100644 (file)
@@ -61,7 +61,6 @@ class AuthenticationService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
         * Find a user (eg. look up the user record in database when a login is sent)
         *
         * @return mixed User array or FALSE
-        * @todo Define visibility
         */
        public function getUser() {
                $user = FALSE;
@@ -135,7 +134,6 @@ class AuthenticationService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
         * @param array $user Data of user.
         * @param array $knownGroups Group data array of already known groups. This is handy if you want select other related groups. Keys in this array are unique IDs of those groups.
         * @return mixed Groups array, keys = uid which must be unique
-        * @todo Define visibility
         */
        public function getGroups($user, $knownGroups) {
                global $TYPO3_CONF_VARS;
@@ -193,7 +191,6 @@ class AuthenticationService extends \TYPO3\CMS\Sv\AbstractAuthenticationService
         * @param array $groups
         * @return array
         * @access private
-        * @todo Define visibility
         */
        public function getSubGroups($grList, $idList = '', &$groups) {
                // Fetching records of the groups in $grList (which are not blocked by lockedToDomain either):
index 7139a22..8e90b5c 100644 (file)
@@ -32,7 +32,6 @@ class ActionTask implements \TYPO3\CMS\Taskcenter\TaskInterface {
 
        /**
         * @var \TYPO3\CMS\Backend\Form\FormEngine
-        * @todo Define visibility
         */
        public $t3lib_TCEforms;
 
index 47dd5ee..390adbf 100644 (file)
@@ -32,7 +32,6 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends \TYPO3\CM
         * @param integer $pageId
         * @param integer $template_uid
         * @return integer
-        * @todo Define visibility
         */
        public function initialize_editor($pageId, $template_uid = 0) {
                // Initializes the module. Done in this function because we may need to re-initialize if data is submitted!
@@ -68,7 +67,6 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends \TYPO3\CM
         *
         * @param string $theOutput
         * @return string
-        * @todo Define visibility
         */
        public function displayExample($theOutput) {
                global $tmpl;
@@ -83,7 +81,6 @@ class TypoScriptTemplateConstantEditorModuleFunctionController extends \TYPO3\CM
         * Main
         *
         * @return string
-        * @todo Define visibility
         */
        public function main() {
                global $TYPO3_CONF_VARS;
index 7d416b4..2c97773 100644 (file)
@@ -67,7 +67,6 @@ class VersionClickMenu {
         * Includes the [extDir]/locallang.php and returns the $LOCAL_LANG array found in that file.
         *
         * @return array Local lang array
-        * @todo Define visibility
         */
        public function includeLL() {
                return $GLOBALS['LANG']->includeLLFile('EXT:version/locallang.xlf', FALSE);
index 1b01916..b90674c 100644 (file)
@@ -30,7 +30,6 @@ class WorkspacesUtility {
         * @param boolean $doSwap If set, then the currently online versions are swapped into the workspace in exchange for the offline versions. Otherwise the workspace is emptied.
         * @param int $pageId
         * @return array Command array for tcemain
-        * @todo Define visibility
         */
        public function getCmdArrayForPublishWS($wsid, $doSwap, $pageId = 0) {
                $wsid = (int)$wsid;
@@ -71,7 +70,6 @@ class WorkspacesUtility {
         * @param integer $stage Stage filter: -99 means no filtering, otherwise it will be used to select only elements with that stage. For publishing, that would be "10
         * @param integer $pageId Page id: Live page for which to find versions in workspace!
         * @return array Array of all records uids etc. First key is table name, second key incremental integer. Records are associative arrays with uid and t3ver_oid fields. The REAL pid of the online record is found as "realpid
-        * @todo Define visibility
         */
        public function selectVersionsInWorkspace($wsid, $filter = 0, $stage = -99, $pageId = -1) {
                $wsid = (int)$wsid;
@@ -104,7 +102,6 @@ class WorkspacesUtility {
         * and publishes them
         *
         * @return void
-        * @todo Define visibility
         */
        public function autoPublishWorkspaces() {
                // Temporarily set admin rights