mambax7 / smartfaq

smartfaq/answer.php

Braces +6 added lines, -2 removed lines

@@ -20,8 +20,12 @@
 $op = 'form';
 
 // Getting the operation we are doing
-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'];
+}
 
 // Getting the faqid
 $faqid = isset($_GET['faqid'])? intval($_GET['faqid']) : 0;

smartfaq/include/functions.render.php

Braces +3 added lines, -1 removed lines

@@ -123,7 +123,9 @@
     global $xoTheme, $xoopsConfig;
     static $icon_handler;
 
-    if (isset($icon_handler)) return $icon_handler;
+    if (isset($icon_handler)) {
+        return $icon_handler;
+    }
 
     if (!class_exists("NewbbIconHandler")) {
         require_once dirname(__DIR__)."/class/icon.php";

smartfaq/include/blocksadmin.inc.php

Switch Indentation +12 added lines, -12 removed lines

@@ -203,18 +203,18 @@
         }
         if ($myblock->getVar('block_type') == 'C') {
             switch ($bctype) {
-            case 'H':
-                $name = _AM_CUSTOMHTML;
-                break;
-            case 'P':
-                $name = _AM_CUSTOMPHP;
-                break;
-            case 'S':
-                $name = _AM_CUSTOMSMILE;
-                break;
-            default:
-                $name = _AM_CUSTOMNOSMILE;
-                break;
+                case 'H':
+                    $name = _AM_CUSTOMHTML;
+                    break;
+                case 'P':
+                    $name = _AM_CUSTOMPHP;
+                    break;
+                case 'S':
+                    $name = _AM_CUSTOMSMILE;
+                    break;
+                default:
+                    $name = _AM_CUSTOMNOSMILE;
+                    break;
             }
             $myblock->setVar('name', $name);
             $myblock->setVar('c_type', $bctype);

Braces +4 added lines, -1 removed lines

@@ -191,7 +191,10 @@
         }
         $myblock = new XoopsBlock($bid);
         // $myblock->setVar('side', $bside); GIJ -
-        if ( $bside >= 0 ) $myblock->setVar('side', $bside); // GIJ +
+        if ( $bside >= 0 ) {
+            $myblock->setVar('side', $bside);
+        }
+        // GIJ +
         $myblock->setVar('weight', $bweight);
         $myblock->setVar('visible', $bvisible);
         $myblock->setVar('title', $btitle);

smartfaq/include/functions.image.php

Braces +24 added lines, -9 removed lines

@@ -80,7 +80,9 @@ 
         $attachmentImage .= '</a>';
     } elseif (file_exists($image)) {
         $attachmentImage = '<img src="'.$image_url.'" alt="'.$source.' '.$img_info.'" />';
-    } else $attachmentImage = '';
+    } else {
+        $attachmentImage = '';
+    }
 
     return $attachmentImage;
 }
@@ -152,17 +154,27 @@ 
     $type = $imginfo[2];
     $supported_types = array();
 
-    if (!extension_loaded('gd')) return false;
-    if (function_exists('imagegif')) $supported_types[] = 1;
-    if (function_exists('imagejpeg'))$supported_types[] = 2;
-    if (function_exists('imagepng')) $supported_types[] = 3;
+    if (!extension_loaded('gd')) {
+        return false;
+    }
+    if (function_exists('imagegif')) {
+        $supported_types[] = 1;
+    }
+    if (function_exists('imagejpeg')) {
+        $supported_types[] = 2;
+    }
+    if (function_exists('imagepng')) {
+        $supported_types[] = 3;
+    }
 
     $imageCreateFunction = (function_exists('imagecreatetruecolor'))? "imagecreatetruecolor" : "imagecreate";
 
     if (in_array($type, $supported_types) ) {
         switch ($type) {
             case 1 :
-                if (!function_exists('imagecreatefromgif')) return false;
+                if (!function_exists('imagecreatefromgif')) {
+                    return false;
+                }
                 $im = imagecreatefromgif ($src_file);
                 $new_im = imagecreate($newWidth, $newHeight);
                 imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
@@ -189,8 +201,11 @@ 
         }
     }
 
-    if (file_exists($new_file))    return true;
-    else return false;
-}
+    if (file_exists($new_file)) {
+        return true;
+    } else {
+        return false;
+    }
+    }
 
 endif;

smartfaq/class/answer.php

