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

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/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/admin.php

Braces +3 added lines, -1 removed lines

@@ -36,7 +36,9 @@
 if (isset($HTTP_GET_VARS['fct'])) {
     $fct = trim($HTTP_GET_VARS['fct']);
 }
-if (empty($fct)) $fct = 'preferences' ;
+if (empty($fct)) {
+    $fct = 'preferences' ;
+}
 include dirname(dirname(dirname(__DIR__))) . '/mainfile.php';
 include XOOPS_ROOT_PATH."/include/cp_functions.php";
 

smartfaq/admin/question.php

Braces +6 added lines, -2 removed lines

@@ -16,8 +16,12 @@
 $category_handler =& sf_gethandler('category');
 
 $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;