[TASK] Clean up phpDoc comments in typo3/sysext/

Clean up phpDoc comments for the following classes:

* typo3/sysext/cshmanual/ext_tables.php
* typo3/sysext/extra_page_cm_options/class.tx_extrapagecmoptions.php
* typo3/sysext/extra_page_cm_options/ext_tables.php
* typo3/sysext/feedit/ext_localconf.php
* typo3/sysext/feedit/view/class.tx_feedit_editpanel.php
* typo3/sysext/felogin/ext_localconf.php
* typo3/sysext/felogin/ext_tables.php
* typo3/sysext/felogin/pi1/class.tx_felogin_pi1.php
* typo3/sysext/felogin/tests/tx_feloginTest.php

Clean up provides:

* Provide correct parameter names in phpDoc
* Provide correct data types in phpDoc
* CGLify phpDoc (remove tabs)

Change-Id: Id09ec3f662b432098eb3b3897008cbeee76e0871
Resolves: #38242
Releases: 6.0
Reviewed-on: http://review.typo3.org/12245
Reviewed-by: Oliver Klee
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
This commit is contained in:
Wouter Wolters 2012-06-21 01:02:36 +02:00 committed by Christian Kuhn
parent adef1719c9
commit 41ee705a92
2 changed files with 64 additions and 55 deletions

View file

@ -1,5 +1,7 @@
<?php <?php
if (!defined ('TYPO3_MODE')) die ('Access denied.'); if (!defined('TYPO3_MODE')) {
die('Access denied.');
}
// Register the edit panel view. // Register the edit panel view.
$TYPO3_CONF_VARS['SC_OPTIONS']['typo3/classes/class.frontendedit.php']['edit'] = 'EXT:feedit/view/class.tx_feedit_editpanel.php:tx_feedit_editpanel'; $TYPO3_CONF_VARS['SC_OPTIONS']['typo3/classes/class.frontendedit.php']['edit'] = 'EXT:feedit/view/class.tx_feedit_editpanel.php:tx_feedit_editpanel';

View file

