[TASK] CGL: fix FunctionCallArgumentSpacingNoSpaceAfterComma violations
authorFelix Kopp <felix-source@phorax.com>
Sun, 3 Jun 2012 11:43:36 +0000 (13:43 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sun, 10 Jun 2012 14:20:19 +0000 (16:20 +0200)
Adds spaces to function definition where needed.

Change-Id: I3265a0fa9a82da91babfd8b87f0019e785f1574b
Fixes: #37709
Releases: 6.0
Reviewed-on: http://review.typo3.org/11807
Reviewed-by: Wouter Wolters
Reviewed-by: Oliver Klee
Reviewed-by: Roland Waldner
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
20 files changed:
t3lib/class.t3lib_extobjbase.php
typo3/sysext/beuser/mod/index.php
typo3/sysext/cms/tslib/class.tslib_pibase.php
typo3/sysext/indexed_search/class.crawler.php
typo3/sysext/indexed_search/class.doublemetaphone.php
typo3/sysext/indexed_search/class.external_parser.php
typo3/sysext/indexed_search/class.indexer.php
typo3/sysext/indexed_search/example/class.crawlerhook.php
typo3/sysext/indexed_search/modfunc2/class.tx_indexedsearch_modfunc2.php
typo3/sysext/indexed_search/pi/class.tx_indexedsearch.php
typo3/sysext/statictemplates/media/scripts/example_callfunction.php
typo3/sysext/statictemplates/media/scripts/example_itemArrayProcFunc.php
typo3/sysext/statictemplates/media/scripts/example_keepRollover.php
typo3/sysext/statictemplates/media/scripts/example_typolinkpop.php
typo3/sysext/statictemplates/media/scripts/gmenu_layers.php
typo3/sysext/statictemplates/media/scripts/testscript_EXT.php
typo3/sysext/statictemplates/media/scripts/tmenu_layers.php
typo3/view_help.php
typo3/wizard_colorpicker.php
typo3/wizard_tsconfig.php

index 046e91c..5d55267 100644 (file)
@@ -85,7 +85,7 @@
  *      class tx_funcwizards_webfunc extends t3lib_extobjbase {
  *              var $localLangFile = "locallang.php";
  *              var $function_key = "wiz";
- *              function init(&$pObj,$conf) {
+ *              function init(&$pObj, $conf) {
  *                              // OK, handles ordinary init. This includes setting up the menu array with ->modMenu
  *                      parent::init($pObj,$conf);
  *                              // Making sure that any further external classes are added to the include_once array. Notice that inclusion happens twice in the main script because of this!!!
index 957cb84..7a8f572 100755 (executable)
@@ -101,7 +101,7 @@ class localPageTree extends t3lib_browseTree {
         * @param       string          [See parent]
         * @return      string
         */
-       function PM_ATagWrap($icon, $cmd,$bMark='') {
+       function PM_ATagWrap($icon, $cmd, $bMark = '') {
                return '';
        }
 
index f6c6361..f7ce948 100644 (file)
@@ -822,7 +822,7 @@ class tslib_pibase {
         * @return      string          The processed content
         * @see tslib_cObj::editIcons()
         */
-       function pi_getEditIcon($content,$fields,$title='',$row='',$tablename='',$oConf=array()) {
+       function pi_getEditIcon($content, $fields, $title = '', $row = '', $tablename = '', $oConf = array()) {
                if ($GLOBALS['TSFE']->beUserLogin){
                        if (!$row || !$tablename) {
                                $row = $this->internal['currentRow'];
index d74c6c6..35efe21 100755 (executable)
@@ -178,7 +178,7 @@ class tx_indexedsearch_crawler {
         * @param       object          Parent object (tx_crawler lib)
         * @return      array           Result array
         */
-       function crawler_execute($params,&$pObj) {
+       function crawler_execute($params, &$pObj) {
 
                        // Indexer configuration ID must exist:
                if ($params['indexConfigUid']) {
@@ -244,7 +244,7 @@ class tx_indexedsearch_crawler {
         * @param       object          Parent object (from "crawler" extension!)
         * @return      void
         */
-       function crawler_execute_type1($cfgRec,&$session_data,$params,&$pObj) {
+       function crawler_execute_type1($cfgRec, &$session_data, $params, &$pObj) {
                if ($cfgRec['table2index'] && isset($GLOBALS['TCA'][$cfgRec['table2index']])) {
 
                                // Init session data array if not already:
@@ -305,7 +305,7 @@ class tx_indexedsearch_crawler {
         * @param       object          Parent object (from "crawler" extension!)
         * @return      void
         */
-       function crawler_execute_type2($cfgRec,&$session_data,$params,&$pObj) {
+       function crawler_execute_type2($cfgRec, &$session_data, $params, &$pObj) {
 
                        // Prepare path, making it absolute and checking:
                $readpath = $params['url'];
@@ -380,7 +380,7 @@ class tx_indexedsearch_crawler {
         * @param       object          Parent object (from "crawler" extension!)
         * @return      void
         */
-       function crawler_execute_type3($cfgRec,&$session_data,$params,&$pObj) {
+       function crawler_execute_type3($cfgRec, &$session_data, $params, &$pObj) {
 
                        // Init session data array if not already:
                if (!is_array($session_data)) {
@@ -430,7 +430,7 @@ class tx_indexedsearch_crawler {
         * @param       object          Parent object (from "crawler" extension!)
         * @return      void
         */
-       function crawler_execute_type4($cfgRec,&$session_data,$params,&$pObj) {
+       function crawler_execute_type4($cfgRec, &$session_data, $params, &$pObj) {
 
                        // Base page uid:
                $pageUid = intval($params['url']);
@@ -567,7 +567,7 @@ class tx_indexedsearch_crawler {
         * @param       string          Base URL of the indexing process (input URL must be "inside" the base URL!)
         * @return      string          Returls the URL if OK, otherwise FALSE
         */
-       function checkUrl($url,$urlLog,$baseUrl) {
+       function checkUrl($url, $urlLog, $baseUrl) {
                $url = preg_replace('/\/\/$/','/',$url);
                list($url) = explode('#',$url);
 
@@ -647,7 +647,7 @@ class tx_indexedsearch_crawler {
         * @param       array           Rootline array to relate indexing to
         * @return      void
         */
-       function indexSingleRecord($r,$cfgRec,$rl=NULL) {
+       function indexSingleRecord($r, $cfgRec, $rl = NULL) {
 
                        // Load indexer if not yet.
                $this->loadIndexerClass();
@@ -921,7 +921,7 @@ class tx_indexedsearch_files {
         * @param       object          Parent object (tx_crawler lib)
         * @return      array           Result array
         */
-       function crawler_execute($params,&$pObj) {
+       function crawler_execute($params, &$pObj) {
 
                        // Load indexer if not yet.
                $this->loadIndexerClass();
index 805896c..29e22dd 100755 (executable)
@@ -54,7 +54,7 @@ class user_DoubleMetaPhone
 //  methods
 
                // TYPO3 specific API to this class. BEGIN
-       function metaphone($string,$sys_language_uid=0) {
+       function metaphone($string, $sys_language_uid = 0) {
                $res = $this->DoubleMetaPhone($string);
                #debug(array($string,$res['primary']));
                return $res['primary'];
index 969ff18..30c21ed 100755 (executable)
@@ -374,7 +374,7 @@ class tx_indexed_search_extparse {
         * @param       string          Pointer to section (zero for all other than PDF which will have an indication of pages into which the document should be splitted.)
         * @return      array           Standard content array (title, description, keywords, body keys)
         */
-       function readFileContent($ext,$absFile,$cPKey) {
+       function readFileContent($ext, $absFile, $cPKey) {
                unset($contentArr);
 
                        // Return immediately if initialization didn't set support up:
@@ -554,7 +554,7 @@ class tx_indexed_search_extparse {
         * @param       string          Absolute filename (must exist and be validated OK before calling function)
         * @return      array           Array of pointers to sections that the document should be divided into
         */
-       function fileContentParts($ext,$absFile) {
+       function fileContentParts($ext, $absFile) {
                $cParts = array(0);
                switch ($ext) {
                        case 'pdf':
index d698a79..9c1ad55 100755 (executable)
@@ -595,7 +595,7 @@ class tx_indexedsearch_indexer {
         * @param       string          Optional charset (otherwise extracted from HTML)
         * @return      string          Converted HTML
         */
-       function convertHTMLToUtf8($content,$charset='') {
+       function convertHTMLToUtf8($content, $charset = '') {
 
                        // Find charset:
                $charset = $charset ? $charset : $this->getHTMLcharset($content);
@@ -623,7 +623,7 @@ class tx_indexedsearch_indexer {
         * @param       string          Passed by reference: Attributes of the found tag.
         * @return      boolean         Returns FALSE if tag was not found, otherwise TRUE.
         */
-       function embracingTags($string,$tagName,&$tagContent,&$stringAfter,&$paramList) {
+       function embracingTags($string, $tagName, &$tagContent, &$stringAfter, &$paramList) {
                $endTag = '</'.$tagName.'>';
                $startTag = '<'.$tagName;
 
@@ -1173,7 +1173,7 @@ class tx_indexedsearch_indexer {
         * @param       string          Absolute filename (must exist and be validated OK before calling function)
         * @return      array           Array of pointers to sections that the document should be divided into
         */
-       function fileContentParts($ext,$absFile) {
+       function fileContentParts($ext, $absFile) {
                $cParts = array(0);
 
                        // Consult relevant external document parser:
@@ -1309,7 +1309,7 @@ class tx_indexedsearch_indexer {
         * @param       integer         Bit-wise priority to type
         * @return      void
         */
-       function analyzeHeaderinfo(&$retArr,$content,$key,$offset) {
+       function analyzeHeaderinfo(&$retArr, $content, $key, $offset) {
                foreach ($content[$key] as $val) {
                        $val = substr($val, 0, 60);     // Cut after 60 chars because the index_words.baseword varchar field has this length. This MUST be the same.
 
@@ -1345,7 +1345,7 @@ class tx_indexedsearch_indexer {
         * @param       array           Standard content array
         * @return      void
         */
-       function analyzeBody(&$retArr,$content) {
+       function analyzeBody(&$retArr, $content) {
                foreach ($content['body'] as $key => $val) {
                        $val = substr($val, 0, 60);     // Cut after 60 chars because the index_words.baseword varchar field has this length. This MUST be the same.
 
@@ -1509,7 +1509,7 @@ class tx_indexedsearch_indexer {
         * @return      void
         * @see update_grlist()
         */
-       function submit_grlist($hash,$phash_x) {
+       function submit_grlist($hash, $phash_x) {
 
                        // Setting the gr_list record
                $fields = array(
@@ -1531,7 +1531,7 @@ class tx_indexedsearch_indexer {
         * @param       integer         phash of the file indexation search record
         * @return      void
         */
-       function submit_section($hash,$hash_t3) {
+       function submit_section($hash, $hash_t3) {
                $fields = array(
                        'phash' => $hash,
                        'phash_t3' => $hash_t3,
@@ -1598,7 +1598,7 @@ class tx_indexedsearch_indexer {
         * @param       array           Standard content array (using only title and body for a file)
         * @return      void
         */
-       function submitFilePage($hash,$file,$subinfo,$ext,$mtime,$ctime,$size,$content_md5h,$contentParts) {
+       function submitFilePage($hash, $file, $subinfo, $ext, $mtime, $ctime, $size, $content_md5h, $contentParts) {
 
                        // Find item Type:
                $storeItemType = $this->external_parsers[$ext]->ext2itemtype_map[$ext];
@@ -1740,7 +1740,7 @@ class tx_indexedsearch_indexer {
         * @param       integer         "phash" used to select any already indexed page to see what its mtime is.
         * @return      integer         Result integer: Generally: <0 = No indexing, >0 = Do indexing (see $this->reasons): -2) Min age was NOT exceeded and so indexing cannot occur.  -1) mtime matched so no need to reindex page. 0) N/A   1) Max age exceeded, page must be indexed again.   2) mtime of indexed page doesn't match mtime given for current content and we must index page.  3) No mtime was set, so we will index...  4) No indexed page found, so of course we will index.
         */
-       function checkMtimeTstamp($mtime,$phash) {
+       function checkMtimeTstamp($mtime, $phash) {
                if (!tx_indexedsearch_util::isTableUsed('index_phash')) {
                                // Not indexed (not in index_phash)
                        $result = 4;
@@ -1816,7 +1816,7 @@ class tx_indexedsearch_indexer {
         * @param       integer         Content hash to check
         * @return      boolean         Returns TRUE if the document needs to be indexed (that is, there was no result)
         */
-       function checkExternalDocContentHash($hashGr,$content_md5h) {
+       function checkExternalDocContentHash($hashGr, $content_md5h) {
                $result = TRUE;
                if (tx_indexedsearch_util::isTableUsed('index_phash')) {
                        $count = $GLOBALS['TYPO3_DB']->exec_SELECTcountRows('*', 'index_phash', 'phash_grouping=' . intval($hashGr) . ' AND contentHash=' . intval($content_md5h));
@@ -2115,7 +2115,7 @@ class tx_indexedsearch_indexer {
         * @param       array           Additional content identifying the (subpart of) content. For instance; PDF files are divided into groups of pages for indexing.
         * @return      array           Array with "phash_grouping" and "phash" inside.
         */
-       function setExtHashes($file,$subinfo=array()) {
+       function setExtHashes($file, $subinfo = array()) {
                        //  Set main array:
                $hash = array();
                $hArray = array(
@@ -2145,7 +2145,7 @@ class tx_indexedsearch_indexer {
         * @param       string          Key (?)
         * @return      void
         */
-       function log_push($msg,$key) {
+       function log_push($msg, $key) {
                if (is_object($GLOBALS['TT']))          $GLOBALS['TT']->push($msg,$key);
        }
 
index 1f7d83e..5c72d11 100755 (executable)
@@ -62,7 +62,7 @@ class tx_indexedsearch_crawlerhook {
         * @param       object          Parent Object (from "indexed_search" extension)
         * @return      void
         */
-       function indexOperation($cfgRec,&$session_data,$params,&$pObj) {
+       function indexOperation($cfgRec, &$session_data, $params, &$pObj) {
 
                        // Init session data array if not already:
                if (!is_array($session_data)) {
@@ -172,4 +172,4 @@ class tx_indexedsearch_crawlerhook {
                }
        }
 }
-?>
\ No newline at end of file
+?>
index 4d4c52f..11b9164 100644 (file)
@@ -101,7 +101,7 @@ class tx_indexedsearch_modfunc2 extends t3lib_extobjbase {
         * @param       array           configuration: words = max words for results, bid = pageid
         * @return      string          html table with results
         */
-       function listSeveralStats($title,$addwhere,$conf) {
+       function listSeveralStats($title, $addwhere, $conf) {
                global $LANG;
 
                $queryParts['SELECT'] = 'word, COUNT(*) AS c';
@@ -179,7 +179,7 @@ class tx_indexedsearch_modfunc2 extends t3lib_extobjbase {
         * @param       string          Perms clause
         * @return      string          Returns the list with a comma in the end (if any pages selected!)
         */
-       function extGetTreeList($id,$depth,$begin = 0,$perms_clause) {
+       function extGetTreeList($id, $depth, $begin = 0, $perms_clause) {
                // TODO: Fix this as this calls a non-static method
                return t3lib_tsfeBeUserAuth::extGetTreeList($id,$depth,$begin,$perms_clause);
        }
index 19685a2..3466896 100755 (executable)
@@ -2094,7 +2094,7 @@ class tx_indexedsearch extends tslib_pibase {
         * @param       array           Template array to modify
         * @return      array           Modified template array
         */
-       function makeInfo($row,$tmplArray) {
+       function makeInfo($row, $tmplArray) {
                $tmplArray['size'] = t3lib_div::formatSize($row['item_size']);
                $tmplArray['created'] = $this->formatCreatedDate($row['item_crdate']);
                $tmplArray['modified'] = $this->formatModifiedDate($row['item_mtime']);
@@ -2465,4 +2465,4 @@ class tx_indexedsearch extends tslib_pibase {
                return $result;
        }
 }
-?>
\ No newline at end of file
+?>
index 7bed71a..fa89d95 100644 (file)
@@ -61,7 +61,7 @@
  * @param      array           TypoScript properties passed on to this function.
  * @return     string          The input string reversed. If the TypoScript property "uppercase" was set it will also be in uppercase.
  */
-function user_reverseString($content,$conf) {
+function user_reverseString($content, $conf) {
        $content = strrev($content);
        if ($conf['uppercase']) {
                $content=strtoupper($content);
@@ -86,7 +86,7 @@ function user_reverseString($content,$conf) {
  * @param      array           TypoScript configuration
  * @return     string          HTML output, showing the current server time.
  */
-function user_printTime($content,$conf) {
+function user_printTime($content, $conf) {
        return '<font color="red">Dynamic time: '.date('H:i:s').'</font><br />';
 }
 
@@ -107,7 +107,7 @@ class user_various  {
         * @return      string          The input string reversed. If the TypoScript property "uppercase" was set it will also be in uppercase. May also be linked.
         * @see user_reverseString()
         */
-       function reverseString($content,$conf) {
+       function reverseString($content, $conf) {
                $content = strrev($content);
                if ($conf['uppercase']) {
                        $content=$this->cObj->caseshift($content,'upper');
@@ -136,7 +136,7 @@ class user_various  {
         * @param       array           TypoScript configuration
         * @return      string          HTML output, showing content elements (in reverse order if configured.)
         */
-       function listContentRecordsOnPage($content,$conf) {
+       function listContentRecordsOnPage($content, $conf) {
                $query = $GLOBALS['TYPO3_DB']->SELECTquery(
                                                'header',
                                                'tt_content',
index 3a36b7b..ff5db83 100644 (file)
@@ -70,7 +70,7 @@
  * @param      array           TypoScript configuration for the function. Notice that the property "parentObj" is a reference to the parent (calling) object (the tslib_Xmenu class instantiated)
  * @return     array           The modified $menuArr array
  */
-function user_itemArrayProcFuncTest($menuArr,$conf) {
+function user_itemArrayProcFuncTest($menuArr, $conf) {
        if ($conf['demoItemStates'])    {               // Used in the example of item states
                $c=0;
                $teststates=explode(',','NO,ACT,IFSUB,CUR,USR,SPC,USERDEF1,USERDEF2');
@@ -99,7 +99,7 @@ function user_itemArrayProcFuncTest($menuArr,$conf) {
  * @return     array           The processed $I array returned (and stored in $this->I of the parent object again)
  * @see tslib_menu::userProcess(), tslib_tmenu::writeMenu(), tslib_gmenu::writeMenu()
  */
-function user_IProcFuncTest($I,$conf) {
+function user_IProcFuncTest($I, $conf) {
        $itemRow = $conf['parentObj']->menuArr[$I['key']];
 
                // Setting the document status content to the value of the page title on mouse over
index bc73b0e..f474335 100644 (file)
@@ -48,7 +48,7 @@
  * @return     array           The processed $I array returned (and stored in $this->I of the parent object again)
  * @see tslib_menu::userProcess(), tslib_tmenu::writeMenu(), tslib_gmenu::writeMenu()
  */
-function user_keepRolloverAtOnClick($I,$conf) {
+function user_keepRolloverAtOnClick($I, $conf) {
        $itemRow = $conf['parentObj']->menuArr[$I['key']];
 
                // Setting the document status content to the value of the page title on mouse over
index 6440cb2..0574cc8 100644 (file)
@@ -66,7 +66,7 @@
  * @return     string          Return the new <a> tag
  * @see tslib_cObj::typoLink()
  */
-function user_typoLinkPopUp($content,$conf) {
+function user_typoLinkPopUp($content, $conf) {
        $aOnClick = 'openPic(\''.$GLOBALS['TSFE']->baseUrlWrap($content['url']).'\',\'popupwin\',\'width=400,height=500,status=0,menubar=0\'); return false;';
        $TAG =  '<a href="#" onclick="'.htmlspecialchars($aOnClick).'"'.$content['aTagParams'].'>';
        return $TAG;
index 3b261ba..64f3f6b 100644 (file)
@@ -216,7 +216,7 @@ GLV_restoreMenu["'.$this->WMid.'"] = "'.$this->WMactiveKey.'";
         * @param       integer         Pointer to $this->menuArr[$key] where the current menu element record is found
         * @return      string          The modified version of $item, going back into $this->I['theItem']
         */
-       function extProc_beforeAllWrap($item,$key) {
+       function extProc_beforeAllWrap($item, $key) {
                if ($this->mconf['relativeToTriggerItem']) {
                        $item = '<div id="anchorID'.t3lib_div::shortmd5($this->I['uid'].'-'.$this->WMid).'" style="position:absolute;visibility:hidden;"></div><div id="itemID'.t3lib_div::shortmd5($this->I['uid'].'-'.$this->WMid).'" style="width:100%; height:100%;">'.$item.'</div>';
                }
@@ -419,7 +419,7 @@ GLV_timeout_count++;
         * @return      string          JavaScript string for correction of the layer position (if $integer is true)
         * @see extProc_finish(), extProc_init()
         */
-       function extCalcBorderWithin($kind,$integer) {
+       function extCalcBorderWithin($kind, $integer) {
                if ($integer) {
                        switch($kind) {
                                case 'right':
index 8c4441b..659ec33 100644 (file)
@@ -12,7 +12,7 @@ if (!is_object($TSFE)) die ('Error: No parent object present.');
  * @param      array           TypoScript configuration passed
  * @return     string          Current time wrapped in <font> tags with red color
  */
-function user_printTimeExt($content,$conf) {
+function user_printTimeExt($content, $conf) {
        return '<font color="red">Dynamic time: '.date('H:i:s').'</font><br />';
 }
 
index e1ac4e6..bb6ca6b 100644 (file)
@@ -216,7 +216,7 @@ GLV_restoreMenu["'.$this->WMid.'"] = "'.$this->WMactiveKey.'";
         * @param       integer         Pointer to $this->menuArr[$key] where the current menu element record is found
         * @return      string          The modified version of $item, going back into $this->I['theItem']
         */
-       function extProc_beforeAllWrap($item,$key) {
+       function extProc_beforeAllWrap($item, $key) {
                if ($this->mconf['relativeToTriggerItem']) {
                        $item = '<div id="anchorID'.t3lib_div::shortmd5($this->I['uid'].'-'.$this->WMid).'" style="position:absolute;visibility:hidden;"></div><div id="itemID'.t3lib_div::shortmd5($this->I['uid'].'-'.$this->WMid).'" style="width:100%; height:100%;">'.$item.'</div>';
                }
@@ -419,7 +419,7 @@ GLV_timeout_count++;
         * @return      string          JavaScript string for correction of the layer position (if $integer is true)
         * @see extProc_finish(), extProc_init()
         */
-       function extCalcBorderWithin($kind,$integer) {
+       function extCalcBorderWithin($kind, $integer) {
                if ($integer) {
                        switch($kind) {
                                case 'right':
index 5103829..1041e20 100644 (file)
@@ -504,7 +504,7 @@ class SC_view_help {
         * @param       string          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
         */
-       function make_seeAlso($value,$anchorTable='') {
+       function make_seeAlso($value, $anchorTable = '') {
                        // Split references by comma or linebreak
                $items = preg_split('/[,' . LF . ']/', $value);
                $lines = array();
index d129994..07d81c5 100644 (file)
@@ -388,7 +388,7 @@ class SC_wizard_colorpicker {
         * @return      string          HEX RGB value for color
         * @see colorImage()
         */
-       function getIndex($im, $x,$y) {
+       function getIndex($im, $x, $y) {
                $rgb = ImageColorAt($im, $x, $y);
                $colorrgb = imagecolorsforindex($im, $rgb);
                $index['r'] = dechex($colorrgb['red']);
index abce934..41b6b3c 100644 (file)
@@ -408,7 +408,7 @@ class SC_wizard_tsconfig {
         * @access private
         * @see getObjTree()
         */
-       function setObj(&$objTree,$strArr,$params) {
+       function setObj(&$objTree, $strArr, $params) {
                $key = current($strArr);
                reset($strArr);
                if (count($strArr)>1) {