geekwright / XoopsCore

htdocs/pmlite.php

Spacing +12 added lines, -12 removed lines

@@ -22,7 +22,7 @@ 
  * @version         $Id$
  */
 
-include __DIR__ . '/mainfile.php';
+include __DIR__.'/mainfile.php';
 $xoops = Xoops::getInstance();
 $xoops_url = \XoopsBaseConfig::get('url');
 $xoops->events()->triggerEvent('core.pmlite.start');
@@ -35,7 +35,7 @@ 
 $msg_id = isset($_GET['msg_id']) ? (int)($_GET['msg_id']) : 0;
 
 if (empty($_GET['refresh']) && isset($_POST['op']) && $_POST['op'] !== "submit") {
-    $jump = "pmlite.php?refresh=" . time();
+    $jump = "pmlite.php?refresh=".time();
     if ($send == 1) {
         $jump .= "&send={$send}";
     } else {
@@ -48,7 +48,7 @@ 
             }
         }
     }
-    header('location: ' . $jump);
+    header('location: '.$jump);
     exit();
 }
 
@@ -64,8 +64,8 @@ 
     $tpl = new XoopsTpl();
     if ($count != 1) {
         $error_message = XoopsLocale::E_SELECTED_USER_DOES_NOT_EXIST;
-        $error_message .= "<br />" . XoopsLocale::E_GO_BACK_AND_TRY_AGAIN;
-        $error_message .= "<br />[ <a href='javascript:history.go(-1)'>" . XoopsLocale::GO_BACK . "</a> ]";
+        $error_message .= "<br />".XoopsLocale::E_GO_BACK_AND_TRY_AGAIN;
+        $error_message .= "<br />[ <a href='javascript:history.go(-1)'>".XoopsLocale::GO_BACK."</a> ]";
         $tpl->assign('error_message', $error_message);
     } else {
         if ($xoops->security()->check()) {
@@ -81,19 +81,19 @@ 
             $pm->setVar("from_userid", $xoops->user->getVar("uid"));
             if (!$pm_handler->insert($pm)) {
                 $error_message = $pm->getHtmlErrors();
-                $error_message .= "<br /><a href='javascript:history.go(-1)'>" . XoopsLocale::GO_BACK . "</a>";
+                $error_message .= "<br /><a href='javascript:history.go(-1)'>".XoopsLocale::GO_BACK."</a>";
                 $tpl->assign('error_message', $error_message);
             } else {
                 // @todo: Send notification email if user has selected this in the profile
                 $info_message = XoopsLocale::S_MESSAGED_HAS_BEEN_POSTED;
                 $info_message .= "<br />";
-                $info_message .= "<br /><a href=\"javascript:window.opener.location='" . $xoops_url . "/viewpmsg.php';window.close();\">" . XoopsLocale::CLICK_HERE_TO_VIEW_YOU_PRIVATE_MESSAGES . "</a>";
-                $info_message .= "<br /><br /><a href=\"javascript:window.close();\">" . XoopsLocale::OR_CLICK_HERE_TO_CLOSE_WINDOW . "</a>";
+                $info_message .= "<br /><a href=\"javascript:window.opener.location='".$xoops_url."/viewpmsg.php';window.close();\">".XoopsLocale::CLICK_HERE_TO_VIEW_YOU_PRIVATE_MESSAGES."</a>";
+                $info_message .= "<br /><br /><a href=\"javascript:window.close();\">".XoopsLocale::OR_CLICK_HERE_TO_CLOSE_WINDOW."</a>";
                 $tpl->assign('info_message', $info_message);
             }
         } else {
             $error_message = implode('<br />', $xoops->security()->getErrors());
-            $error_message .= "<br /><a href=\"javascript:window.close();\">" . XoopsLocale::OR_CLICK_HERE_TO_CLOSE_WINDOW . "</a>";
+            $error_message .= "<br /><a href=\"javascript:window.close();\">".XoopsLocale::OR_CLICK_HERE_TO_CLOSE_WINDOW."</a>";
             $tpl->assign('error_message', $error_message);
         }
     }
@@ -110,7 +110,7 @@ 
                 $pm_uname = XoopsUser::getUnameFromId($pm->getVar("from_userid"));
                 $message = "[quote]\n";
                 $message .= sprintf(XoopsLocale::CF_WROTE, $pm_uname);
