XOOPS / XoopsCore

htdocs/class/xml/rss/xmlrss2parser.php

Spacing +1 added lines, -1 removed lines

@@ -156,7 +156,7 @@
         if (!isset($this->_tempArr[$name])) {
             $this->_tempArr[$name] = $value;
         } else {
-            $this->_tempArr[$name] .= $delim . $value;
+            $this->_tempArr[$name] .= $delim.$value;
         }
     }
 

htdocs/class/xml/themesetparser.php

Spacing +16 added lines, -16 removed lines

@@ -133,7 +133,7 @@ 
         if (!isset($this->tempArr[$name])) {
             $this->tempArr[$name] = $value;
         } else {
-            $this->tempArr[$name] .= $delim . $value;
+            $this->tempArr[$name] .= $delim.$value;
         }
     }
 
@@ -182,7 +182,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('date', $data);
@@ -213,7 +213,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         $parser->resetTempArr();
     }
 
@@ -223,7 +223,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         //todo where does this method come from??
         $parser->setCreditsData($parser->getTempArr());
     }
@@ -249,7 +249,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'template':
                 $parser->setTempArr('description', $data);
@@ -283,7 +283,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('generator', $data);
@@ -312,7 +312,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('name', $data);
@@ -346,7 +346,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'author':
                 $parser->setTempArr('email', $data);
@@ -377,7 +377,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'author':
                 $parser->setTempArr('link', $data);
@@ -408,7 +408,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         $parser->resetTempArr();
         if (isset($attributes['name'])) {
             $parser->setTempArr('name', $attributes['name']);
@@ -421,7 +421,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         $value = $parser->getTempArr();
         $parser->setTemplatesData($value);
     }
@@ -447,7 +447,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         $parser->resetTempArr();
         if (isset($attributes['name'])) {
             $parser->setTempArr('name', $attributes['name']);
@@ -460,7 +460,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         $value = $parser->getTempArr();
         $parser->setImagesData($value);
     }
@@ -486,7 +486,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'template':
             case 'image':
@@ -518,7 +518,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'template':
                 $parser->setTempArr('type', $data);
@@ -549,7 +549,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser, 'XoopsThemeSetParser')) return;
         switch ($parser->getParentTag()) {
             case 'image':
                 $parser->setTempArr('tag', $data);

Braces +45 added lines, -15 removed lines

@@ -182,7 +182,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('date', $data);
@@ -213,7 +215,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         $parser->resetTempArr();
     }
 
@@ -223,7 +227,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         //todo where does this method come from??
         $parser->setCreditsData($parser->getTempArr());
     }
@@ -249,7 +255,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'template':
                 $parser->setTempArr('description', $data);
@@ -283,7 +291,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('generator', $data);
@@ -312,7 +322,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'themeset':
                 $parser->setThemeSetData('name', $data);
@@ -346,7 +358,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'author':
                 $parser->setTempArr('email', $data);
@@ -377,7 +391,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'author':
                 $parser->setTempArr('link', $data);
@@ -408,7 +424,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         $parser->resetTempArr();
         if (isset($attributes['name'])) {
             $parser->setTempArr('name', $attributes['name']);
@@ -421,7 +439,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         $value = $parser->getTempArr();
         $parser->setTemplatesData($value);
     }
@@ -447,7 +467,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         $parser->resetTempArr();
         if (isset($attributes['name'])) {
             $parser->setTempArr('name', $attributes['name']);
@@ -460,7 +482,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         $value = $parser->getTempArr();
         $parser->setImagesData($value);
     }
@@ -486,7 +510,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'template':
             case 'image':
@@ -518,7 +544,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'template':
                 $parser->setTempArr('type', $data);