Braces +31 added lines, -13 removed lines

@@ -62,17 +62,24 @@ 
     // attachment functions    TODO: there should be a file/attachment management class
     function getAttachment()
     {
-        if (count($this->attachment_array)) return $this->attachment_array;
+        if (count($this->attachment_array)) {
+            return $this->attachment_array;
+        }
         $attachment = $this->getVar('attachment');
-        if (empty($attachment)) $this->attachment_array = null;
-        else $this->attachment_array = @unserialize(base64_decode($attachment));
+        if (empty($attachment)) {
+            $this->attachment_array = null;
+        } else {
+            $this->attachment_array = @unserialize(base64_decode($attachment));
+        }
 
         return $this->attachment_array;
     }
 
     function incrementDownload($attach_key)
     {
-        if (!$attach_key) return false;
+        if (!$attach_key) {
+            return false;
+        }
         $this->attachment_array[strval($attach_key)]['num_download'] ++;
 
         return $this->attachment_array[strval($attach_key)]['num_download'];
@@ -80,9 +87,11 @@ 
 
     function saveAttachment()
     {
-        if (is_array($this->attachment_array) && count($this->attachment_array) > 0)
-            $attachment_save = base64_encode(serialize($this->attachment_array));
-        else $attachment_save = '';
+        if (is_array($this->attachment_array) && count($this->attachment_array) > 0) {
+                    $attachment_save = base64_encode(serialize($this->attachment_array));
+        } else {
+            $attachment_save = '';
+        }
         $this->setVar('attachment', $attachment_save);
         $sql = "UPDATE " . $GLOBALS["xoopsDB"]->prefix("smartfaq_answers") . " SET attachment=" . $GLOBALS["xoopsDB"]->quoteString($attachment_save) . " WHERE post_id = " . $this->getVar('answerid');
         if (!$result = $GLOBALS["xoopsDB"]->queryF($sql)) {
@@ -98,11 +107,18 @@ 
         global $xoopsModuleConfig;
 
         $attach_old = $this->getAttachment();
-        if (!is_array($attach_old) || count($attach_old) < 1) return true;
+        if (!is_array($attach_old) || count($attach_old) < 1) {
+            return true;
+        }
         $this->attachment_array = array();
 
-        if ($attach_array === null) $attach_array = array_keys($attach_old); // to delete all!
-        if (!is_array($attach_array)) $attach_array = array($attach_array);
+        if ($attach_array === null) {
+            $attach_array = array_keys($attach_old);
+        }
+        // to delete all!
+        if (!is_array($attach_array)) {
+            $attach_array = array($attach_array);
+        }
 
         foreach ($attach_old as $key => $attach) {
             if (in_array($key, $attach_array)) {
@@ -112,9 +128,11 @@ 
             }
             $this->attachment_array[$key] = $attach;
         }
-        if (is_array($this->attachment_array) && count($this->attachment_array) > 0)
-            $attachment_save = base64_encode(serialize($this->attachment_array));
-        else $attachment_save = '';
+        if (is_array($this->attachment_array) && count($this->attachment_array) > 0) {
+                    $attachment_save = base64_encode(serialize($this->attachment_array));
+        } else {
+            $attachment_save = '';
+        }
         $this->setVar('attachment', $attachment_save);
 
         return true;

smartfaq/submit.php

Switch Indentation +145 added lines, -145 removed lines

@@ -47,161 +47,161 @@ 
 }
 
 switch ($op) {
-    case 'preview':
+        case 'preview':
 
-    Global $xoopsUser, $xoopsConfig, $xoopsModule, $xoopsModuleConfig, $xoopsDB;
+        Global $xoopsUser, $xoopsConfig, $xoopsModule, $xoopsModuleConfig, $xoopsDB;
 
-    $faqObj = $faq_handler->create();
-    $answerObj = $answer_handler->create();
-    $categoryObj = $category_handler->get($_POST['categoryid']);
+        $faqObj = $faq_handler->create();
+        $answerObj = $answer_handler->create();
+        $categoryObj = $category_handler->get($_POST['categoryid']);
 
-    if (!$xoopsUser) {
-        if ($xoopsModuleConfig['anonpost'] == 1) {
-            $uid = 0;
+        if (!$xoopsUser) {
+            if ($xoopsModuleConfig['anonpost'] == 1) {
+                $uid = 0;
+            } else {
+                redirect_header("index.php", 3, _NOPERM);
+                exit();
+            }
         } else {
-            redirect_header("index.php", 3, _NOPERM);
-            exit();
+            $uid = $xoopsUser->uid();
         }
-    } else {
-        $uid = $xoopsUser->uid();
-    }
 
-    $notifypub = isset($_POST['notifypub'])? $_POST['notifypub'] : 0;
+        $notifypub = isset($_POST['notifypub'])? $_POST['notifypub'] : 0;
 
-    // Putting the values about the FAQ in the FAQ object
-    $faqObj->setVar('categoryid', $_POST['categoryid']);
-    $faqObj->setVar('uid', $uid);
-    $faqObj->setVar('question', $_POST['question']);
-    $faqObj->setVar('howdoi', $_POST['howdoi']);
-    $faqObj->setVar('diduno', $_POST['diduno']);
-    $faqObj->setVar('datesub', time());
+        // Putting the values about the FAQ in the FAQ object
+        $faqObj->setVar('categoryid', $_POST['categoryid']);
+        $faqObj->setVar('uid', $uid);
+        $faqObj->setVar('question', $_POST['question']);
+        $faqObj->setVar('howdoi', $_POST['howdoi']);
+        $faqObj->setVar('diduno', $_POST['diduno']);
+        $faqObj->setVar('datesub', time());
 
-    // Putting the values in the answer object
-    $answerObj->setVar('status', _SF_AN_STATUS_APPROVED);
-    $answerObj->setVar('faqid', $faqObj->faqid());
-    $answerObj->setVar('answer', $_POST['answer']);
-    $answerObj->setVar('uid', $uid);
+        // Putting the values in the answer object
+        $answerObj->setVar('status', _SF_AN_STATUS_APPROVED);
+        $answerObj->setVar('faqid', $faqObj->faqid());
+        $answerObj->setVar('answer', $_POST['answer']);
+        $answerObj->setVar('uid', $uid);
 
-    global $xoopsUser, $myts;
+        global $xoopsUser, $myts;
 
-    $xoopsOption['template_main'] = 'smartfaq_submit.tpl';
-    include_once(XOOPS_ROOT_PATH . "/header.php");
-    include_once __DIR__ . '/footer.php';
+        $xoopsOption['template_main'] = 'smartfaq_submit.tpl';
+        include_once(XOOPS_ROOT_PATH . "/header.php");
+        include_once __DIR__ . '/footer.php';
 
-    $name = ($xoopsUser)? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
+        $name = ($xoopsUser)? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
 
-    $moduleName = $myts->displayTarea($xoopsModule->getVar('name'));
-    $faq = $faqObj->toArray(null, $categoryObj, false);
-    $faq['categoryPath'] = $categoryObj->getCategoryPath(true);
-    $faq['answer'] = $answerObj->answer();
-    $faq['who_when'] = $faqObj->getWhoAndWhen();
+        $moduleName = $myts->displayTarea($xoopsModule->getVar('name'));
+        $faq = $faqObj->toArray(null, $categoryObj, false);
+        $faq['categoryPath'] = $categoryObj->getCategoryPath(true);
+        $faq['answer'] = $answerObj->answer();
+        $faq['who_when'] = $faqObj->getWhoAndWhen();
 
-    $faq['comments'] = -1;
-    $xoopsTpl->assign('faq', $faq);
-    $xoopsTpl->assign('op', 'preview');
-    $xoopsTpl->assign('whereInSection', $moduleName);
-    $xoopsTpl->assign('lang_submit', _MD_SF_SUB_SNEWNAME);
+        $faq['comments'] = -1;
+        $xoopsTpl->assign('faq', $faq);
+        $xoopsTpl->assign('op', 'preview');
+        $xoopsTpl->assign('whereInSection', $moduleName);
+        $xoopsTpl->assign('lang_submit', _MD_SF_SUB_SNEWNAME);
 
-    $xoopsTpl->assign('lang_intro_title', sprintf(_MD_SF_SUB_SNEWNAME, ucwords($xoopsModule->name())));
-    $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY . "<b>$name</b>, " . _MD_SF_SUB_INTRO);
+        $xoopsTpl->assign('lang_intro_title', sprintf(_MD_SF_SUB_SNEWNAME, ucwords($xoopsModule->name())));
+        $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY . "<b>$name</b>, " . _MD_SF_SUB_INTRO);
 
-    include_once 'include/submit.inc.php';
+        include_once 'include/submit.inc.php';
 
-    include_once XOOPS_ROOT_PATH . '/footer.php';
+        include_once XOOPS_ROOT_PATH . '/footer.php';
 
-    exit();
-    break;
+        exit();
+        break;
 
-    case 'post':
+        case 'post':
 
-    Global $xoopsUser, $xoopsConfig, $xoopsModule, $xoopsModuleConfig, $xoopsDB;
+        Global $xoopsUser, $xoopsConfig, $xoopsModule, $xoopsModuleConfig, $xoopsDB;
 
-    $newFaqObj = $faq_handler->create();
-    $newAnswerObj = $answer_handler->create();
+        $newFaqObj = $faq_handler->create();
+        $newAnswerObj = $answer_handler->create();
 
-    if (!$xoopsUser) {
-        if ($xoopsModuleConfig['anonpost'] == 1) {
-            $uid = 0;
+        if (!$xoopsUser) {
+            if ($xoopsModuleConfig['anonpost'] == 1) {
+                $uid = 0;
+            } else {
+                redirect_header("index.php", 3, _NOPERM);
+                exit();
+            }
         } else {
-            redirect_header("index.php", 3, _NOPERM);
-            exit();
+            $uid = $xoopsUser->uid();
         }
-    } else {
-        $uid = $xoopsUser->uid();
-    }
 
-    $notifypub = isset($_POST['notifypub'])? $_POST['notifypub'] : 0;
-
-    // Putting the values about the FAQ in the FAQ object
-    $newFaqObj->setVar('categoryid', $_POST['categoryid']);
-    $newFaqObj->setVar('uid', $uid);
-    $newFaqObj->setVar('question', $_POST['question']);
-    $newFaqObj->setVar('howdoi', $_POST['howdoi']);
-    $newFaqObj->setVar('diduno', $_POST['diduno']);
-    $newFaqObj->setVar('notifypub', $notifypub);
-    //$newFaqObj->setVar('modulelink', $_POST['modulelink']);
-    //$newFaqObj->setVar('contextpage', $_POST['contextpage']);
-
-    // Setting the status of the FAQ
-
-    // if user is admin, FAQ are automatically published
-    $isAdmin = sf_userIsAdmin();
-    if ($isAdmin) {
-        $newFaqObj->setVar('status', _SF_STATUS_PUBLISHED);
-    } elseif ($xoopsModuleConfig['autoapprove_submitted_faq'] ==  1) {
-        $newFaqObj->setVar('status', _SF_STATUS_PUBLISHED);
-    } else {
-        $newFaqObj->setVar('status', _SF_STATUS_SUBMITTED);
-    }
+        $notifypub = isset($_POST['notifypub'])? $_POST['notifypub'] : 0;
+
+        // Putting the values about the FAQ in the FAQ object
+        $newFaqObj->setVar('categoryid', $_POST['categoryid']);
+        $newFaqObj->setVar('uid', $uid);
+        $newFaqObj->setVar('question', $_POST['question']);
+        $newFaqObj->setVar('howdoi', $_POST['howdoi']);
+        $newFaqObj->setVar('diduno', $_POST['diduno']);
+        $newFaqObj->setVar('notifypub', $notifypub);
+        //$newFaqObj->setVar('modulelink', $_POST['modulelink']);
+        //$newFaqObj->setVar('contextpage', $_POST['contextpage']);
+
+        // Setting the status of the FAQ
+
+        // if user is admin, FAQ are automatically published
+        $isAdmin = sf_userIsAdmin();
+        if ($isAdmin) {
+            $newFaqObj->setVar('status', _SF_STATUS_PUBLISHED);
+        } elseif ($xoopsModuleConfig['autoapprove_submitted_faq'] ==  1) {
+            $newFaqObj->setVar('status', _SF_STATUS_PUBLISHED);
+        } else {
+            $newFaqObj->setVar('status', _SF_STATUS_SUBMITTED);
+        }
 
-    // Storing the FAQ object in the database
-    if ( !$newFaqObj->store() ) {
-        redirect_header("javascript:history.go(-1)", 2, _MD_SF_SUBMIT_ERROR);
-        exit();
-    }
+        // Storing the FAQ object in the database
+        if ( !$newFaqObj->store() ) {
+            redirect_header("javascript:history.go(-1)", 2, _MD_SF_SUBMIT_ERROR);
+            exit();
+        }
 
-    // Putting the values in the answer object
-    $newAnswerObj->setVar('status', _SF_AN_STATUS_APPROVED);
-    $newAnswerObj->setVar('faqid', $newFaqObj->faqid());
-    $newAnswerObj->setVar('answer', $_POST['answer']);
-    $newAnswerObj->setVar('uid', $uid);
-
-//====================================================================================
-        //TODO post Attachment
-        $attachments_tmp = array();
-        if (!empty($_POST["attachments_tmp"])) {
-            $attachments_tmp = unserialize(base64_decode($_POST["attachments_tmp"]));
-            if (isset($_POST["delete_tmp"]) && count($_POST["delete_tmp"])) {
-                foreach ($_POST["delete_tmp"] as $key) {
-                    unlink(XOOPS_ROOT_PATH . "/" . $xoopsModuleConfig['dir_attachments'] . "/" . $attachments_tmp[$key][0]);
-                    unset($attachments_tmp[$key]);
+        // Putting the values in the answer object
+        $newAnswerObj->setVar('status', _SF_AN_STATUS_APPROVED);
+        $newAnswerObj->setVar('faqid', $newFaqObj->faqid());
+        $newAnswerObj->setVar('answer', $_POST['answer']);
+        $newAnswerObj->setVar('uid', $uid);
+
+    //====================================================================================
+            //TODO post Attachment
+            $attachments_tmp = array();
+            if (!empty($_POST["attachments_tmp"])) {
+                $attachments_tmp = unserialize(base64_decode($_POST["attachments_tmp"]));
+                if (isset($_POST["delete_tmp"]) && count($_POST["delete_tmp"])) {
+                    foreach ($_POST["delete_tmp"] as $key) {
+                        unlink(XOOPS_ROOT_PATH . "/" . $xoopsModuleConfig['dir_attachments'] . "/" . $attachments_tmp[$key][0]);
+                        unset($attachments_tmp[$key]);
+                    }
                 }
             }
-        }
-        if (count($attachments_tmp)) {
-            foreach ($attachments_tmp as $key => $attach) {
-                if (rename(XOOPS_CACHE_PATH . "/" . $attachments_tmp[$key][0], XOOPS_ROOT_PATH . "/" . $xoopsModuleConfig['dir_attachments'] . "/" . $attachments_tmp[$key][0])) {
-                    $post_obj->setAttachment($attach[0], $attach[1], $attach[2]);
+            if (count($attachments_tmp)) {
+                foreach ($attachments_tmp as $key => $attach) {
+                    if (rename(XOOPS_CACHE_PATH . "/" . $attachments_tmp[$key][0], XOOPS_ROOT_PATH . "/" . $xoopsModuleConfig['dir_attachments'] . "/" . $attachments_tmp[$key][0])) {
+                        $post_obj->setAttachment($attach[0], $attach[1], $attach[2]);
+                    }
                 }
             }
-        }
-        $error_upload = '';
-
-        if (isset($_FILES['userfile']['name']) && $_FILES['userfile']['name'] != '' && $topic_handler->getPermission($forum_obj, $topic_status, 'attach') ) {
-            require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar("dirname", "n") . '/class/uploader.php';
-            $maxfilesize = $forum_obj->getVar('attach_maxkb') * 1024;
-            $uploaddir = XOOPS_CACHE_PATH;
-
-            $uploader = new sf_uploader(
-                $uploaddir,
-                $newAnswerObj->getVar('attach_ext'),
-                intval($maxfilesize),
-                intval($xoopsModuleConfig['max_img_width']),
-                intval($xoopsModuleConfig['max_img_height'])
-            );
-
-            if ($_FILES['userfile']['error'] > 0) {
-                switch ($_FILES['userfile']['error']) {
+            $error_upload = '';
+
+            if (isset($_FILES['userfile']['name']) && $_FILES['userfile']['name'] != '' && $topic_handler->getPermission($forum_obj, $topic_status, 'attach') ) {
+                require_once XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar("dirname", "n") . '/class/uploader.php';
+                $maxfilesize = $forum_obj->getVar('attach_maxkb') * 1024;
+                $uploaddir = XOOPS_CACHE_PATH;
+
+                $uploader = new sf_uploader(
+                    $uploaddir,
+                    $newAnswerObj->getVar('attach_ext'),
+                    intval($maxfilesize),
+                    intval($xoopsModuleConfig['max_img_width']),
+                    intval($xoopsModuleConfig['max_img_height'])
+                );
+
+                if ($_FILES['userfile']['error'] > 0) {
+                    switch ($_FILES['userfile']['error']) {
                     case 1:
                         $error_message[] = _MD_NEWBB_MAXUPLOADFILEINI;
                         break;
@@ -211,7 +211,7 @@ 
                     default:
                         $error_message[] = _MD_NEWBB_UPLOAD_ERRNODEF;
                         break;
-                }
+                    }
             } else {
                 $uploader->setCheckMediaTypeByExt();
 
@@ -277,30 +277,30 @@ 
     exit();
     break;
 
-    case 'form':
-    default:
+        case 'form':
+        default:
 
-    global $xoopsUser, $myts;
+        global $xoopsUser, $myts;
 
-    $faqObj = $faq_handler->create();
-    $answerObj = $answer_handler->create();
-    $categoryObj = $category_handler->create();
+        $faqObj = $faq_handler->create();
+        $answerObj = $answer_handler->create();
+        $categoryObj = $category_handler->create();
 
-    $xoopsOption['template_main'] = 'smartfaq_submit.html';
-    include_once(XOOPS_ROOT_PATH . "/header.php");
-    include_once __DIR__ . '/footer.php';
+        $xoopsOption['template_main'] = 'smartfaq_submit.html';
+        include_once(XOOPS_ROOT_PATH . "/header.php");
+        include_once __DIR__ . '/footer.php';
 
-    $name = ($xoopsUser)? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
-    $notifypub = 1;
-    $moduleName = $myts->displayTarea($xoopsModule->getVar('name'));
-    $xoopsTpl->assign('whereInSection', $moduleName);
-    $xoopsTpl->assign('lang_submit', _MD_SF_SUB_SNEWNAME);
+        $name = ($xoopsUser)? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
+        $notifypub = 1;
+        $moduleName = $myts->displayTarea($xoopsModule->getVar('name'));
+        $xoopsTpl->assign('whereInSection', $moduleName);
+        $xoopsTpl->assign('lang_submit', _MD_SF_SUB_SNEWNAME);
 
-    $xoopsTpl->assign('lang_intro_title', sprintf(_MD_SF_SUB_SNEWNAME, ucwords($xoopsModule->name())));
-    $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY . "<b>$name</b>, " . _MD_SF_SUB_INTRO);
+        $xoopsTpl->assign('lang_intro_title', sprintf(_MD_SF_SUB_SNEWNAME, ucwords($xoopsModule->name())));
+        $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY . "<b>$name</b>, " . _MD_SF_SUB_INTRO);
 
-    include_once 'include/submit.inc.php';
+        include_once 'include/submit.inc.php';
 
-    include_once XOOPS_ROOT_PATH . '/footer.php';
-    break;
+        include_once XOOPS_ROOT_PATH . '/footer.php';
+        break;
 }

smartfaq/admin/answer.php

Braces +6 added lines, -2 removed lines

@@ -12,8 +12,12 @@
 $op = '';
 
 // Getting the operation we are doing
-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'];
+}
 
 // Creating the answer handler object
 $answer_handler =& sf_gethandler('answer');

smartfaq/admin/faq.php

Braces +6 added lines, -2 removed lines

@@ -19,8 +19,12 @@
 $answer_handler =& sf_gethandler('answer');
 
 $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'];
+}
 
 // Where shall we start?
 $startfaq = isset($_GET['startfaq'])? intval($_GET['startfaq']) : 0;

smartfaq/admin/main.php

Switch Indentation +6 added lines, -6 removed lines

@@ -182,13 +182,13 @@
 }
 
 switch ($ordersel) {
-    case 'ASC':
-    $ordertxtasc = "selected='selected'";
-    break;
+        case 'ASC':
+        $ordertxtasc = "selected='selected'";
+        break;
 
-    default :
-    $ordertxtdesc = "selected='selected'";
-    break;
+        default :
+        $ordertxtdesc = "selected='selected'";
+        break;
 }
 
 switch ($statussel) {