mambax7 / planet

admin/about.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
  * @version    $Id $
  */
 
-include_once dirname(__FILE__) . '/admin_header.php';
+include_once dirname(__FILE__).'/admin_header.php';
 
 xoops_cp_header();
 

admin/header.php

Indentation +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
 // include the default language file for the admin interface
 if(!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/" . $xoopsConfig['language'] . "/main.php")){
-    include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
+	include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
 }
 
 planet_define_url_delimiter();

Spacing +2 added lines, -2 removed lines

@@ -31,10 +31,10 @@
 include_once(XOOPS_ROOT_PATH."/Frameworks/art/functions.admin.php");
 
 // include the default language file for the admin interface
-if(!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/" . $xoopsConfig['language'] . "/main.php")){
+if (!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/".$xoopsConfig['language']."/main.php")) {
     include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
 }
 
 planet_define_url_delimiter();
-$myts =& MyTextSanitizer::getInstance();
+$myts = & MyTextSanitizer::getInstance();
 ?>
\ No newline at end of file

admin/admin.category.php

Indentation +88 added lines, -88 removed lines

@@ -42,66 +42,66 @@ 
 
 switch($op){
 	case "save":
-        if ($cat_id) {
-            $category_obj =& $category_handler->get($cat_id);
-        } else {
-            $category_obj =& $category_handler->create();
-        }
+		if ($cat_id) {
+			$category_obj =& $category_handler->get($cat_id);
+		} else {
+			$category_obj =& $category_handler->create();
+		}
 
-        $category_obj->setVar('cat_title', $_POST['cat_title']);
-        $category_obj->setVar('cat_order', $_POST['cat_order']);
+		$category_obj->setVar('cat_title', $_POST['cat_title']);
+		$category_obj->setVar('cat_order', $_POST['cat_order']);
 
-        if (!$category_handler->insert($category_obj)) {
-        	$message = planet_constant("AM_ERROR");
-        }else{
-        	$message = planet_constant("AM_DBUPDATED");
-    	}
-        redirect_header("admin.category.php", 2, $message);
-        exit();	
+		if (!$category_handler->insert($category_obj)) {
+			$message = planet_constant("AM_ERROR");
+		}else{
+			$message = planet_constant("AM_DBUPDATED");
+		}
+		redirect_header("admin.category.php", 2, $message);
+		exit();	
         
 	case "del":
 		if(!is_array($cat_id)) $cat_id = array($cat_id);
 		foreach($cat_id as $cid){
-	        $category_obj =& $category_handler->get($cid);
-	        if (!$category_handler->delete($category_obj)) {
-	        }
-       	}
-        $message = planet_constant("AM_DBUPDATED");
-        redirect_header("admin.category.php", 2, $message);
-        exit();	
+			$category_obj =& $category_handler->get($cid);
+			if (!$category_handler->delete($category_obj)) {
+			}
+	   	}
+		$message = planet_constant("AM_DBUPDATED");
+		redirect_header("admin.category.php", 2, $message);
+		exit();	
         
 	case "order":
 		$count = count($_POST['cat_order']);
 		for($i=0; $i < $count; $i++){
-        	$category_obj =& $category_handler->get($_POST['cat'][$i]);
-        	$category_obj->setVar("cat_order", $_POST['cat_order'][$i]);
-        	$category_handler->insert($category_obj, true);
-        	unset($category_obj);
+			$category_obj =& $category_handler->get($_POST['cat'][$i]);
+			$category_obj->setVar("cat_order", $_POST['cat_order'][$i]);
+			$category_handler->insert($category_obj, true);
+			unset($category_obj);
 		}
-        $message = planet_constant("AM_DBUPDATED");
-        redirect_header("admin.category.php", 2, $message);
-        exit();	
+		$message = planet_constant("AM_DBUPDATED");
+		redirect_header("admin.category.php", 2, $message);
+		exit();	
         
 	case "edit":
-        $category_obj =& $category_handler->get($cat_id);
-        $form = new XoopsThemeForm(_EDIT, "edit", xoops_getenv('PHP_SELF'));
-	    $form->addElement(new XoopsFormText(planet_constant("AM_TITLE"), 'cat_title', 50, 80, $category_obj->getVar('cat_title', 'E')), true);
-	    $form->addElement(new XoopsFormText(planet_constant("AM_ORDER"), 'cat_order', 5, 10, $category_obj->getVar('cat_order')), false);
-	    $form->addElement(new XoopsFormHidden('category', $cat_id));
-	    $form->addElement(new XoopsFormHidden('op', 'save'));
+		$category_obj =& $category_handler->get($cat_id);
+		$form = new XoopsThemeForm(_EDIT, "edit", xoops_getenv('PHP_SELF'));
+		$form->addElement(new XoopsFormText(planet_constant("AM_TITLE"), 'cat_title', 50, 80, $category_obj->getVar('cat_title', 'E')), true);
+		$form->addElement(new XoopsFormText(planet_constant("AM_ORDER"), 'cat_order', 5, 10, $category_obj->getVar('cat_order')), false);
+		$form->addElement(new XoopsFormHidden('category', $cat_id));
+		$form->addElement(new XoopsFormHidden('op', 'save'));
 	
-	    $button_tray = new XoopsFormElementTray('', '');
-	    $butt_save = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
-	    $button_tray->addElement($butt_save);
-        $butt_cancel = new XoopsFormButton('', '', _CANCEL, 'reset');
-        $button_tray->addElement($butt_cancel);
-	    $form->addElement($button_tray);
+		$button_tray = new XoopsFormElementTray('', '');
+		$butt_save = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
+		$button_tray->addElement($butt_save);
+		$butt_cancel = new XoopsFormButton('', '', _CANCEL, 'reset');
+		$button_tray->addElement($butt_cancel);
+		$form->addElement($button_tray);
         
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _EDIT . "</legend>";
-        echo"<br />";
-        $form->display();
-        echo "</fieldset>";
-        break;	
+		echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _EDIT . "</legend>";
+		echo"<br />";
+		$form->display();
+		echo "</fieldset>";
+		break;	
         
 	default:
 		$crit = new Criteria("1", 1);
@@ -115,60 +115,60 @@ 
 			}
 		}
 		
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . planet_constant("AM_LIST") . "</legend>";
-        echo "<br style=\"clear:both\" />";
+		echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . planet_constant("AM_LIST") . "</legend>";
+		echo "<br style=\"clear:both\" />";
 
-        echo "<form name='list' method='post'>";
-        echo "<table border='0' cellpadding='4' cellspacing='1' width='100%' class='outer'>";
-        echo "<tr align='center'>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_ORDER") . "</td>";
-        echo "<th align='left' class='bg3' width='80%'>" . planet_constant("AM_TITLE") . "</td>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_BLOGCOUNT") . "</td>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_ACTIONS") . "</td>";
+		echo "<form name='list' method='post'>";
+		echo "<table border='0' cellpadding='4' cellspacing='1' width='100%' class='outer'>";
+		echo "<tr align='center'>";
+		echo "<th class='bg3' width='5%'>" . planet_constant("AM_ORDER") . "</td>";
+		echo "<th align='left' class='bg3' width='80%'>" . planet_constant("AM_TITLE") . "</td>";
+		echo "<th class='bg3' width='5%'>" . planet_constant("AM_BLOGCOUNT") . "</td>";
+		echo "<th class='bg3' width='5%'>" . planet_constant("AM_ACTIONS") . "</td>";
 //        echo "<td class='bg3' width='5%'>" . _DELETE . "</td>";
 
-        echo "</tr>";
+		echo "</tr>";
         