@@ -549,7 +577,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsThemeSetParser')) return;
+        if (!is_a($parser,'XoopsThemeSetParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'image':
                 $parser->setTempArr('tag', $data);

htdocs/class/xml/rpc/metaweblogapi.php

Switch Indentation +17 added lines, -17 removed lines

@@ -207,23 +207,23 @@
                         foreach ($ret[$i] as $key => $value) {
                             $maptag = $this->_getXoopsTagMap($key);
                             switch($maptag) {
-                            case 'userid':
-                                $struct->add('userid', new XoopsXmlRpcString($value));
-                                break;
-                            case 'dateCreated':
-                                $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
-                                break;
-                            case 'postid':
-                                $struct->add('postid', new XoopsXmlRpcString($value));
-                                $struct->add('link', new XoopsXmlRpcString($xoops_url.'/modules/news/article.php?item_id='.$value));
-                                $struct->add('permaLink', new XoopsXmlRpcString($xoops_url.'/modules/news/article.php?item_id='.$value));
-                                break;
-                            case 'title':
-                                $struct->add('title', new XoopsXmlRpcString($value));
-                                break;
-                            default :
-                                $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
-                                break;
+                                case 'userid':
+                                    $struct->add('userid', new XoopsXmlRpcString($value));
+                                    break;
+                                case 'dateCreated':
+                                    $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
+                                    break;
+                                case 'postid':
+                                    $struct->add('postid', new XoopsXmlRpcString($value));
+                                    $struct->add('link', new XoopsXmlRpcString($xoops_url.'/modules/news/article.php?item_id='.$value));
+                                    $struct->add('permaLink', new XoopsXmlRpcString($xoops_url.'/modules/news/article.php?item_id='.$value));
+                                    break;
+                                case 'title':
+                                    $struct->add('title', new XoopsXmlRpcString($value));
+                                    break;
+                                default :
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
+                                    break;
                             }
                         }
                         $struct->add('description', new XoopsXmlRpcString($content));

Spacing +7 added lines, -7 removed lines

@@ -62,7 +62,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                         echo $m;
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
@@ -103,7 +103,7 @@ 
                     $maptag = $this->_getXoopsTagMap($tag);
                     if (!isset($this->params[3][$maptag])) {
                         $data = $this->_getTagCdata($this->params[3]['description'], $maptag, true);
-                        if (trim($data) == ''){
+                        if (trim($data) == '') {
                             if ($detail['required']) {
                                 $missing[] = $tag;
                             }
@@ -117,7 +117,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
                 } else {
@@ -167,14 +167,14 @@ 
                             break;
                         case 'postid':
                             $struct->add('postid', new XoopsXmlRpcString($value));
-                            $struct->add('link', new XoopsXmlRpcString($xoops_url . '/modules/xoopssections/item.php?item=' . $value));
-                            $struct->add('permaLink', new XoopsXmlRpcString($xoops_url . '/modules/xoopssections/item.php?item=' . $value));
+                            $struct->add('link', new XoopsXmlRpcString($xoops_url.'/modules/xoopssections/item.php?item='.$value));
+                            $struct->add('permaLink', new XoopsXmlRpcString($xoops_url.'/modules/xoopssections/item.php?item='.$value));
                             break;
                         case 'title':
                             $struct->add('title', new XoopsXmlRpcString($value));
                             break;
                         default :
-                            $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
+                            $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
                             break;
                     }
                 }
@@ -206,7 +206,7 @@ 
                         $content = '';
                         foreach ($ret[$i] as $key => $value) {
                             $maptag = $this->_getXoopsTagMap($key);
-                            switch($maptag) {
+                            switch ($maptag) {
                             case 'userid':
                                 $struct->add('userid', new XoopsXmlRpcString($value));
                                 break;

htdocs/class/xml/rpc/bloggerapi.php

Indentation +1 added lines, -1 removed lines

@@ -231,7 +231,7 @@
         } else {
             $arr = new XoopsXmlRpcArray();
             $struct = new XoopsXmlRpcStruct();
-			$xoops_url = \XoopsBaseConfig::get('url');
+            $xoops_url = \XoopsBaseConfig::get('url');
             $struct->add('url', new XoopsXmlRpcString($xoops_url.'/modules/'.$this->module->getVar('dirname').'/'));
             $struct->add('blogid', new XoopsXmlRpcString($this->module->getVar('mid')));
             $struct->add('blogName', new XoopsXmlRpcString('XOOPS Blog'));

Switch Indentation +33 added lines, -33 removed lines

@@ -154,18 +154,18 @@ 
                 foreach ($ret as $key => $value) {
                     $maptag = $this->_getXoopsTagMap($key);
                     switch($maptag) {
-                    case 'userid':
-                        $struct->add('userid', new XoopsXmlRpcString($value));
-                        break;
-                    case 'dateCreated':
-                        $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
-                        break;
-                    case 'postid':
-                        $struct->add('postid', new XoopsXmlRpcString($value));
-                        break;
-                    default :
-                        $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
-                        break;
+                        case 'userid':
+                            $struct->add('userid', new XoopsXmlRpcString($value));
+                            break;
+                        case 'dateCreated':
+                            $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
+                            break;
+                        case 'postid':
+                            $struct->add('postid', new XoopsXmlRpcString($value));
+                            break;
+                        default :
+                            $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
+                            break;
                     }
                 }
                 $struct->add('content', new XoopsXmlRpcString($content));
@@ -198,18 +198,18 @@ 
                         foreach($ret[$i] as $key => $value) {
                             $maptag = $this->_getXoopsTagMap($key);
                             switch($maptag) {
-                            case 'userid':
-                                $struct->add('userid', new XoopsXmlRpcString($value));
-                                break;
-                            case 'dateCreated':
-                                $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
-                                break;
-                            case 'postid':
-                                $struct->add('postid', new XoopsXmlRpcString($value));
-                                break;
-                            default :
-                                $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
-                                break;
+                                case 'userid':
+                                    $struct->add('userid', new XoopsXmlRpcString($value));
+                                    break;
+                                case 'dateCreated':
+                                    $struct->add('dateCreated', new XoopsXmlRpcDatetime($value));
+                                    break;
+                                case 'postid':
+                                    $struct->add('postid', new XoopsXmlRpcString($value));
+                                    break;
+                                default :
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
+                                    break;
                             }
                         }
                         $struct->add('content', new XoopsXmlRpcString($content));
@@ -262,15 +262,15 @@ 
             $this->response->add(new XoopsXmlRpcFault(104));
         } else {
             switch ($this->params[5]) {
-            case 'main':
-                $this->response->add(new XoopsXmlRpcFault(107));
-                break;
-            case 'archiveIndex':
-                $this->response->add(new XoopsXmlRpcFault(107));
-                break;
-            default:
-                $this->response->add(new XoopsXmlRpcFault(107));
-                break;
+                case 'main':
+                    $this->response->add(new XoopsXmlRpcFault(107));
+                    break;
+                case 'archiveIndex':
+                    $this->response->add(new XoopsXmlRpcFault(107));
+                    break;
+                default:
+                    $this->response->add(new XoopsXmlRpcFault(107));
+                    break;
             }
         }
     }

Spacing +8 added lines, -8 removed lines

@@ -38,7 +38,7 @@ 
         if (!$this->_checkUser($this->params[2], $this->params[3])) {
             $this->response->add(new XoopsXmlRpcFault(104));
         } else {
-            if (!$fields =& $this->_getPostFields(null, $this->params[1])) {
+            if (!$fields = & $this->_getPostFields(null, $this->params[1])) {
                 $this->response->add(new XoopsXmlRpcFault(106));
             } else {
                 $missing = array();
@@ -46,7 +46,7 @@ 
                 foreach ($fields as $tag => $detail) {
                     $maptag = $this->_getXoopsTagMap($tag);
                     $data = $this->_getTagCdata($this->params[4], $maptag, true);
-                    if (trim($data) == ''){
+                    if (trim($data) == '') {
                         if ($detail['required']) {
                             $missing[] = $maptag;
                         }
@@ -57,7 +57,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
                 } else {
@@ -91,7 +91,7 @@ 
                 $post = array();
                 foreach ($fields as $tag => $detail) {
                     $data = $this->_getTagCdata($this->params[4], $tag, true);
-                    if (trim($data) == ''){
+                    if (trim($data) == '') {
                         if ($detail['required']) {
                             $missing[] = $tag;
                         }
@@ -102,7 +102,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
                 } else {
@@ -153,7 +153,7 @@ 
                 $content = '';
                 foreach ($ret as $key => $value) {
                     $maptag = $this->_getXoopsTagMap($key);
-                    switch($maptag) {
+                    switch ($maptag) {
                     case 'userid':
                         $struct->add('userid', new XoopsXmlRpcString($value));
                         break;
@@ -195,9 +195,9 @@ 
                     for ($i = 0; $i < $count; ++$i) {
                         $struct = new XoopsXmlRpcStruct();
                         $content = '';
-                        foreach($ret[$i] as $key => $value) {
+                        foreach ($ret[$i] as $key => $value) {
                             $maptag = $this->_getXoopsTagMap($key);
-                            switch($maptag) {
+                            switch ($maptag) {
                             case 'userid':
                                 $struct->add('userid', new XoopsXmlRpcString($value));
                                 break;

htdocs/class/xml/rpc/xmlrpctag.php

Spacing +13 added lines, -13 removed lines

@@ -54,10 +54,10 @@ 
             if (!$tag->isFault()) {
                 $payload .= $tag->render();
             } else {
-                return '<?xml version="1.0"?><methodResponse>' . $tag->render() . '</methodResponse>';
+                return '<?xml version="1.0"?><methodResponse>'.$tag->render().'</methodResponse>';
             }
         }
-        return '<?xml version="1.0"?><methodResponse><params><param>' . $payload . '</param></params></methodResponse>';
+        return '<?xml version="1.0"?><methodResponse><params><param>'.$payload.'</param></params></methodResponse>';
     }
 }
 
@@ -88,9 +88,9 @@ 
         $payload = '';
         foreach ($this->_tags as $tag) {
             /* @var $tag XoopsXmlRpcTag */
-            $payload .= '<param>' . $tag->render() . '</param>';
+            $payload .= '<param>'.$tag->render().'</param>';
         }
-        return '<?xml version="1.0"?><methodCall><methodName>' . $this->methodName . '</methodName><params>' . $payload . '</params></methodCall>';
+        return '<?xml version="1.0"?><methodCall><methodName>'.$this->methodName.'</methodName><params>'.$payload.'</params></methodCall>';
     }
 }
 
@@ -209,9 +209,9 @@ 
                 $string = 'Method response error';
                 break;
         }
-        $string .= "\n" . $this->_extra;
+        $string .= "\n".$this->_extra;
 
-        return '<fault><value><struct><member><name>faultCode</name><value>' . $this->_code . '</value></member><member><name>faultString</name><value>' . $this->encode($string) . '</value></member></struct></value></fault>';
+        return '<fault><value><struct><member><name>faultCode</name><value>'.$this->_code.'</value></member><member><name>faultString</name><value>'.$this->encode($string).'</value></member></struct></value></fault>';
     }
 }
 
@@ -238,7 +238,7 @@ 
      */
     public function render()
     {
-        return '<value><int>' . $this->_value . '</int></value>';
+        return '<value><int>'.$this->_value.'</int></value>';
     }
 }
 
@@ -265,7 +265,7 @@ 
      */
     public function render()
     {
-        return '<value><double>' . $this->_value . '</double></value>';
+        return '<value><double>'.$this->_value.'</double></value>';
     }
 }
 
@@ -292,7 +292,7 @@ 
      */
     public function render()
     {
-        return '<value><boolean>' . $this->_value . '</boolean></value>';
+        return '<value><boolean>'.$this->_value.'</boolean></value>';
     }
 }
 
@@ -319,7 +319,7 @@ 
      */
     public function render()
     {
-        return '<value><string>' . $this->encode($this->_value) . '</string></value>';
+        return '<value><string>'.$this->encode($this->_value).'</string></value>';
     }
 }
 
@@ -350,7 +350,7 @@ 
      */
     public function render()
     {
-        return '<value><dateTime.iso8601>' . gmstrftime("%Y%m%dT%H:%M:%S", $this->_value) . '</dateTime.iso8601></value>';
+        return '<value><dateTime.iso8601>'.gmstrftime("%Y%m%dT%H:%M:%S", $this->_value).'</dateTime.iso8601></value>';
     }
 }
 
@@ -377,7 +377,7 @@ 
      */
     public function render()
     {
-        return '<value><base64>' . $this->_value . '</base64></value>';
+        return '<value><base64>'.$this->_value.'</base64></value>';
     }
 }
 
@@ -443,7 +443,7 @@ 
         $ret = '<value><struct>';
         foreach ($this->_tags as $tag) {
             /* @var $tag['value'] XoopsXmlRplTag */
-            $ret .= '<member><name>' . $this->encode($tag['name']) . '</name>' . $tag['value']->render() . '</member>';
+            $ret .= '<member><name>'.$this->encode($tag['name']).'</name>'.$tag['value']->render().'</member>';
         }
         $ret .= '</struct></value>';
         return $ret;

htdocs/class/xml/rpc/xmlrpcapi.php

Spacing +2 added lines, -2 removed lines

@@ -180,7 +180,7 @@ 
     {
         $ret = '';
         $match = array();
-        if (preg_match("/\<" . $tag . "\>(.*)\<\/" . $tag . "\>/is", $text, $match)) {
+        if (preg_match("/\<".$tag."\>(.*)\<\/".$tag."\>/is", $text, $match)) {
             if ($remove) {
                 $text = str_replace($match[0], '', $text);
             }
@@ -201,7 +201,7 @@ 
     {
         if (strtolower(get_class($this)) !== 'xoopsapi') {
             $xoops_root_path = \XoopsBaseConfig::get('root-path');
-            require_once($xoops_root_path . '/class/xml/rpc/xoopsapi.php');
+            require_once($xoops_root_path.'/class/xml/rpc/xoopsapi.php');
             return new XoopsApi($params, $this->response, $this->module);
         } else {
             return $this;

htdocs/class/xml/rpc/xmlrpcparser.php

Indentation +6 added lines, -6 removed lines

@@ -336,8 +336,8 @@ 
 {
 
     /**
-    * @return string[]
-    */
+     * @return string[]
+     */
     public function getName()
     {
         return array('int', 'i4');
@@ -647,10 +647,10 @@ 
     }
 
     /**
-    * @param SaxParser $parser
-    * @param array $attributes
-    * @return void
-    */
+     * @param SaxParser $parser
+     * @param array $attributes
+     * @return void
+     */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
         if (!is_a($parser,'XoopsXmlRpcParser')) return;

Spacing +17 added lines, -17 removed lines

@@ -324,7 +324,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setMethodName($data);
     }
 }
@@ -350,7 +350,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setTempValue((int)($data));
     }
 }
@@ -376,7 +376,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $data = (float)$data;
         $parser->setTempValue($data);
     }
@@ -403,7 +403,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $data = (boolean)$data;
         $parser->setTempValue($data);
     }
@@ -430,7 +430,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setTempValue((string)($data));
     }
 }
@@ -456,7 +456,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $matches = array();
         if (!preg_match("/^([0-9]{4})([0-9]{2})([0-9]{2})T([0-9]{2}):([0-9]{2}):([0-9]{2})$/", $data, $matches)) {
             $parser->setTempValue(time());
@@ -487,7 +487,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setTempValue(base64_decode($data));
     }
 }
@@ -513,7 +513,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         switch ($parser->getParentTag()) {
             case 'member':
                 $parser->setTempName($data);
@@ -545,7 +545,7 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         switch ($parser->getParentTag()) {
             case 'member':
                 $parser->setTempValue($data);
@@ -566,7 +566,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         //$parser->resetTempValue();
     }
 
@@ -576,7 +576,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         switch ($parser->getCurrentTag()) {
             case 'member':
                 $parser->setTempMember($parser->getTempName(), $parser->getTempValue());
@@ -614,7 +614,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setWorkingLevel();
         $parser->resetTempMember();
     }
@@ -625,7 +625,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $member = $parser->getTempMember();
         $parser->releaseWorkingLevel();
         $parser->setTempStruct($member);
@@ -653,7 +653,7 @@ 
     */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setWorkingLevel();
         $parser->resetTempArray();
     }
@@ -664,7 +664,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setTempValue($parser->getTempArray());
         $parser->releaseWorkingLevel();
     }
@@ -691,7 +691,7 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setWorkingLevel();
         $parser->resetTempStruct();
     }
@@ -702,7 +702,7 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser, 'XoopsXmlRpcParser')) return;
         $parser->setTempValue($parser->getTempStruct());
         $parser->releaseWorkingLevel();
     }

