geekwright / XoopsCore

htdocs/uploads/filemanager/media/depot-members/config.php

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
     $usergroups = $GLOBALS['xoopsUser']->getGroups();
     $result_upload = array_intersect($usergroups, $allowed_groups_upload);
         if ($result_upload || $result_upload!=null) {
-           $upload_files=true;
+            $upload_files=true;
         }
-  $result_createfolder = array_intersect($usergroups, $allowed_groups_createfolder);
-       if ($result_createfolder || $result_createfolder!=null) {
-          $create_folders=true;
+    $result_createfolder = array_intersect($usergroups, $allowed_groups_createfolder);
+        if ($result_createfolder || $result_createfolder!=null) {
+            $create_folders=true;
     }
 }
 // XOOPS

Spacing +7 added lines, -7 removed lines

@@ -4,7 +4,7 @@ 
 // YOU CAN COPY AND CHANGE THESE VARIABLES IN FOLDERS config.php FILES
 //------------------------------------------------------------------------------
 // $image_max_width=0;
-$image_max_width=1000;
+$image_max_width = 1000;
 
 // XOOPS
 global $xoopsUser;
@@ -12,17 +12,17 @@ 
 // 2 : members
 // 3 : anonymous
 // 4 : your new group
-$allowed_groups_upload=array(1,2); // id des groupes autorisés en upload
-$allowed_groups_createfolder=array(1); // id des groupes autorisés create folder
+$allowed_groups_upload = array(1, 2); // id des groupes autorisés en upload
+$allowed_groups_createfolder = array(1); // id des groupes autorisés create folder
 if ($xoopsUser) {
     $usergroups = $GLOBALS['xoopsUser']->getGroups();
     $result_upload = array_intersect($usergroups, $allowed_groups_upload);
-        if ($result_upload || $result_upload!=null) {
-           $upload_files=true;
+        if ($result_upload || $result_upload != null) {
+           $upload_files = true;
         }
   $result_createfolder = array_intersect($usergroups, $allowed_groups_createfolder);
-       if ($result_createfolder || $result_createfolder!=null) {
-          $create_folders=true;
+       if ($result_createfolder || $result_createfolder != null) {
+          $create_folders = true;
     }
 }
 // XOOPS

htdocs/viewpmsg.php

Spacing +4 added lines, -4 removed lines

@@ -21,14 +21,14 @@ 
  * @version         $Id$
  */
 
-include __DIR__ . '/mainfile.php';
+include __DIR__.'/mainfile.php';
 
 $xoops = Xoops::getInstance();
 $xoops->events()->triggerEvent('core.viewpmsg.start');
 
 if (!$xoops->isUser()) {
-    $errormessage = XoopsLocale::E_YOU_ARE_NOT_REGISTERED . "<br />"
-        . XoopsLocale::E_REGISTER_FIRST_TO_SEND_PRIVATE_MESSAGES . "";
+    $errormessage = XoopsLocale::E_YOU_ARE_NOT_REGISTERED."<br />"
+        . XoopsLocale::E_REGISTER_FIRST_TO_SEND_PRIVATE_MESSAGES."";
     $xoops->redirect("user.php", 2, $errormessage);
 } else {
     $pm_handler = $xoops->getHandlerPrivateMessage();
@@ -54,7 +54,7 @@ 
             $clean_msg_id = array_map("intval", $clean_msg_id);
         }
         $size = count($clean_msg_id);
-        $msg =& $clean_msg_id;
+        $msg = & $clean_msg_id;
         for ($i = 0; $i < $size; ++$i) {
             $pm = $pm_handler->get((int)($msg[$i]));
             if ($pm->getVar('to_userid') == $xoops->user->getVar('uid')) {

htdocs/class/xoopslists.php

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
      */
     public static function getLangList()
     {
-        $lang_list = XoopsLists::getDirListAsArray(\XoopsBaseConfig::get('root-path') . '/language/');
+        $lang_list = XoopsLists::getDirListAsArray(\XoopsBaseConfig::get('root-path').'/language/');
 
         return $lang_list;
     }

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;