-        $ii = 0;
+		$ii = 0;
 		foreach(array_keys($categories) as $cid){
-            echo "<tr class='odd' align='left'>";
-            echo "<td><input type='hidden' name='cat[]' value='".$cid."' />";
-            echo "<input type='text' name='cat_order[]' value='".($ii*10)."' /></td>";
-            echo "<td>".$categories[$cid]."</td>";
-            echo "<td align='center'>".@$blog_counts[$cid]."</td>";
+			echo "<tr class='odd' align='left'>";
+			echo "<td><input type='hidden' name='cat[]' value='".$cid."' />";
+			echo "<input type='text' name='cat_order[]' value='".($ii*10)."' /></td>";
+			echo "<td>".$categories[$cid]."</td>";
+			echo "<td align='center'>".@$blog_counts[$cid]."</td>";
 
-        echo "<td align='center'><a href='admin.category.php?op=edit &amp;category='".$cid."' title='"._EDIT."'><img src='".  $pathIcon16 ."/edit.png '". "alt='" . _EDIT . "' title='" ._EDIT."' </a>&nbsp;
+		echo "<td align='center'><a href='admin.category.php?op=edit &amp;category='".$cid."' title='"._EDIT."'><img src='".  $pathIcon16 ."/edit.png '". "alt='" . _EDIT . "' title='" ._EDIT."' </a>&nbsp;
                   <a href='admin.category.php?op=del &amp;category='".$cid."' title='"._DELETE."'><img src='". $pathIcon16 ."/delete.png '"." alt='" . _EDIT . "' title='"._DELETE."' </a></td>";
-            echo "</tr>";			
-            $ii++;
+			echo "</tr>";			
+			$ii++;
 		}
-        echo "<tr class='even' align='center'>";
-        echo "<td colspan='5'>";
-        echo "<input name='submit' value='"._SUBMIT."' type='submit' />";
-        echo "<input name='' value='"._CANCEL."' type='reset' />";
-        echo "<input name='op' value='order' type='hidden' />";
-        echo "</td>";
-        echo "</tr>";			
-        echo "</table></form>";
-        echo "</fieldset><br style='clear:both;'>";
+		echo "<tr class='even' align='center'>";
+		echo "<td colspan='5'>";
+		echo "<input name='submit' value='"._SUBMIT."' type='submit' />";
+		echo "<input name='' value='"._CANCEL."' type='reset' />";
+		echo "<input name='op' value='order' type='hidden' />";
+		echo "</td>";
+		echo "</tr>";			
+		echo "</table></form>";
+		echo "</fieldset><br style='clear:both;'>";
         
-        $form = new XoopsThemeForm(_ADD, "mod", xoops_getenv('PHP_SELF'));
-	    $form->addElement(new XoopsFormText(planet_constant("AM_TITLE"), 'cat_title', 50, 80), true);
-	    $form->addElement(new XoopsFormText(planet_constant("AM_ORDER"), 'cat_order', 5, 10), false);
-	    $form->addElement(new XoopsFormHidden('op', 'save'));
+		$form = new XoopsThemeForm(_ADD, "mod", xoops_getenv('PHP_SELF'));
+		$form->addElement(new XoopsFormText(planet_constant("AM_TITLE"), 'cat_title', 50, 80), true);
+		$form->addElement(new XoopsFormText(planet_constant("AM_ORDER"), 'cat_order', 5, 10), false);
+		$form->addElement(new XoopsFormHidden('op', 'save'));
 	
-	    $button_tray = new XoopsFormElementTray('', '');
-	    $butt_save = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
-	    $button_tray->addElement($butt_save);
-        $butt_cancel = new XoopsFormButton('', '', _CANCEL, 'reset');
-        $button_tray->addElement($butt_cancel);
-	    $form->addElement($button_tray);
+		$button_tray = new XoopsFormElementTray('', '');
+		$butt_save = new XoopsFormButton('', 'submit', _SUBMIT, 'submit');
+		$button_tray->addElement($butt_save);
+		$butt_cancel = new XoopsFormButton('', '', _CANCEL, 'reset');
+		$button_tray->addElement($butt_cancel);
+		$form->addElement($button_tray);
         
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _ADD . "</legend>";
-        echo"<br />";
-        $form->display();
-        echo "</fieldset>";
-        break;		
+		echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _ADD . "</legend>";
+		echo"<br />";
+		$form->display();
+		echo "</fieldset>";
+		break;		
 }
 
 xoops_cp_footer();

Spacing +30 added lines, -30 removed lines

@@ -24,8 +24,8 @@ 
 // URL: http://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include( "admin_header.php" );
-require_once(XOOPS_ROOT_PATH . "/class/xoopsformloader.php");
+include("admin_header.php");
+require_once(XOOPS_ROOT_PATH."/class/xoopsformloader.php");
 
 xoops_cp_header();
 $indexAdmin = new ModuleAdmin();
@@ -33,19 +33,19 @@ 
 require XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
 //planet_adminmenu(1);
 
-$op = !empty($_POST["op"])?$_POST["op"]:(!empty($_GET["op"])?$_GET["op"]:"");
-$cat_id = !empty($_POST["category"])?$_POST["category"]:(!empty($_GET["category"])?$_GET["category"]:0);
-$cat_id = is_array($cat_id)?array_map("intval", $cat_id):intval($cat_id);
+$op = !empty($_POST["op"]) ? $_POST["op"] : (!empty($_GET["op"]) ? $_GET["op"] : "");
+$cat_id = !empty($_POST["category"]) ? $_POST["category"] : (!empty($_GET["category"]) ? $_GET["category"] : 0);
+$cat_id = is_array($cat_id) ? array_map("intval", $cat_id) : intval($cat_id);
 
-$category_handler =& xoops_getmodulehandler("category", $GLOBALS["moddirname"]);
-$blog_handler =& xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
+$category_handler = & xoops_getmodulehandler("category", $GLOBALS["moddirname"]);
+$blog_handler = & xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
 