Braces +51 added lines, -17 removed lines

@@ -324,7 +324,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setMethodName($data);
     }
 }
@@ -350,7 +352,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setTempValue((int)($data));
     }
 }
@@ -376,7 +380,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $data = (float)$data;
         $parser->setTempValue($data);
     }
@@ -403,7 +409,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $data = (boolean)$data;
         $parser->setTempValue($data);
     }
@@ -430,7 +438,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setTempValue((string)($data));
     }
 }
@@ -456,7 +466,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $matches = array();
         if (!preg_match("/^([0-9]{4})([0-9]{2})([0-9]{2})T([0-9]{2}):([0-9]{2}):([0-9]{2})$/", $data, $matches)) {
             $parser->setTempValue(time());
@@ -487,7 +499,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setTempValue(base64_decode($data));
     }
 }
@@ -513,7 +527,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'member':
                 $parser->setTempName($data);
@@ -545,7 +561,9 @@ 
      */
     public function handleCharacterData(SaxParser $parser, &$data)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         switch ($parser->getParentTag()) {
             case 'member':
                 $parser->setTempValue($data);
@@ -566,7 +584,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         //$parser->resetTempValue();
     }
 
@@ -576,7 +596,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         switch ($parser->getCurrentTag()) {
             case 'member':
                 $parser->setTempMember($parser->getTempName(), $parser->getTempValue());
@@ -614,7 +636,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setWorkingLevel();
         $parser->resetTempMember();
     }
