mambax7 / smartfaq

smartfaq/admin/myblocksadmin.php

Braces +19 added lines, -7 removed lines

@@ -22,7 +22,9 @@ 
 
 // language files
 $language = $xoopsConfig['language'] ;
-if (!file_exists("$xoops_system_path/language/$language/admin/blocksadmin.php")) $language = 'english';
+if (!file_exists("$xoops_system_path/language/$language/admin/blocksadmin.php")) {
+    $language = 'english';
+}
 
 // to prevent from notice that constants already defined
 $error_reporting_level = error_reporting(0);
@@ -34,15 +36,21 @@ 
 
 $group_defs = file( "$xoops_system_path/language/$language/admin/groups.php" ) ;
 foreach ($group_defs as $def) {
-    if ( strstr( $def , '_AM_ACCESSRIGHTS' ) || strstr( $def , '_AM_ACTIVERIGHTS' ) ) eval( $def ) ;
-}
+    if ( strstr( $def , '_AM_ACCESSRIGHTS' ) || strstr( $def , '_AM_ACTIVERIGHTS' ) ) {
+        eval( $def ) ;
+    }
+    }
 
 // check $xoopsModule
-if ( ! is_object( $xoopsModule ) ) redirect_header( XOOPS_URL.'/user.php' , 1 , _NOPERM ) ;
+if ( ! is_object( $xoopsModule ) ) {
+    redirect_header( XOOPS_URL.'/user.php' , 1 , _NOPERM ) ;
+}
 
 // check access right (needs system_admin of BLOCK)
 $sysperm_handler =& xoops_gethandler('groupperm');
-if (!$sysperm_handler->checkRight('system_admin', XOOPS_SYSTEM_BLOCK, $xoopsUser->getGroups())) redirect_header( XOOPS_URL.'/user.php' , 1 , _NOPERM ) ;
+if (!$sysperm_handler->checkRight('system_admin', XOOPS_SYSTEM_BLOCK, $xoopsUser->getGroups())) {
+    redirect_header( XOOPS_URL.'/user.php' , 1 , _NOPERM ) ;
+}
 
 // get blocks owned by the module
 $block_arr =& XoopsBlock::getByModule( $xoopsModule->mid() ) ;
@@ -86,10 +94,12 @@ 
         // visible and side
         if ( $block_arr[$i]->getVar("visible") != 1 ) {
             $sseln = " checked='checked' style='background-color:#FF0000;'";
-        } else switch ( $block_arr[$i]->getVar("side") ) {
+        } else {
+            switch ( $block_arr[$i]->getVar("side") ) {
             default :
             case XOOPS_SIDEBLOCK_LEFT :
             $ssel0 = " checked='checked' style='background-color:#00FF00;'";
+        }
             break ;
             case XOOPS_SIDEBLOCK_RIGHT :
             $ssel1 = " checked='checked' style='background-color:#00FF00;'";
@@ -217,7 +227,9 @@ 
 }
 
 xoops_cp_header() ;
-if (file_exists('./mymenu.php')) include('./mymenu.php');
+if (file_exists('./mymenu.php')) {
+    include('./mymenu.php');
+}
 //sf_adminMenu(5, _AM_SF_BLOCKSANDGROUPS);
 
 list_blocks() ;

smartfaq/admin/mygrouppermform.php

Braces +3 added lines, -1 removed lines

@@ -376,8 +376,10 @@
             $tree .= ' checked="checked"';
         }
         $tree .= " />".$option['name']."<input type=\"hidden\" name=\"".$this->getName()."[parents][".$option['id']."]\" value=\"".implode(':', $parentIds)."\" /><input type=\"hidden\" name=\"".$this->getName()."[itemname][".$option['id']."]\" value=\"".htmlspecialchars($option['name'])."\" /><br />\n";
-        if ( isset( $option['children'] ) ) foreach ($option['children'] as $child) {
+        if ( isset( $option['children'] ) ) {
+            foreach ($option['children'] as $child) {
             array_push($parentIds, $option['id']);
+        }
             $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix.'&nbsp;-', $parentIds);
         }
     }

smartfaq/admin/mygroupperm.php

Braces +3 added lines, -1 removed lines

@@ -50,7 +50,9 @@
             // var_dump( $HTTP_POST_VARS['perms'] ) ;
             // exit ;
             if (false != myDeleteByModule($gperm_handler->db,$modid,$perm_name,$item_id)) {
-                if ( empty( $perm_data['groups'] ) ) continue ;
+                if ( empty( $perm_data['groups'] ) ) {
+                    continue ;
+                }
                 foreach ($perm_data['groups'] as $group_id => $item_ids) {
     //				foreach ($item_ids as $item_id => $selected) {
                     $selected = isset( $item_ids[ $item_id ] )? $item_ids[ $item_id ] : 0 ;

smartfaq/admin/import.php

Braces +6 added lines, -2 removed lines

@@ -11,8 +11,12 @@
 
 $op='none';
 
-if (isset($_GET['op'])) $op = $_GET['op'];
-if (isset($_POST['op'])) $op = $_POST['op'];
+if (isset($_GET['op'])) {
+    $op = $_GET['op'];
+}
+if (isset($_POST['op'])) {
+    $op = $_POST['op'];
+}
 
 global $xoopsDB;
 

smartfaq/request.php

Braces +6 added lines, -2 removed lines

@@ -36,8 +36,12 @@
 
 $op = '';
 
-if (isset($_GET['op'])) $op = $_GET['op'];
-if (isset($_POST['op'])) $op = $_POST['op'];
+if (isset($_GET['op'])) {
+    $op = $_GET['op'];
+}
+if (isset($_POST['op'])) {
+    $op = $_POST['op'];
+}
 
 switch ($op) {
     case 'post':