-switch($op){
+switch ($op) {
 	case "save":
         if ($cat_id) {
-            $category_obj =& $category_handler->get($cat_id);
-        } else {
-            $category_obj =& $category_handler->create();
+            $category_obj = & $category_handler->get($cat_id);
+        }else {
+            $category_obj = & $category_handler->create();
         }
 
         $category_obj->setVar('cat_title', $_POST['cat_title']);
@@ -53,16 +53,16 @@ 
 
         if (!$category_handler->insert($category_obj)) {
         	$message = planet_constant("AM_ERROR");
-        }else{
+        }else {
         	$message = planet_constant("AM_DBUPDATED");
     	}
         redirect_header("admin.category.php", 2, $message);
         exit();	
         
 	case "del":
-		if(!is_array($cat_id)) $cat_id = array($cat_id);
-		foreach($cat_id as $cid){
-	        $category_obj =& $category_handler->get($cid);
+		if (!is_array($cat_id)) $cat_id = array($cat_id);
+		foreach ($cat_id as $cid) {
+	        $category_obj = & $category_handler->get($cid);
 	        if (!$category_handler->delete($category_obj)) {
 	        }
        	}
@@ -72,8 +72,8 @@ 
         
 	case "order":
 		$count = count($_POST['cat_order']);
-		for($i=0; $i < $count; $i++){
-        	$category_obj =& $category_handler->get($_POST['cat'][$i]);
+		for ($i = 0; $i<$count; $i++) {
+        	$category_obj = & $category_handler->get($_POST['cat'][$i]);
         	$category_obj->setVar("cat_order", $_POST['cat_order'][$i]);
         	$category_handler->insert($category_obj, true);
         	unset($category_obj);
@@ -83,7 +83,7 @@ 
         exit();	
         
 	case "edit":
-        $category_obj =& $category_handler->get($cat_id);
+        $category_obj = & $category_handler->get($cat_id);
         $form = new XoopsThemeForm(_EDIT, "edit", xoops_getenv('PHP_SELF'));
 	    $form->addElement(new XoopsFormText(planet_constant("AM_TITLE"), 'cat_title', 50, 80, $category_obj->getVar('cat_title', 'E')), true);
 	    $form->addElement(new XoopsFormText(planet_constant("AM_ORDER"), 'cat_order', 5, 10, $category_obj->getVar('cat_order')), false);
@@ -97,7 +97,7 @@ 
         $button_tray->addElement($butt_cancel);
 	    $form->addElement($button_tray);
         
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _EDIT . "</legend>";
+        echo "<fieldset><legend style='font-weight: bold; color: #900;'>"._EDIT."</legend>";
         echo"<br />";
         $form->display();
         echo "</fieldset>";
@@ -109,36 +109,36 @@ 
 		$crit->setOrder("ASC");
 		$categories = $category_handler->getList($crit);
 		$blog_counts = $blog_handler->getCountsByCategory();
-		foreach(array_keys($categories) as $cid){
-			if(!empty($blog_counts[$cid])){
+		foreach (array_keys($categories) as $cid) {
+			if (!empty($blog_counts[$cid])) {
 				$categories[$cid] .= " (".intval($blog_counts[$cid]).")";
 			}
 		}
 		
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . planet_constant("AM_LIST") . "</legend>";
+        echo "<fieldset><legend style='font-weight: bold; color: #900;'>".planet_constant("AM_LIST")."</legend>";
         echo "<br style=\"clear:both\" />";
 
         echo "<form name='list' method='post'>";
         echo "<table border='0' cellpadding='4' cellspacing='1' width='100%' class='outer'>";
         echo "<tr align='center'>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_ORDER") . "</td>";
-        echo "<th align='left' class='bg3' width='80%'>" . planet_constant("AM_TITLE") . "</td>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_BLOGCOUNT") . "</td>";
-        echo "<th class='bg3' width='5%'>" . planet_constant("AM_ACTIONS") . "</td>";
+        echo "<th class='bg3' width='5%'>".planet_constant("AM_ORDER")."</td>";
+        echo "<th align='left' class='bg3' width='80%'>".planet_constant("AM_TITLE")."</td>";
+        echo "<th class='bg3' width='5%'>".planet_constant("AM_BLOGCOUNT")."</td>";
+        echo "<th class='bg3' width='5%'>".planet_constant("AM_ACTIONS")."</td>";
 //        echo "<td class='bg3' width='5%'>" . _DELETE . "</td>";
 
         echo "</tr>";
         
         $ii = 0;
-		foreach(array_keys($categories) as $cid){
+		foreach (array_keys($categories) as $cid) {
             echo "<tr class='odd' align='left'>";
             echo "<td><input type='hidden' name='cat[]' value='".$cid."' />";
             echo "<input type='text' name='cat_order[]' value='".($ii*10)."' /></td>";
             echo "<td>".$categories[$cid]."</td>";
             echo "<td align='center'>".@$blog_counts[$cid]."</td>";
 
-        echo "<td align='center'><a href='admin.category.php?op=edit &amp;category='".$cid."' title='"._EDIT."'><img src='".  $pathIcon16 ."/edit.png '". "alt='" . _EDIT . "' title='" ._EDIT."' </a>&nbsp;
-                  <a href='admin.category.php?op=del &amp;category='".$cid."' title='"._DELETE."'><img src='". $pathIcon16 ."/delete.png '"." alt='" . _EDIT . "' title='"._DELETE."' </a></td>";
+        echo "<td align='center'><a href='admin.category.php?op=edit &amp;category='".$cid."' title='"._EDIT."'><img src='".$pathIcon16."/edit.png '"."alt='"._EDIT."' title='"._EDIT."' </a>&nbsp;
+                  <a href='admin.category.php?op=del &amp;category='".$cid."' title='"._DELETE."'><img src='".$pathIcon16."/delete.png '"." alt='"._EDIT."' title='"._DELETE."' </a></td>";
             echo "</tr>";			
             $ii++;
 		}
@@ -164,7 +164,7 @@ 
         $button_tray->addElement($butt_cancel);
 	    $form->addElement($button_tray);
         
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _ADD . "</legend>";
+        echo "<fieldset><legend style='font-weight: bold; color: #900;'>"._ADD."</legend>";
         echo"<br />";
         $form->display();
         echo "</fieldset>";

Braces +4 added lines, -2 removed lines

@@ -53,14 +53,16 @@
 
         if (!$category_handler->insert($category_obj)) {
         	$message = planet_constant("AM_ERROR");
-        }else{
+        } else{
         	$message = planet_constant("AM_DBUPDATED");
     	}
         redirect_header("admin.category.php", 2, $message);
         exit();	
         
 	case "del":
-		if(!is_array($cat_id)) $cat_id = array($cat_id);
+		if(!is_array($cat_id)) {
+			$cat_id = array($cat_id);
+		}
 		foreach($cat_id as $cid){
 	        $category_obj =& $category_handler->get($cid);
 	        if (!$category_handler->delete($category_obj)) {

admin/admin_header.php

Indentation +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
 // include the default language file for the admin interface
 if(!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/" . $xoopsConfig['language'] . "/main.php")){
-    include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
+	include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
 }
 
 planet_define_url_delimiter();

Spacing +5 added lines, -5 removed lines

@@ -19,9 +19,9 @@ 
  */
 
 $path = dirname(dirname(dirname(dirname(__FILE__))));
-include_once $path . '/mainfile.php';
-include_once $path . '/include/cp_functions.php';
-require_once $path . '/include/cp_header.php';
+include_once $path.'/mainfile.php';
+include_once $path.'/include/cp_functions.php';
+require_once $path.'/include/cp_header.php';
 
 global $xoopsModule;
 
@@ -47,9 +47,9 @@ 
 include_once(XOOPS_ROOT_PATH."/Frameworks/art/functions.admin.php");
 
 // include the default language file for the admin interface
-if(!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/" . $xoopsConfig['language'] . "/main.php")){
+if (!@include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/".$xoopsConfig['language']."/main.php")) {
     include_once(XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/language/english/main.php");
 }
 
 planet_define_url_delimiter();
-$myts =& MyTextSanitizer::getInstance();
+$myts = & MyTextSanitizer::getInstance();

admin/admin.article.php

Spacing +11 added lines, -11 removed lines

@@ -24,7 +24,7 @@ 
 // URL: http://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include( "admin_header.php" );
+include("admin_header.php");
 
 xoops_cp_header();
 $indexAdmin = new ModuleAdmin();
@@ -39,29 +39,29 @@ 
 require XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
 //planet_adminmenu(3);
 
-$article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
-if(!empty($xoopsModuleConfig["article_expire"])){
+$article_handler = & xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
+if (!empty($xoopsModuleConfig["article_expire"])) {
 	$criteria = new Criteria("art_time", time()-$xoopsModuleConfig["article_expire"]*60*60*24, "<");
-	if(!empty($_GET["purge"])){
+	if (!empty($_GET["purge"])) {
 		$crit = new CriteriaCompo($criteria);
 		$crit->add(new Criteria("art_comments", 0));
-		$article_expires =& $article_handler->getObjects($criteria);
-		foreach($article_expires as $id=>$article_obj){
+		$article_expires = & $article_handler->getObjects($criteria);
+		foreach ($article_expires as $id=>$article_obj) {
 			$article_handler->delete($article_obj);
 		}
 	}
 	$article_count_expire = $article_handler->getCount($criteria);
-}else{
+}else {
 	$article_count_expire = 0;
 }
 $article_count = $article_handler->getCount();
 
-echo "<fieldset><legend style=\"font-weight: bold; color: #900;\">" . planet_constant("AM_ARTICLES") . "</legend>";
+echo "<fieldset><legend style=\"font-weight: bold; color: #900;\">".planet_constant("AM_ARTICLES")."</legend>";
 echo "<div style=\"padding: 8px;\">";
-echo "<br clear=\"all\" />" . planet_constant("AM_COUNT").": ". $article_count;
+echo "<br clear=\"all\" />".planet_constant("AM_COUNT").": ".$article_count;
 echo "<br clear=\"all\" />";
-if($article_count_expire>0){
-	echo "<br clear=\"all\" /><a href=\"".XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/admin/admin.article.php?purge=1\" >" . planet_constant("AM_EXPIRED").": ".$article_count_expire."</a>";
+if ($article_count_expire>0) {
+	echo "<br clear=\"all\" /><a href=\"".XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/admin/admin.article.php?purge=1\" >".planet_constant("AM_EXPIRED").": ".$article_count_expire."</a>";
 	echo "<br clear=\"all\" />";
 }
 echo "</div>";

Braces +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 		}
 	}
 	$article_count_expire = $article_handler->getCount($criteria);
-}else{
+} else{
 	$article_count_expire = 0;
 }
 $article_count = $article_handler->getCount();

action.rate.php

Indentation +4 added lines, -4 removed lines

@@ -32,8 +32,8 @@ 
 $blog_id = intval( !empty($_POST["blog"])?$_POST["blog"]:(!empty($_GET["blog"])?$_GET["blog"]:0) );
 
 if(empty($article_id) && empty($blog_id)){
-    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
-    exit();
+	redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
+	exit();
 }
 
 $article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
@@ -69,8 +69,8 @@ 
 		$rate_obj->setVar("rate_rating", $rate);
 		$rate_obj->setVar("rate_time", time());
 		if(!$rate_id = $rate_handler->insert($rate_obj, true)){
-		    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_NOTSAVED"));
-		    exit();
+			redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_NOTSAVED"));
+			exit();
 		}
 		if($article_id>0){
 			$article_obj =& $article_handler->get($article_id);

Spacing +26 added lines, -26 removed lines

@@ -27,58 +27,58 @@ 
 
 include 'header.php';
 
-$rate = intval( !empty($_POST["rate"])?$_POST["rate"]:(!empty($_GET["rate"])?$_GET["rate"]:0) );
-$article_id = intval( !empty($_POST["article"])?$_POST["article"]:(!empty($_GET["article"])?$_GET["article"]:0) );
-$blog_id = intval( !empty($_POST["blog"])?$_POST["blog"]:(!empty($_GET["blog"])?$_GET["blog"]:0) );
+$rate = intval(!empty($_POST["rate"]) ? $_POST["rate"] : (!empty($_GET["rate"]) ? $_GET["rate"] : 0));
+$article_id = intval(!empty($_POST["article"]) ? $_POST["article"] : (!empty($_GET["article"]) ? $_GET["article"] : 0));
+$blog_id = intval(!empty($_POST["blog"]) ? $_POST["blog"] : (!empty($_GET["blog"]) ? $_GET["blog"] : 0));
 
-if(empty($article_id) && empty($blog_id)){
+if (empty($article_id) && empty($blog_id)) {
     redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
     exit();
 }
 
-$article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
-$blog_handler =& xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
-if(empty($xoopsModuleConfig["anonymous_rate"]) && !is_object($xoopsUser)){
+$article_handler = & xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
+$blog_handler = & xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
+if (empty($xoopsModuleConfig["anonymous_rate"]) && !is_object($xoopsUser)) {
 	$message = planet_constant("MD_NOACCESS");
-}else{
-	$uid = (is_object($xoopsUser))?$xoopsUser->getVar("uid"):0;
+}else {
+	$uid = (is_object($xoopsUser)) ? $xoopsUser->getVar("uid") : 0;
 	$ip = planet_getIP();
-	if($article_id>0){
+	if ($article_id>0) {
 		$criteria = new CriteriaCompo(new Criteria("art_id", $article_id));
-	}else{
+	}else {
 		$criteria = new CriteriaCompo(new Criteria("blog_id", $blog_id));
 	}
-	if($uid>0){
+	if ($uid>0) {
 		$criteria->add(new Criteria("rate_uid", $uid));
-	}else{
+	}else {
 		$criteria->add(new Criteria("rate_ip", $ip));
 		$criteria->add(new Criteria("rate_time", time()-24*3600, ">"));
 	}
-	$rate_handler =& xoops_getmodulehandler("rate", $GLOBALS["moddirname"]);
-	if($count=$rate_handler->getCount($criteria)){
+	$rate_handler = & xoops_getmodulehandler("rate", $GLOBALS["moddirname"]);
+	if ($count = $rate_handler->getCount($criteria)) {
 		$message = planet_constant("MD_ALREADYRATED");
-	}else{
-		$rate_obj =& $rate_handler->create();
-		if($article_id>0){
+	}else {
+		$rate_obj = & $rate_handler->create();
+		if ($article_id>0) {
 			$rate_obj->setVar("art_id", $article_id);
-		}else{
+		}else {
 			$rate_obj->setVar("blog_id", $blog_id);
 		}
 		$rate_obj->setVar("rate_uid", $uid);
 		$rate_obj->setVar("rate_ip", $ip);
 		$rate_obj->setVar("rate_rating", $rate);
 		$rate_obj->setVar("rate_time", time());
-		if(!$rate_id = $rate_handler->insert($rate_obj, true)){
+		if (!$rate_id = $rate_handler->insert($rate_obj, true)) {
 		    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_NOTSAVED"));
 		    exit();
 		}
-		if($article_id>0){
-			$article_obj =& $article_handler->get($article_id);
+		if ($article_id>0) {
+			$article_obj = & $article_handler->get($article_id);
 			$article_obj->setVar("art_rating", $article_obj->getVar("art_rating")+$rate);
 			$article_obj->setVar("art_rates", $article_obj->getVar("art_rates")+1);
 			$article_handler->insert($article_obj, true);
-		}else{
-			$blog_obj =& $blog_handler->get($blog_id);
+		}else {
+			$blog_obj = & $blog_handler->get($blog_id);
 			$blog_obj->setVar("blog_rating", $blog_obj->getVar("blog_rating")+$rate);
 			$blog_obj->setVar("blog_rates", $blog_obj->getVar("blog_rates")+1);
 			$blog_handler->insert($blog_obj, true);
@@ -86,9 +86,9 @@ 
 		$message = planet_constant("MD_ACTIONDONE");
 	}
 }
-if($article_id>0){
+if ($article_id>0) {
 	redirect_header(XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_id, 2, $message);
-}else{
+}else {
 	redirect_header(XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php".URL_DELIMITER."b".$blog_id, 2, $message);
 }
 include 'footer.php';

Braces +7 added lines, -7 removed lines

@@ -40,28 +40,28 @@ 
 $blog_handler =& xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
 if(empty($xoopsModuleConfig["anonymous_rate"]) && !is_object($xoopsUser)){
 	$message = planet_constant("MD_NOACCESS");
-}else{
+} else{
 	$uid = (is_object($xoopsUser))?$xoopsUser->getVar("uid"):0;
 	$ip = planet_getIP();
 	if($article_id>0){
 		$criteria = new CriteriaCompo(new Criteria("art_id", $article_id));
-	}else{
+	} else{
 		$criteria = new CriteriaCompo(new Criteria("blog_id", $blog_id));
 	}
 	if($uid>0){
 		$criteria->add(new Criteria("rate_uid", $uid));
-	}else{
+	} else{
 		$criteria->add(new Criteria("rate_ip", $ip));
 		$criteria->add(new Criteria("rate_time", time()-24*3600, ">"));
 	}
 	$rate_handler =& xoops_getmodulehandler("rate", $GLOBALS["moddirname"]);
 	if($count=$rate_handler->getCount($criteria)){
 		$message = planet_constant("MD_ALREADYRATED");
-	}else{
+	} else{
 		$rate_obj =& $rate_handler->create();
 		if($article_id>0){
 			$rate_obj->setVar("art_id", $article_id);
-		}else{
+		} else{
 			$rate_obj->setVar("blog_id", $blog_id);
 		}
 		$rate_obj->setVar("rate_uid", $uid);
@@ -77,7 +77,7 @@ 
 			$article_obj->setVar("art_rating", $article_obj->getVar("art_rating")+$rate);
 			$article_obj->setVar("art_rates", $article_obj->getVar("art_rates")+1);
 			$article_handler->insert($article_obj, true);
-		}else{
+		} else{
 			$blog_obj =& $blog_handler->get($blog_id);
 			$blog_obj->setVar("blog_rating", $blog_obj->getVar("blog_rating")+$rate);
 			$blog_obj->setVar("blog_rates", $blog_obj->getVar("blog_rates")+1);
@@ -88,7 +88,7 @@ 
 }
 if($article_id>0){
 	redirect_header(XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_id, 2, $message);
-}else{
+} else{
 	redirect_header(XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php".URL_DELIMITER."b".$blog_id, 2, $message);
 }
 include 'footer.php';

action.article.php

Indentation +16 added lines, -16 removed lines

@@ -29,12 +29,12 @@ 
 
 $art_id = intval( isset($_GET["article"]) ? $_GET["article"] : (isset($_POST["article"]) ? $_POST["article"]: 0) );
 if(empty($art_id)){
-    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
-    exit();
+	redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
+	exit();
 }
 if (!$xoopsUser->isAdmin()){
-    redirect_header("javascript:history.go(-1);", 2, _NOPERM);
-    exit();
+	redirect_header("javascript:history.go(-1);", 2, _NOPERM);
+	exit();
 }
 include XOOPS_ROOT_PATH."/header.php";
 include XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
@@ -52,28 +52,28 @@ 
 }elseif ( $op == "save" ) {
 	
 	if(empty($_POST["art_content"])){
-	    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_TEXTEMPTY"));
-	    exit();
+		redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_TEXTEMPTY"));
+		exit();
 	}
 
 	foreach(array(
-	    "art_title",
-	    "art_link",
-	    "art_author",
-	    "art_content"
+		"art_title",
+		"art_link",
+		"art_author",
+		"art_content"
 		) as $tag){
 		if(@$_POST[$tag] != $article_obj->getVar($tag))
-    	$article_obj->setVar($tag, @$_POST[$tag]);
+		$article_obj->setVar($tag, @$_POST[$tag]);
 	}
 
-    $art_id_new = $article_handler->insert($article_obj);
+	$art_id_new = $article_handler->insert($article_obj);
 	if(!$article_obj->getVar("art_id")){
 		$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php";
-	    $message = planet_constant("MD_INSERTERROR");
+		$message = planet_constant("MD_INSERTERROR");
 	}else{
-	    $redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_obj->getVar("art_id");
-	    $message = planet_constant("MD_SAVED");
-    }
+		$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_obj->getVar("art_id");
+		$message = planet_constant("MD_SAVED");
+	}
 	redirect_header($redirect,2,$message);
 }else{
 	require_once(XOOPS_ROOT_PATH . "/class/xoopsformloader.php");

Spacing +16 added lines, -16 removed lines

@@ -27,56 +27,56 @@
 
 include "header.php";
 
-$art_id = intval( isset($_GET["article"]) ? $_GET["article"] : (isset($_POST["article"]) ? $_POST["article"]: 0) );
-if(empty($art_id)){
+$art_id = intval(isset($_GET["article"]) ? $_GET["article"] : (isset($_POST["article"]) ? $_POST["article"] : 0));
+if (empty($art_id)) {
     redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_INVALID"));
     exit();
 }
-if (!$xoopsUser->isAdmin()){
+if (!$xoopsUser->isAdmin()) {
     redirect_header("javascript:history.go(-1);", 2, _NOPERM);
     exit();
 }
 include XOOPS_ROOT_PATH."/header.php";
 include XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
 
-$article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
+$article_handler = & xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
 $article_obj = $article_handler->get($art_id);
 
 $op = isset($_POST["op"]) ? $_POST["op"] : "";
 
-if ( $op == "del" || !empty($_POST["del"])) {
+if ($op == "del" || !empty($_POST["del"])) {
 	$article_handler->delete($article_obj);
 	$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php";
 	$message = planet_constant("MD_SAVED");
-	redirect_header($redirect,2,$message);
-}elseif ( $op == "save" ) {
+	redirect_header($redirect, 2, $message);
+}elseif ($op == "save") {
 	
-	if(empty($_POST["art_content"])){
+	if (empty($_POST["art_content"])) {
 	    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_TEXTEMPTY"));
 	    exit();
 	}
 
-	foreach(array(
+	foreach (array(
 	    "art_title",
 	    "art_link",
 	    "art_author",
 	    "art_content"
-		) as $tag){
-		if(@$_POST[$tag] != $article_obj->getVar($tag))
+		) as $tag) {
+		if (@$_POST[$tag] != $article_obj->getVar($tag))
     	$article_obj->setVar($tag, @$_POST[$tag]);
 	}
 
     $art_id_new = $article_handler->insert($article_obj);
-	if(!$article_obj->getVar("art_id")){
+	if (!$article_obj->getVar("art_id")) {
 		$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php";
 	    $message = planet_constant("MD_INSERTERROR");
-	}else{
+	}else {
 	    $redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_obj->getVar("art_id");
 	    $message = planet_constant("MD_SAVED");
     }
-	redirect_header($redirect,2,$message);
-}else{
-	require_once(XOOPS_ROOT_PATH . "/class/xoopsformloader.php");
+	redirect_header($redirect, 2, $message);
+}else {
+	require_once(XOOPS_ROOT_PATH."/class/xoopsformloader.php");
 	
 	$form = new XoopsThemeForm(_EDIT, "formarticle", xoops_getenv('PHP_SELF'));
 	

Braces +6 added lines, -5 removed lines

@@ -49,7 +49,7 @@ 
 	$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php";
 	$message = planet_constant("MD_SAVED");
 	redirect_header($redirect,2,$message);
-}elseif ( $op == "save" ) {
+} elseif ( $op == "save" ) {
 	
 	if(empty($_POST["art_content"])){
 	    redirect_header("javascript:history.go(-1);", 1, planet_constant("MD_TEXTEMPTY"));
@@ -62,20 +62,21 @@ 
 	    "art_author",
 	    "art_content"
 		) as $tag){
-		if(@$_POST[$tag] != $article_obj->getVar($tag))
-    	$article_obj->setVar($tag, @$_POST[$tag]);
+		if(@$_POST[$tag] != $article_obj->getVar($tag)) {
+		    	$article_obj->setVar($tag, @$_POST[$tag]);
+		}
 	}
 
     $art_id_new = $article_handler->insert($article_obj);
 	if(!$article_obj->getVar("art_id")){
 		$redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/index.php";
 	    $message = planet_constant("MD_INSERTERROR");
-	}else{
+	} else{
 	    $redirect = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_obj->getVar("art_id");
 	    $message = planet_constant("MD_SAVED");
     }
 	redirect_header($redirect,2,$message);
-}else{
+} else{
 	require_once(XOOPS_ROOT_PATH . "/class/xoopsformloader.php");
 	
 	$form = new XoopsThemeForm(_EDIT, "formarticle", xoops_getenv('PHP_SELF'));

view.archive.php

Indentation +69 added lines, -69 removed lines

@@ -60,9 +60,9 @@ 
 }
 $time = array("year"=>$year, "month"=>$month, "day"=>$day);
 if ($xoopsUser) {
-    $timeoffset = ($xoopsUser->getVar("timezone_offset")- $xoopsConfig['server_TZ'])*3600;
+	$timeoffset = ($xoopsUser->getVar("timezone_offset")- $xoopsConfig['server_TZ'])*3600;
 } else {
-    $timeoffset = ($xoopsConfig['default_TZ']- $xoopsConfig['server_TZ'])*3600;
+	$timeoffset = ($xoopsConfig['default_TZ']- $xoopsConfig['server_TZ'])*3600;
 }
 
 $criteria = new CriteriaCompo();
@@ -127,20 +127,20 @@ 
 	}
 	// Get monthly list
 	if(empty($month)){
-    	$sql = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count 
+		$sql = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count 
             FROM ".planet_DB_prefix("article")."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             ".$blog_criteria."
             GROUP BY mon
             ";
-        $result = $xoopsDB->query($sql);
-        $months = array();
-        while ($myrow = $xoopsDB->fetchArray($result)) {
-            $months[] = array(
-            	"title"=>planet_constant("MD_MONTH_".intval($myrow["mon"]))." (".intval($myrow["count"]).")",
+		$result = $xoopsDB->query($sql);
+		$months = array();
+		while ($myrow = $xoopsDB->fetchArray($result)) {
+			$months[] = array(
+				"title"=>planet_constant("MD_MONTH_".intval($myrow["mon"]))." (".intval($myrow["count"]).")",
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$year."/".$myrow["mon"]."/b".$blog_id,
-            	);
-        }
+				);
+		}
 		$timenav["prev"] = array(
 			"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".($year-1)."/b".$blog_id,
 			"title"=>sprintf(planet_constant("MD_TIME_Y"), ($year-1))
@@ -154,37 +154,37 @@ 
 	}
 	// Get daily list
 	elseif(empty($day)){
-    	$sql = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count 
+		$sql = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count 
             FROM ".planet_DB_prefix("article")."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             AND MONTH(FROM_UNIXTIME(art_time - $timeoffset)) = $month
             ".$blog_criteria."
             GROUP BY day
             ";
-        $result = $xoopsDB->query($sql);
-        $days = array();
-        while ($myrow = $xoopsDB->fetchArray($result)) {
-            $days[$myrow["day"]]["count"] = $myrow["count"];
-        }
-        for($i=1;$i<=31;$i++){
-	        if(!isset($days[$i])) continue;
-	        $days[$i] = array(
-            	"title"=>$days[$i]["count"],
+		$result = $xoopsDB->query($sql);
+		$days = array();
+		while ($myrow = $xoopsDB->fetchArray($result)) {
+			$days[$myrow["day"]]["count"] = $myrow["count"];
+		}
+		for($i=1;$i<=31;$i++){
+			if(!isset($days[$i])) continue;
+			$days[$i] = array(
+				"title"=>$days[$i]["count"],
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$year."/".$month."/".$i."/b".$blog_id
-            	);
-        }
-        $calendar = planet_getCalendar($year, $month, $days);
-        $month_next = $month+1;
-        $month_prev = $month-1;
-        $_year = $year;
-        if($month==12) {
-	        $month_next = 1;
-	        $_year = $year +1;
-        }
-        if($month==1) {
-	        $month_pre = 12;
-	        $_year = $year - 1;
-        }
+				);
+		}
+		$calendar = planet_getCalendar($year, $month, $days);
+		$month_next = $month+1;
+		$month_prev = $month-1;
+		$_year = $year;
+		if($month==12) {
+			$month_next = 1;
+			$_year = $year +1;
+		}
+		if($month==1) {
+			$month_pre = 12;
+			$_year = $year - 1;
+		}
 		$timenav["prev"] = array(
 			"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$_year."/".$month_prev."/b".$blog_id,
 			"title"=>planet_constant("MD_MONTH_".$month_prev)
@@ -218,59 +218,59 @@ 
 {
 	$year = empty($year)?date("Y"):$year;
 	$month = empty($month)?date("n"):$month;
-    $unixmonth = mktime(0, 0 , 0, $month, 1, $year);
+	$unixmonth = mktime(0, 0 , 0, $month, 1, $year);
 	
-    ob_start();
-    echo '<table id="calendar">';
-    echo '<caption>';
-    printf(planet_constant("MD_TIME_YM"), $year, planet_constant("MD_MONTH_".$month));
-    echo '</caption>';
+	ob_start();
+	echo '<table id="calendar">';
+	echo '<caption>';
+	printf(planet_constant("MD_TIME_YM"), $year, planet_constant("MD_MONTH_".$month));
+	echo '</caption>';
 	
-    for ($i=1;$i<=7;$i++) {
-        echo "\n\t\t<th abbr=\"".planet_constant("MD_WEEK_".$i)."\" scope=\"col\" title=\"".planet_constant("MD_WEEK_".$i)."\">" . planet_constant("MD_WEEK_".$i) . '</th>';
-    }
+	for ($i=1;$i<=7;$i++) {
+		echo "\n\t\t<th abbr=\"".planet_constant("MD_WEEK_".$i)."\" scope=\"col\" title=\"".planet_constant("MD_WEEK_".$i)."\">" . planet_constant("MD_WEEK_".$i) . '</th>';
+	}
 
 	echo '<tr>';
 	
 	// See how much we should pad in the beginning
 	$week_begins = 1;
-    $pad = planet_calendar_week_mod(date('w', $unixmonth)-$week_begins);
-    if (0 != $pad) echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
+	$pad = planet_calendar_week_mod(date('w', $unixmonth)-$week_begins);
+	if (0 != $pad) echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
 
-    $daysinmonth = intval(date('t', $unixmonth));
-    for ($day = 1; $day <= $daysinmonth; ++$day) {
-        if (isset($newrow) && $newrow){
-            echo "\n\t</tr>\n\t<tr>\n\t\t";
-        }
-        $newrow = false;
+	$daysinmonth = intval(date('t', $unixmonth));
+	for ($day = 1; $day <= $daysinmonth; ++$day) {
+		if (isset($newrow) && $newrow){
+			echo "\n\t</tr>\n\t<tr>\n\t\t";
+		}
+		$newrow = false;
 
-       	echo '<td>';
+	   	echo '<td>';
 
-        if (!empty($days[$day]["url"])) {
+		if (!empty($days[$day]["url"])) {
 			echo '<a href="' . $days[$day]["url"] . "\"";
 			if(!empty($days[$day]["title"])) echo "title=\"".$days[$day]["title"]."\"";
 			echo ">$day</a>";
-        } elseif(!empty($days[$day]["title"])) {
-            echo "<acronym title=\"".$days[$day]["title"]."\">$day</acronym>";
-        } else {
-            echo $day;
-        }
-        echo '</td>';
+		} elseif(!empty($days[$day]["title"])) {
+			echo "<acronym title=\"".$days[$day]["title"]."\">$day</acronym>";
+		} else {
+			echo $day;
+		}
+		echo '</td>';
 
 	if (6 == planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins))
-            $newrow = true;
-    }
+			$newrow = true;
+	}
 
-    $pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins);
-    if ($pad != 0 && $pad != 7){
-        echo "\n\t\t".'<td class="pad" colspan="'.$pad.'">&nbsp;</td>';
-    }
+	$pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins);
+	if ($pad != 0 && $pad != 7){
+		echo "\n\t\t".'<td class="pad" colspan="'.$pad.'">&nbsp;</td>';
+	}
 
-    echo "\n\t</tr>\n\t</tbody>\n\t</table>";
-    $calendar = ob_get_contents();
-    ob_end_clean();    
+	echo "\n\t</tr>\n\t</tbody>\n\t</table>";
+	$calendar = ob_get_contents();
+	ob_end_clean();    
     
-    return $calendar;
+	return $calendar;
 }
 
 // Used in get_calendar

Spacing +56 added lines, -56 removed lines

@@ -26,60 +26,60 @@ 
 // ------------------------------------------------------------------------ //
 include "header.php";
 
-if(planet_parse_args($args_num, $args, $args_str)){
+if (planet_parse_args($args_num, $args, $args_str)) {
 	$args["year"] = @$args_num[0];
 	$args["month"] = @$args_num[1];
 	$args["day"] = @$args_num[2];
 }
 
-$day = intval( empty($_GET["day"])?@$args["day"]:$_GET["day"] );
-$year = intval( empty($_GET["year"])?@$args["year"]:$_GET["year"] );
-$month = intval( empty($_GET["month"])?@$args["month"]:$_GET["month"] );
-$blog_id = intval( empty($_GET["blog"])?@$args["blog"]:$_GET["blog"] );
-$start = intval( empty($_GET["start"])?@$args["start"]:$_GET["start"] );
+$day = intval(empty($_GET["day"]) ? @$args["day"] : $_GET["day"]);
+$year = intval(empty($_GET["year"]) ? @$args["year"] : $_GET["year"]);
+$month = intval(empty($_GET["month"]) ? @$args["month"] : $_GET["month"]);
+$blog_id = intval(empty($_GET["blog"]) ? @$args["blog"] : $_GET["blog"]);
+$start = intval(empty($_GET["start"]) ? @$args["start"] : $_GET["start"]);
 
 $page["title"] = planet_constant("MD_ACHIVE");
 
-$article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
-$blog_handler =& xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
+$article_handler = & xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
+$blog_handler = & xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
 
-$xoopsOption["xoops_pagetitle"] = $xoopsModule->getVar("name"). " - " .planet_constant("MD_ACHIVE");
+$xoopsOption["xoops_pagetitle"] = $xoopsModule->getVar("name")." - ".planet_constant("MD_ACHIVE");
 $xoopsOption["template_main"] = planet_getTemplate("archive");
-include_once( XOOPS_ROOT_PATH . "/header.php" );
+include_once(XOOPS_ROOT_PATH."/header.php");
 include XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
 
-$year = empty($year)?date("Y"):$year;
-if($month<1){
+$year = empty($year) ? date("Y") : $year;
+if ($month<1) {
 	$month = $day = 0;
 	$page["time"] = sprintf(planet_constant("MD_TIME_Y"), $year);
-}elseif($day<1){
+}elseif ($day<1) {
 	$day = 0;
 	$page["time"] = sprintf(planet_constant("MD_TIME_YM"), $year, $month);
-}else{
+}else {
 	$page["time"] = sprintf(planet_constant("MD_TIME_YMD"), $year, $month, $day);
 }
 $time = array("year"=>$year, "month"=>$month, "day"=>$day);
 if ($xoopsUser) {
-    $timeoffset = ($xoopsUser->getVar("timezone_offset")- $xoopsConfig['server_TZ'])*3600;
-} else {
-    $timeoffset = ($xoopsConfig['default_TZ']- $xoopsConfig['server_TZ'])*3600;
+    $timeoffset = ($xoopsUser->getVar("timezone_offset")-$xoopsConfig['server_TZ'])*3600;
+}else {
+    $timeoffset = ($xoopsConfig['default_TZ']-$xoopsConfig['server_TZ'])*3600;
 }
 
 $criteria = new CriteriaCompo();
-if($blog_id){
+if ($blog_id) {
 	$criteria->add(new Criteria("blog_id", $blog_id));
 }
 $criteria->add(new Criteria("YEAR(FROM_UNIXTIME(art_time_publish - $timeoffset))", $year));
-if($month){
+if ($month) {
 	$criteria->add(new Criteria("MONTH(FROM_UNIXTIME(art_time_publish - $timeoffset))", $month));
-	if($day){
+	if ($day) {
 		$criteria->add(new Criteria("DAY(FROM_UNIXTIME(art_time_publish - $timeoffset))", $day));
 	}
 }
 $criteria->setStart($start);
 $criteria->setLimit($xoopsModuleConfig["articles_perpage"]);
 
-$articles_obj =& $article_handler->getAll(
+$articles_obj = & $article_handler->getAll(
 	$criteria,
 	array("uid", "art_title", "art_time", "blog_id", "art_content")
 );
@@ -87,10 +87,10 @@ 
 
 $articles = array();
 $blogs_id = array();
-foreach($articles_obj as $id=>$article){
+foreach ($articles_obj as $id=>$article) {
 	$articles[] = array(
 		"id"=>$id,
-		"blog"=>array("id"=>$article->getVar("blog_id"),"title"=>""),
+		"blog"=>array("id"=>$article->getVar("blog_id"), "title"=>""),
 		"title"=>$article->getVar("art_title"),
 		"time"=>$article->getTime(),
 		"content"=>$article->getVar("art_content")
@@ -101,32 +101,32 @@ 
 }
 $criteria_blog = new Criteria("blog_id", "(".implode(",", array_keys($blog_array)).")", "IN");
 $blogs = $blog_handler->getList($criteria_blog);
-foreach(array_keys($articles) as $key){
+foreach (array_keys($articles) as $key) {
 	$articles[$key]["blog"]["title"] = $blogs[$articles[$key]["blog"]["id"]];
 }
-if($blog_id>0){
-	$page["blog"] = $blogs[$blog_id];;
+if ($blog_id>0) {
+	$page["blog"] = $blogs[$blog_id]; ;
 }
 
-if ( $articles_count > $xoopsModuleConfig["articles_perpage"]) {
+if ($articles_count>$xoopsModuleConfig["articles_perpage"]) {
 	include(XOOPS_ROOT_PATH."/class/pagenav.php");
 	$nav = new XoopsPageNav($articles_count, $xoopsModuleConfig["articles_perpage"], $start, "start", "month=".$month."&amp;day=".$day."&amp;year=".$year."&amp;blog=".intval($blog_id));
 	$pagenav = $nav->renderNav(4);
-} else {
+}else {
 	$pagenav = "";
 }
 
 $timenav = null;
 $calendar = null;
 $months = null;
-if(empty($start)){
-	if($blog_id){
+if (empty($start)) {
+	if ($blog_id) {
 		$blog_criteria = " AND blog_id=".$blog_id;
-	}else{
+	}else {
 		$blog_criteria = "";
 	}
 	// Get monthly list
-	if(empty($month)){
+	if (empty($month)) {
     	$sql = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count 
             FROM ".planet_DB_prefix("article")."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
@@ -145,7 +145,7 @@ 
 			"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".($year-1)."/b".$blog_id,
 			"title"=>sprintf(planet_constant("MD_TIME_Y"), ($year-1))
 			);
-		if($year<date("Y")){
+		if ($year<date("Y")) {
 			$timenav["next"] = array(
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".($year+1)."/b".$blog_id,
 				"title"=>sprintf(planet_constant("MD_TIME_Y"), ($year+1))
@@ -153,7 +153,7 @@ 
 		}
 	}
 	// Get daily list
-	elseif(empty($day)){
+	elseif (empty($day)) {
     	$sql = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count 
             FROM ".planet_DB_prefix("article")."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
@@ -166,8 +166,8 @@ 
         while ($myrow = $xoopsDB->fetchArray($result)) {
             $days[$myrow["day"]]["count"] = $myrow["count"];
         }
-        for($i=1;$i<=31;$i++){
-	        if(!isset($days[$i])) continue;
+        for ($i = 1; $i<=31; $i++) {
+	        if (!isset($days[$i])) continue;
 	        $days[$i] = array(
             	"title"=>$days[$i]["count"],
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$year."/".$month."/".$i."/b".$blog_id
@@ -177,19 +177,19 @@ 
         $month_next = $month+1;
         $month_prev = $month-1;
         $_year = $year;
-        if($month==12) {
+        if ($month == 12) {
 	        $month_next = 1;
-	        $_year = $year +1;
+	        $_year = $year+1;
         }
-        if($month==1) {
+        if ($month == 1) {
 	        $month_pre = 12;
-	        $_year = $year - 1;
+	        $_year = $year-1;
         }
 		$timenav["prev"] = array(
 			"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$_year."/".$month_prev."/b".$blog_id,
 			"title"=>planet_constant("MD_MONTH_".$month_prev)
 			);
-		if($year<date("Y") || $month < date("n")){
+		if ($year<date("Y") || $month<date("n")) {
 			$timenav["next"] = array(
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$_year."/".$month_next."/b".$blog_id,
 				"title"=>planet_constant("MD_MONTH_".$month_next)
@@ -216,9 +216,9 @@ 
 
 function planet_getCalendar($year = null, $month = null, $days = null)
 {
-	$year = empty($year)?date("Y"):$year;
-	$month = empty($month)?date("n"):$month;
-    $unixmonth = mktime(0, 0 , 0, $month, 1, $year);
+	$year = empty($year) ? date("Y") : $year;
+	$month = empty($month) ? date("n") : $month;
+    $unixmonth = mktime(0, 0, 0, $month, 1, $year);
 	
     ob_start();
     echo '<table id="calendar">';
@@ -226,8 +226,8 @@ 
     printf(planet_constant("MD_TIME_YM"), $year, planet_constant("MD_MONTH_".$month));
     echo '</caption>';
 	
-    for ($i=1;$i<=7;$i++) {
-        echo "\n\t\t<th abbr=\"".planet_constant("MD_WEEK_".$i)."\" scope=\"col\" title=\"".planet_constant("MD_WEEK_".$i)."\">" . planet_constant("MD_WEEK_".$i) . '</th>';
+    for ($i = 1; $i<=7; $i++) {
+        echo "\n\t\t<th abbr=\"".planet_constant("MD_WEEK_".$i)."\" scope=\"col\" title=\"".planet_constant("MD_WEEK_".$i)."\">".planet_constant("MD_WEEK_".$i).'</th>';
     }
 
 	echo '<tr>';
@@ -238,8 +238,8 @@ 
     if (0 != $pad) echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
 
     $daysinmonth = intval(date('t', $unixmonth));
-    for ($day = 1; $day <= $daysinmonth; ++$day) {
-        if (isset($newrow) && $newrow){
+    for ($day = 1; $day<=$daysinmonth; ++$day) {
+        if (isset($newrow) && $newrow) {
             echo "\n\t</tr>\n\t<tr>\n\t\t";
         }
         $newrow = false;
@@ -247,22 +247,22 @@ 
        	echo '<td>';
 
         if (!empty($days[$day]["url"])) {
-			echo '<a href="' . $days[$day]["url"] . "\"";
-			if(!empty($days[$day]["title"])) echo "title=\"".$days[$day]["title"]."\"";
+			echo '<a href="'.$days[$day]["url"]."\"";
+			if (!empty($days[$day]["title"])) echo "title=\"".$days[$day]["title"]."\"";
 			echo ">$day</a>";
-        } elseif(!empty($days[$day]["title"])) {
+        } elseif (!empty($days[$day]["title"])) {
             echo "<acronym title=\"".$days[$day]["title"]."\">$day</acronym>";
-        } else {
+        }else {
             echo $day;
         }
         echo '</td>';
 
-	if (6 == planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins))
+	if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins))
             $newrow = true;
     }
 
-    $pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins);
-    if ($pad != 0 && $pad != 7){
+    $pad = 7-planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins);
+    if ($pad != 0 && $pad != 7) {
         echo "\n\t\t".'<td class="pad" colspan="'.$pad.'">&nbsp;</td>';
     }
 
@@ -276,7 +276,7 @@ 
 // Used in get_calendar
 function planet_calendar_week_mod($num) {
 	$base = 7;
-	return ($num - $base*floor($num/$base));
+	return ($num-$base*floor($num/$base));
 }
 
 ?>
\ No newline at end of file

Braces +15 added lines, -8 removed lines

@@ -52,10 +52,10 @@ 
 if($month<1){
 	$month = $day = 0;
 	$page["time"] = sprintf(planet_constant("MD_TIME_Y"), $year);
-}elseif($day<1){
+} elseif($day<1){
 	$day = 0;
 	$page["time"] = sprintf(planet_constant("MD_TIME_YM"), $year, $month);
-}else{
+} else{
 	$page["time"] = sprintf(planet_constant("MD_TIME_YMD"), $year, $month, $day);
 }
 $time = array("year"=>$year, "month"=>$month, "day"=>$day);
@@ -122,7 +122,7 @@ 
 if(empty($start)){
 	if($blog_id){
 		$blog_criteria = " AND blog_id=".$blog_id;
-	}else{
+	} else{
 		$blog_criteria = "";
 	}
 	// Get monthly list
@@ -167,7 +167,9 @@ 
             $days[$myrow["day"]]["count"] = $myrow["count"];
         }
         for($i=1;$i<=31;$i++){
-	        if(!isset($days[$i])) continue;
+	        if(!isset($days[$i])) {
+	        	continue;
+	        }
 	        $days[$i] = array(
             	"title"=>$days[$i]["count"],
 				"url"=>XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.archive.php".URL_DELIMITER."".$year."/".$month."/".$i."/b".$blog_id
@@ -235,7 +237,9 @@ 
 	// See how much we should pad in the beginning
 	$week_begins = 1;
     $pad = planet_calendar_week_mod(date('w', $unixmonth)-$week_begins);
-    if (0 != $pad) echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
+    if (0 != $pad) {
+    	echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
+    }
 
     $daysinmonth = intval(date('t', $unixmonth));
     for ($day = 1; $day <= $daysinmonth; ++$day) {
@@ -248,7 +252,9 @@ 
 
         if (!empty($days[$day]["url"])) {
 			echo '<a href="' . $days[$day]["url"] . "\"";
-			if(!empty($days[$day]["title"])) echo "title=\"".$days[$day]["title"]."\"";
+			if(!empty($days[$day]["title"])) {
+				echo "title=\"".$days[$day]["title"]."\"";
+			}
 			echo ">$day</a>";
         } elseif(!empty($days[$day]["title"])) {
             echo "<acronym title=\"".$days[$day]["title"]."\">$day</acronym>";
@@ -257,8 +263,9 @@ 
         }
         echo '</td>';
 
-	if (6 == planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins))
-            $newrow = true;
+	if (6 == planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins)) {
+	            $newrow = true;
+	}
     }
 
     $pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0 , 0, $month, $day, $year))-$week_begins);

transfer.php

Indentation +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 	exit();
 }else{
 	$data = array();
-    $data["id"] = $article_id;
+	$data["id"] = $article_id;
 	$data["title"] = $article_obj->getVar("art_title");
 	$data["time"] = $article_obj->getTime("l");
 	$data["image"] = "";

Spacing +17 added lines, -17 removed lines

@@ -15,38 +15,38 @@ 
 
 include "header.php";
 
-if(planet_parse_args($args_num, $args, $args_str)){
+if (planet_parse_args($args_num, $args, $args_str)) {
 	$args["article"] = @$args_num[0];
 	$args["op"] = @$args_str[0];
 }
 
-$article_id = intval( empty($_GET["article"]) ? ( empty($_POST["article"]) ? @$args["article"] : $_POST["article"] ) : $_GET["article"] );
+$article_id = intval(empty($_GET["article"]) ? (empty($_POST["article"]) ? @$args["article"] : $_POST["article"]) : $_GET["article"]);
 
-$op = empty($_GET["op"]) ? ( empty($_POST["op"])? @$args["op"] : $_POST["op"] ) : $_GET["op"];
+$op = empty($_GET["op"]) ? (empty($_POST["op"]) ? @$args["op"] : $_POST["op"]) : $_GET["op"];
 $op = strtolower(trim($op));
 
-if ( empty($article_id) )  {
-	if(empty($_SERVER['HTTP_REFERER'])){
+if (empty($article_id)) {
+	if (empty($_SERVER['HTTP_REFERER'])) {
 		include XOOPS_ROOT_PATH."/header.php";
 		xoops_error(_NOPERM);
 		$xoopsOption['output_type'] = "plain";
 		include XOOPS_ROOT_PATH."/footer.php";
 		exit();
-	}else{
+	}else {
 		$ref_parser = parse_url($_SERVER['HTTP_REFERER']);
 		$uri_parser = parse_url($_SERVER['REQUEST_URI']);
-		if(
+		if (
 			(!empty($ref_parser['host']) && !empty($uri_parser['host']) && $uri_parser['host'] != $ref_parser['host']) 
 			|| 
 			($ref_parser["path"] != $uri_parser["path"])
-		){
+		) {
 			include XOOPS_ROOT_PATH."/header.php";
 			include XOOPS_ROOT_PATH."/modules/".$xoopsModule->getVar("dirname")."/include/vars.php";
-			xoops_confirm(array(), "javascript: window.close();", sprintf(planet_constant("MD_TRANSFER_DONE"),""), _CLOSE, $_SERVER['HTTP_REFERER']);
+			xoops_confirm(array(), "javascript: window.close();", sprintf(planet_constant("MD_TRANSFER_DONE"), ""), _CLOSE, $_SERVER['HTTP_REFERER']);
 			$xoopsOption['output_type'] = "plain";
 			include XOOPS_ROOT_PATH."/footer.php";
 			exit();
-		}else{
+		}else {
 			include XOOPS_ROOT_PATH."/header.php";
 			xoops_error(_NOPERM);
 			$xoopsOption['output_type'] = "plain";
@@ -56,15 +56,15 @@ 
 	}
 }
 
-$article_handler =& xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
-$article_obj =& $article_handler->get($article_id);
+$article_handler = & xoops_getmodulehandler("article", $GLOBALS["moddirname"]);
+$article_obj = & $article_handler->get($article_id);
 
 // Display option form
-if(empty($op)){
-	$module_variables .=  "<input type=\"hidden\" name=\"article\" id=\"article\" value=\"{$article_id}\">";
+if (empty($op)) {
+	$module_variables .= "<input type=\"hidden\" name=\"article\" id=\"article\" value=\"{$article_id}\">";
 	include XOOPS_ROOT_PATH."/Frameworks/transfer/option.transfer.php";
 	exit();
-}else{
+}else {
 	$data = array();
     $data["id"] = $article_id;
 	$data["title"] = $article_obj->getVar("art_title");
@@ -74,7 +74,7 @@ 
 	$data["url"] = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/view.article.php".URL_DELIMITER."".$article_obj->getVar("art_id");
 	$data["author"] = $article_obj->getVar("art_author");
 	
-	switch($op){
+	switch ($op) {
 		
 		// Use title
 		case "bookmark";	
@@ -82,7 +82,7 @@ 
 		
 		case "print":
 		case "pdf":
-			${"{$op}_data"} =& $data;
+			${"{$op}_data"} = & $data;
 			${"{$op}_data"}["date"] = $pdf_data["time"];
 			${"{$op}_data"}["content"] = $article_obj->getVar("art_content");
 			break;

Braces +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 		$xoopsOption['output_type'] = "plain";
 		include XOOPS_ROOT_PATH."/footer.php";
 		exit();
-	}else{
+	} else{
 		$ref_parser = parse_url($_SERVER['HTTP_REFERER']);
 		$uri_parser = parse_url($_SERVER['REQUEST_URI']);
 		if(
@@ -46,7 +46,7 @@ 
 			$xoopsOption['output_type'] = "plain";
 			include XOOPS_ROOT_PATH."/footer.php";
 			exit();
-		}else{
+		} else{
 			include XOOPS_ROOT_PATH."/header.php";
 			xoops_error(_NOPERM);
 			$xoopsOption['output_type'] = "plain";
@@ -64,7 +64,7 @@ 
 	$module_variables .=  "<input type=\"hidden\" name=\"article\" id=\"article\" value=\"{$article_id}\">";
 	include XOOPS_ROOT_PATH."/Frameworks/transfer/option.transfer.php";
 	exit();
-}else{
+} else{
 	$data = array();
     $data["id"] = $article_id;
 	$data["title"] = $article_obj->getVar("art_title");