@@ -625,7 +649,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $member = $parser->getTempMember();
         $parser->releaseWorkingLevel();
         $parser->setTempStruct($member);
@@ -653,7 +679,9 @@ 
     */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setWorkingLevel();
         $parser->resetTempArray();
     }
@@ -664,7 +692,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setTempValue($parser->getTempArray());
         $parser->releaseWorkingLevel();
     }
@@ -691,7 +721,9 @@ 
      */
     public function handleBeginElement(SaxParser $parser, &$attributes)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setWorkingLevel();
         $parser->resetTempStruct();
     }
@@ -702,7 +734,9 @@ 
      */
     public function handleEndElement(SaxParser $parser)
     {
-        if (!is_a($parser,'XoopsXmlRpcParser')) return;
+        if (!is_a($parser,'XoopsXmlRpcParser')) {
+            return;
+        }
         $parser->setTempValue($parser->getTempStruct());
         $parser->releaseWorkingLevel();
     }

htdocs/class/xml/rpc/xoopsapi.php

Spacing +12 added lines, -12 removed lines

@@ -61,7 +61,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
                 } else {
@@ -151,7 +151,7 @@ 
                 if (count($missing) > 0) {
                     $msg = '';
                     foreach ($missing as $m) {
-                        $msg .= '<' . $m . '> ';
+                        $msg .= '<'.$m.'> ';
                     }
                     $this->response->add(new XoopsXmlRpcFault(109, $msg));
                 } else {
@@ -249,7 +249,7 @@ 
                     $struct = new XoopsXmlRpcStruct();
                     $content = '';
                     foreach ($ret as $key => $value) {
-                        switch($key) {
+                        switch ($key) {
                             case 'uid':
                                 $struct->add('userid', new XoopsXmlRpcString($value));
                                 break;
@@ -258,14 +258,14 @@ 
                                 break;
                             case 'storyid':
                                 $struct->add('postid', new XoopsXmlRpcString($value));
-                                $struct->add('link', new XoopsXmlRpcString(\XoopsBaseConfig::get('url') . '/modules/news/article.php?item_id=' . $value));
-                                $struct->add('permaLink', new XoopsXmlRpcString(\XoopsBaseConfig::get('url') . '/modules/news/article.php?item_id=' . $value));
+                                $struct->add('link', new XoopsXmlRpcString(\XoopsBaseConfig::get('url').'/modules/news/article.php?item_id='.$value));
+                                $struct->add('permaLink', new XoopsXmlRpcString(\XoopsBaseConfig::get('url').'/modules/news/article.php?item_id='.$value));
                                 break;
                             case 'title':
                                 $struct->add('title', new XoopsXmlRpcString($value));
                                 break;
                             default :
-                                $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
+                                $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
                                 break;
                         }
                     }
@@ -319,8 +319,8 @@ 
                     for ($i = 0; $i < $count; ++$i) {
                         $struct = new XoopsXmlRpcStruct();
                         $content = '';
-                        foreach($ret[$i] as $key => $value) {
-                            switch($key) {
+                        foreach ($ret[$i] as $key => $value) {
+                            switch ($key) {
                                 case 'uid':
                                     $struct->add('userid', new XoopsXmlRpcString($value));
                                     break;
@@ -329,14 +329,14 @@ 
                                     break;
                                 case 'storyid':
                                     $struct->add('postid', new XoopsXmlRpcString($value));
-                                    $struct->add('link', new XoopsXmlRpcString(\XoopsBaseConfig::get('url') . '/modules/news/article.php?item_id=' . $value));
-                                    $struct->add('permaLink', new XoopsXmlRpcString(\XoopsBaseConfig::get('url') . '/modules/news/article.php?item_id=' . $value));
+                                    $struct->add('link', new XoopsXmlRpcString(\XoopsBaseConfig::get('url').'/modules/news/article.php?item_id='.$value));
+                                    $struct->add('permaLink', new XoopsXmlRpcString(\XoopsBaseConfig::get('url').'/modules/news/article.php?item_id='.$value));
                                     break;
                                 case 'title':
                                     $struct->add('title', new XoopsXmlRpcString($value));
                                     break;
                                 default :
-                                    $content .= '<' . $key . '>' . trim($value) . '</' . $key . '>';
+                                    $content .= '<'.$key.'>'.trim($value).'</'.$key.'>';
                                     break;
                             }
                         }
@@ -351,7 +351,7 @@ 
         return null;
     }
 
-    function getCategories($respond=true)
+    function getCategories($respond = true)
     {
         global $xoopsDB;
         if (!$this->_checkUser($this->params[1], $this->params[2])) {

htdocs/class/xml/saxparser.php

Spacing +1 added lines, -1 removed lines

@@ -380,7 +380,7 @@
             $ret = '';
             if (count($this->errors) > 0) {
                 foreach ($this->errors as $error) {
-                    $ret .= $error . '<br />';
+                    $ret .= $error.'<br />';
                 }
             }
             return $ret;