@ -57,13 +57,17 @@ class tx_feedit_editpanel {
* With the "edit panel" the user will see buttons with links to editing, moving, hiding, deleting the element * With the "edit panel" the user will see buttons with links to editing, moving, hiding, deleting the element
* This function is used for the cObject EDITPANEL and the stdWrap property ".editPanel" * This function is used for the cObject EDITPANEL and the stdWrap property ".editPanel"
* *
* @param string A content string containing the content related to the edit panel. For cObject "EDITPANEL" this is empty but not so for the stdWrap property. The edit panel is appended to this string and returned. * @param string $content A content string containing the content related to the edit panel. For cObject "EDITPANEL" this is empty but not so for the stdWrap property. The edit panel is appended to this string and returned.
* @param array TypoScript configuration properties for the editPanel * @param array $conf TypoScript configuration properties for the editPanel
* @param string The "table:uid" of the record being shown. If empty string then $this->currentRecord is used. For new records (set by $conf['newRecordFromTable']) it's auto-generated to "[tablename]:NEW" * @param string $currentRecord The "table:uid" of the record being shown. If empty string then $this->currentRecord is used. For new records (set by $conf['newRecordFromTable']) it's auto-generated to "[tablename]:NEW"
* @param array Alternative data array to use. Default is $this->data * @param array $dataArr Alternative data array to use. Default is $this->data
* @param string $table
* @param string $allow
* @param integer $newUID
* @param array $hiddenFields
* @return string The input content string with the editPanel appended. This function returns only an edit panel appended to the content string if a backend user is logged in (and has the correct permissions). Otherwise the content string is directly returned. * @return string The input content string with the editPanel appended. This function returns only an edit panel appended to the content string if a backend user is logged in (and has the correct permissions). Otherwise the content string is directly returned.
*/ */
public function editPanel($content, array $conf, $currentRecord='', array $dataArr=array(), $table='', $allow='', $newUID=0, array $hiddenFields=array()) { public function editPanel($content, array $conf, $currentRecord = '', array $dataArr = array(), $table = '', $allow = '', $newUID = 0, array $hiddenFields = array()) {
// Special content is about to be shown, so the cache must be disabled. // Special content is about to be shown, so the cache must be disabled.
$GLOBALS['TSFE']->set_no_cache(); $GLOBALS['TSFE']->set_no_cache();
$formName = 'TSFE_EDIT_FORM_' . substr($GLOBALS['TSFE']->uniqueHash(), 0, 4); $formName = 'TSFE_EDIT_FORM_' . substr($GLOBALS['TSFE']->uniqueHash(), 0, 4);
@ -79,7 +83,7 @@ class tx_feedit_editpanel {
$theCmd =$TSFE_EDIT['cmd']; $theCmd =$TSFE_EDIT['cmd'];
} }
switch($theCmd) { switch ($theCmd) {
case 'edit': case 'edit':
case 'new': case 'new':
$finalOut = $this->editContent($formTag, $formName, $theCmd, $newUID, $dataArr, $table, $currentRecord, $blackLine); $finalOut = $this->editContent($formTag, $formName, $theCmd, $newUID, $dataArr, $table, $currentRecord, $blackLine);
@ -119,7 +123,7 @@ class tx_feedit_editpanel {
// Final // Final
$labelTxt = $this->cObj->stdWrap($conf['label'], $conf['label.']); $labelTxt = $this->cObj->stdWrap($conf['label'], $conf['label.']);
foreach((array) $hiddenFields as $name => $value) { foreach ((array) $hiddenFields as $name => $value) {
$hiddenFieldString .= '<input type="hidden" name="TSFE_EDIT[' . $name . ']" value="' . $value . '"/>' . LF; $hiddenFieldString .= '<input type="hidden" name="TSFE_EDIT[' . $name . ']" value="' . $value . '"/>' . LF;
} }
@ -137,7 +141,7 @@ class tx_feedit_editpanel {
</tr> </tr>
</table> </table>
</form>'; </form>';
// wrap the panel // Wrap the panel
if ($conf['innerWrap']) { if ($conf['innerWrap']) {
$panel = $this->cObj->wrap($panel, $conf['innerWrap']); $panel = $this->cObj->wrap($panel, $conf['innerWrap']);
} }
@ -145,10 +149,10 @@ class tx_feedit_editpanel {
$panel = $this->cObj->stdWrap($panel, $conf['innerWrap.']); $panel = $this->cObj->stdWrap($panel, $conf['innerWrap.']);
} }
// add black line: // Add black line:
$panel .= $blackLine; $panel .= $blackLine;
// wrap the complete panel // Wrap the complete panel
if ($conf['outerWrap']) { if ($conf['outerWrap']) {
$panel = $this->cObj->wrap($panel, $conf['outerWrap']); $panel = $this->cObj->wrap($panel, $conf['outerWrap']);
} }
@ -177,15 +181,18 @@ class tx_feedit_editpanel {
* Adds an edit icon to the content string. The edit icon links to alt_doc.php with proper parameters for editing the table/fields of the context. * Adds an edit icon to the content string. The edit icon links to alt_doc.php with proper parameters for editing the table/fields of the context.
* This implements TYPO3 context sensitive editing facilities. Only backend users will have access (if properly configured as well). * This implements TYPO3 context sensitive editing facilities. Only backend users will have access (if properly configured as well).
* *
* @param string The content to which the edit icons should be appended * @param string $content The content to which the edit icons should be appended
* @param string The parameters defining which table and fields to edit. Syntax is [tablename]:[fieldname],[fieldname],[fieldname],... OR [fieldname],[fieldname],[fieldname],... (basically "[tablename]:" is optional, default table is the one of the "current record" used in the function). The fieldlist is sent as "&columnsOnly=" parameter to alt_doc.php * @param string $params The parameters defining which table and fields to edit. Syntax is [tablename]:[fieldname],[fieldname],[fieldname],... OR [fieldname],[fieldname],[fieldname],... (basically "[tablename]:" is optional, default table is the one of the "current record" used in the function). The fieldlist is sent as "&columnsOnly=" parameter to alt_doc.php
* @param array TypoScript properties for configuring the edit icons. * @param array $conf TypoScript properties for configuring the edit icons.
* @param string The "table:uid" of the record being shown. If empty string then $this->currentRecord is used. For new records (set by $conf['newRecordFromTable']) it's auto-generated to "[tablename]:NEW" * @param string $currentRecord The "table:uid" of the record being shown. If empty string then $this->currentRecord is used. For new records (set by $conf['newRecordFromTable']) it's auto-generated to "[tablename]:NEW"
* @param array Alternative data array to use. Default is $this->data * @param array $dataArr Alternative data array to use. Default is $this->data
* @param string Additional URL parameters for the link pointing to alt_doc.php * @param string $addUrlParamStr Additional URL parameters for the link pointing to alt_doc.php
* @param string $table
* @param integer $editUid
* @param string $fieldList
* @return string The input content string, possibly with edit icons added (not necessarily in the end but just after the last string of normal content. * @return string The input content string, possibly with edit icons added (not necessarily in the end but just after the last string of normal content.
*/ */
public function editIcons($content, $params, array $conf=array(), $currentRecord='', array $dataArr=array(), $addUrlParamStr='', $table, $editUid, $fieldList) { public function editIcons($content, $params, array $conf =array(), $currentRecord = '', array $dataArr = array(), $addUrlParamStr = '', $table, $editUid, $fieldList) {
// Special content is about to be shown, so the cache must be disabled. // Special content is about to be shown, so the cache must be disabled.
$GLOBALS['TSFE']->set_no_cache(); $GLOBALS['TSFE']->set_no_cache();
$style = $conf['styleAttribute'] ? ' style="' . htmlspecialchars($conf['styleAttribute']) . '"' : ''; $style = $conf['styleAttribute'] ? ' style="' . htmlspecialchars($conf['styleAttribute']) . '"' : '';
@ -216,16 +223,16 @@ class tx_feedit_editpanel {
* Helper function for editPanel() which wraps icons in the panel in a link with the action of the panel. * Helper function for editPanel() which wraps icons in the panel in a link with the action of the panel.
* The links are for some of them not simple hyperlinks but onclick-actions which submits a little form which the panel is wrapped in. * The links are for some of them not simple hyperlinks but onclick-actions which submits a little form which the panel is wrapped in.
* *
* @param string The string to wrap in a link, typ. and image used as button in the edit panel. * @param string $string The string to wrap in a link, typ. and image used as button in the edit panel.
* @param string The name of the form wrapping the edit panel. * @param string $formName The name of the form wrapping the edit panel.
* @param string The command of the link. There is a predefined list available: edit, new, up, down etc. * @param string $cmd The command of the link. There is a predefined list available: edit, new, up, down etc.
* @param string The "table:uid" of the record being processed by the panel. * @param string $currentRecord The "table:uid" of the record being processed by the panel.
* @param string Text string with confirmation message; If set a confirm box will be displayed before carrying out the action (if Yes is pressed) * @param string $confirm Text string with confirmation message; If set a confirm box will be displayed before carrying out the action (if Yes is pressed)
* @param integer "New pid" - for new records * @param integer $nPid "New pid" - for new records
* @return string A <a> tag wrapped string. * @return string A <a> tag wrapped string.
* @see editPanel(), editIcons(), t3lib_tsfeBeUserAuth::extEditAction() * @see editPanel(), editIcons(), t3lib_tsfeBeUserAuth::extEditAction()
*/ */
protected function editPanelLinkWrap($string, $formName, $cmd, $currentRecord='', $confirm='', $nPid='') { protected function editPanelLinkWrap($string, $formName, $cmd, $currentRecord = '', $confirm = '', $nPid = '') {
// Editing forms on page only supported in Live workspace (because of incomplete implementation) // Editing forms on page only supported in Live workspace (because of incomplete implementation)
$editFormsOnPage = $GLOBALS['BE_USER']->uc['TSFE_adminConfig']['edit_editFormsOnPage'] && $GLOBALS['BE_USER']->workspace === 0; $editFormsOnPage = $GLOBALS['BE_USER']->uc['TSFE_adminConfig']['edit_editFormsOnPage'] && $GLOBALS['BE_USER']->workspace === 0;
$nV=t3lib_div::_GP('ADMCMD_view') ? 1 : 0; $nV=t3lib_div::_GP('ADMCMD_view') ? 1 : 0;
@ -263,9 +270,9 @@ class tx_feedit_editpanel {
/** /**
* Creates a link to a script (eg. typo3/alt_doc.php or typo3/db_new.php) which either opens in the current frame OR in a pop-up window. * Creates a link to a script (eg. typo3/alt_doc.php or typo3/db_new.php) which either opens in the current frame OR in a pop-up window.
* *
* @param string The string to wrap in a link, typ. and image used as button in the edit panel. * @param string $string The string to wrap in a link, typ. and image used as button in the edit panel.
* @param string The URL of the link. Should be absolute if supposed to work with <base> path set. * @param string $url The URL of the link. Should be absolute if supposed to work with <base> path set.
* @param string The "table:uid" of the record being processed by the panel. * @param string $currentRecord The "table:uid" of the record being processed by the panel.
* @return string A <a> tag wrapped string. * @return string A <a> tag wrapped string.
* @see editPanelLinkWrap() * @see editPanelLinkWrap()
*/ */
@ -289,11 +296,11 @@ class tx_feedit_editpanel {
* Wraps the input content string in a table with a gray border if the table/row combination evaluates to being disabled/hidden. * Wraps the input content string in a table with a gray border if the table/row combination evaluates to being disabled/hidden.
* Used for marking previewed records in the frontend. * Used for marking previewed records in the frontend.
* *
* @param string The table name * @param string $table The table name
* @param array The data record from $table * @param array $row The data record from $table
* @param string The content string to wrap * @param string $content The content string to wrap
* @param integer The thickness of the border * @param integer $thick The thickness of the border
* @param array The array with TypoScript properties for the content object * @param array $conf The array with TypoScript properties for the content object
* @return string The input string wrapped in a table with a border color of #cccccc and thickness = $thick * @return string The input string wrapped in a table with a border color of #cccccc and thickness = $thick
* @see editPanel() * @see editPanel()
*/ */
@ -322,8 +329,8 @@ class tx_feedit_editpanel {
/** /**
* Returns TRUE if the input table/row would be hidden in the frontend (according nto the current time and simulate user group) * Returns TRUE if the input table/row would be hidden in the frontend (according nto the current time and simulate user group)
* *
* @param string The table name * @param string $table The table name
* @param array The data record * @param array $row The data record
* @return boolean * @return boolean
* @see editPanelPreviewBorder() * @see editPanelPreviewBorder()
*/ */
@ -339,14 +346,14 @@ class tx_feedit_editpanel {
/** /**
* Returns the editing form for a content element. * Returns the editing form for a content element.
* *
* @param string Form tag * @param string $formTag Form tag
* @param string Form name * @param string $formName Form name
* @param string the command * @param string $theCmd The command
* @param integer newUID * @param integer $newUID newUID
* @param array dataArray for element * @param array $dataArray dataArray for element
* @param string Table name of element * @param string $table Table name of element
* @param string Current record * @param string $currentRecord Current record
* @param string Blackline * @param string $blackLine Blackline
* @return string * @return string
*/ */
protected function editContent($formTag, $formName, $theCmd, $newUID, array $dataArray, $table, $currentRecord, $blackLine) { protected function editContent($formTag, $formName, $theCmd, $newUID, array $dataArray, $table, $currentRecord, $blackLine) {
@ -363,7 +370,7 @@ class tx_feedit_editpanel {
// Icon only mode for CSH destroys the layout for frontend editing so force full text mode instead. // Icon only mode for CSH destroys the layout for frontend editing so force full text mode instead.
// @todo Make sure the necessary Javascript and CSS are included so that CSH can work properly in all modes. // @todo Make sure the necessary Javascript and CSS are included so that CSH can work properly in all modes.
if($tceforms->edit_showFieldHelp == 'icon') { if ($tceforms->edit_showFieldHelp == 'icon') {
$tceforms->edit_showFieldHelp = 'text'; $tceforms->edit_showFieldHelp = 'text';
} }
@ -390,7 +397,7 @@ class tx_feedit_editpanel {
$panel .= $tceforms->intoTemplate(array('ITEM' => $buttons)); $panel .= $tceforms->intoTemplate(array('ITEM' => $buttons));
$panel .= $tceforms->getMainFields($table, $processedDataArr); $panel .= $tceforms->getMainFields($table, $processedDataArr);
$hiddenF = ""; $hiddenF = '';
if ($theCmd == 'new') { if ($theCmd == 'new') {
$hiddenF .= '<input type="hidden" name="TSFE_EDIT[data][' . $table . '][NEW][pid]" value="' . $newUID . '" />'; $hiddenF .= '<input type="hidden" name="TSFE_EDIT[data][' . $table . '][NEW][pid]" value="' . $newUID . '" />';
if ($table == 'pages') { if ($table == 'pages') {