-                $message .= "\n" . $pm->getVar("msg_text", "E") . "\n[/quote]";
+                $message .= "\n".$pm->getVar("msg_text", "E")."\n[/quote]";
             } else {
                 unset($pm);
                 $reply = 0;
@@ -123,8 +123,8 @@ 
 
         if ($reply == 1) {
             $subject = $pm->getVar('subject', 'E');
-            if (!preg_match("/^" . XoopsLocale::C_RE . "/i", $subject)) {
-                $subject = XoopsLocale::C_RE . ' ' . $subject;
+            if (!preg_match("/^".XoopsLocale::C_RE."/i", $subject)) {
+                $subject = XoopsLocale::C_RE.' '.$subject;
             }
             $form->addElement(new Xoops\Form\Label(XoopsLocale::C_TO, $pm_uname));
             $form->addElement(new Xoops\Form\Hidden('to_userid', $pm->getVar("from_userid")));

htdocs/lostpass.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
  * @version         $Id$
  */
 
-include __DIR__ . '/mainfile.php';
+include __DIR__.'/mainfile.php';
 $xoops = Xoops::getInstance();
 $xoops->events()->triggerEvent('core.lostpass.start');
 
@@ -54,7 +54,7 @@ 
         $xoopsMailer->setTemplate("lostpass2.tpl");
         $xoopsMailer->assign("SITENAME", $xoops->getConfig('sitename'));
         $xoopsMailer->assign("ADMINMAIL", $xoops->getConfig('adminmail'));
-        $xoopsMailer->assign("SITEURL", $xoops_url . "/");
+        $xoopsMailer->assign("SITEURL", $xoops_url."/");
         $xoopsMailer->assign("IP", $_SERVER['REMOTE_ADDR']);
         $xoopsMailer->assign("NEWPWD", $newpass);
         $xoopsMailer->setToUsers($userObject);
@@ -79,9 +79,9 @@ 
         $xoopsMailer->setTemplate("lostpass1.tpl");
         $xoopsMailer->assign("SITENAME", $xoops->getConfig('sitename'));
         $xoopsMailer->assign("ADMINMAIL", $xoops->getConfig('adminmail'));
-        $xoopsMailer->assign("SITEURL", $xoops_url . "/");
+        $xoopsMailer->assign("SITEURL", $xoops_url."/");
         $xoopsMailer->assign("IP", $_SERVER['REMOTE_ADDR']);
-        $xoopsMailer->assign("NEWPWD_LINK", $xoops_url . "/lostpass.php?email=" . $email . "&code=" . $areyou);
+        $xoopsMailer->assign("NEWPWD_LINK", $xoops_url."/lostpass.php?email=".$email."&code=".$areyou);
         $xoopsMailer->setToUsers($userObject);
         $xoopsMailer->setFromEmail($xoops->getConfig('adminmail'));
         $xoopsMailer->setFromName($xoops->getConfig('sitename'));

htdocs/user.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
  * @author          Kazumi Ono <webmaster@myweb.ne.jp>
  */
 
-include __DIR__ . '/mainfile.php';
+include __DIR__.'/mainfile.php';
 
 $xoops = Xoops::getInstance();
 $xoops_url = \XoopsBaseConfig::get('url');
@@ -58,12 +58,12 @@ 
         $xoops->theme()->addMeta(
             'meta',
             'keywords',
-            XoopsLocale::USERNAME . ", " . XoopsLocale::PASSWORD . ", " . XoopsLocale::Q_LOST_YOUR_PASSWORD
+            XoopsLocale::USERNAME.", ".XoopsLocale::PASSWORD.", ".XoopsLocale::Q_LOST_YOUR_PASSWORD
         );
         $xoops->theme()->addMeta(
             'meta',
             'description',
-            XoopsLocale::Q_LOST_YOUR_PASSWORD . " " . XoopsLocale::NO_PROBLEM_ENTER_EMAIL_WE_HAVE_ON_FILE
+            XoopsLocale::Q_LOST_YOUR_PASSWORD." ".XoopsLocale::NO_PROBLEM_ENTER_EMAIL_WE_HAVE_ON_FILE
         );
         $xoops->tpl()->assign('lang_login', XoopsLocale::A_LOGIN);
         $xoops->tpl()->assign('lang_username', XoopsLocale::C_USERNAME);
@@ -91,11 +91,11 @@ 
             }
         }
         if (!$isExternal) {
-            header('Location: ' . $redirect);
+            header('Location: '.$redirect);
             exit();
         }
     }
-    header('Location: ' . $xoops_url . '/userinfo.php?uid=' . $xoopsUser->getVar('uid'));
+    header('Location: '.$xoops_url.'/userinfo.php?uid='.$xoopsUser->getVar('uid'));
     exit();
 }
 
@@ -106,8 +106,8 @@ 
     if ($xoops->isUser()) {
         $xoops->getHandlerOnline()->destroy($xoops->user->getVar('uid'));
     }
-    $message = XoopsLocale::S_YOU_ARE_NOW_LOGGED_OUT . '<br />' . XoopsLocale::S_THANK_YOU_FOR_VISITING_OUR_SITE;
-    $xoops->redirect($xoops_url . '/', 1, $message);
+    $message = XoopsLocale::S_YOU_ARE_NOW_LOGGED_OUT.'<br />'.XoopsLocale::S_THANK_YOU_FOR_VISITING_OUR_SITE;
+    $xoops->redirect($xoops_url.'/', 1, $message);
 }
 
 if ($op === 'delete') {
@@ -120,12 +120,12 @@ 
             // users in the webmasters group may not be deleted
             $xoops->redirect('user.php', 5, XoopsLocale::E_USER_IN_WEBMASTER_GROUP_CANNOT_BE_REMOVED);
         }
-        if (!(bool) $ok) {
+        if (!(bool)$ok) {
             $xoops->header();
             echo $xoops->confirm(
                 array('op' => 'delete', 'ok' => 1),
                 'user.php',
-                XoopsLocale::Q_ARE_YOU_SURE_TO_DELETE_ACCOUNT . '<br/>' . XoopsLocale::THIS_WILL_REMOVE_ALL_YOUR_INFO
+                XoopsLocale::Q_ARE_YOU_SURE_TO_DELETE_ACCOUNT.'<br/>'.XoopsLocale::THIS_WILL_REMOVE_ALL_YOUR_INFO
             );
             $xoops->footer();
         } else {

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;