mambax7 / smartfaq

include/search.inc.php

Spacing +3 added lines, -3 removed lines

@@ -14,13 +14,13 @@
 
     $ret = array();
 
-    $faq_handler =& sf_gethandler('faq');
+    $faq_handler = & sf_gethandler('faq');
 
-    $faqsObj =& $faq_handler->getFaqsFromSearch($queryarray, $andor, $limit, $offset, $userid);
+    $faqsObj = & $faq_handler->getFaqsFromSearch($queryarray, $andor, $limit, $offset, $userid);
 
     for ($i = 0; $i < count($faqsObj); ++$i) {
         $ret[$i]['image'] = "assets/images/smartfaq.gif";
-        $ret[$i]['link'] = "faq.php?faqid=" . $faqsObj[$i]->faqid();
+        $ret[$i]['link'] = "faq.php?faqid=".$faqsObj[$i]->faqid();
         $ret[$i]['title'] = $faqsObj[$i]->question(50);
         $ret[$i]['time'] = $faqsObj[$i]->getVar('datesub');
         $ret[$i]['uid'] = $faqsObj[$i]->uid();

include/functions.image.php

Spacing +18 added lines, -18 removed lines

@@ -47,25 +47,25 @@ 
     $image_url = $img_url.'/'.$source;
 
     $imginfo = @getimagesize($image);
-    $img_info = ( count($imginfo)>0 ) ? $imginfo[0]."X".$imginfo[1].' px':"";
+    $img_info = (count($imginfo) > 0) ? $imginfo[0]."X".$imginfo[1].' px' : "";
 
     if ($xoopsModuleConfig['max_image_width'] > 0 && $xoopsModuleConfig['max_image_height'] > 0) {
         if ($imginfo[0] > $xoopsModuleConfig['max_image_width'] || $imginfo[1] > $xoopsModuleConfig['max_image_height']) {
             //if (!file_exists($thumb_path.'/'.$source) && $imginfo[0] > $xoopsModuleConfig['max_img_width']) {
-            if (!file_exists($thumb_path.'/'.$source) ) {
+            if (!file_exists($thumb_path.'/'.$source)) {
                 sf_createThumbnail($source, $xoopsModuleConfig['max_image_width']);
             }
         }
 
         if ($imginfo[0] > $xoopsModuleConfig['max_image_width'] || $imginfo[1] > $xoopsModuleConfig['max_image_height']) {
             $pseudo_width = $xoopsModuleConfig['max_image_width'];
-            $pseudo_height = $xoopsModuleConfig['max_image_width']*($imginfo[1]/$imginfo[0]);
+            $pseudo_height = $xoopsModuleConfig['max_image_width'] * ($imginfo[1] / $imginfo[0]);
             $pseudo_size = "width='".$pseudo_width."px' height='".$pseudo_height."px'";
         }
         // irmtfan to fix Undefined variable: pseudo_height
         if (!empty($pseudo_height) && $xoopsModuleConfig['max_image_height'] > 0 && $pseudo_height > $xoopsModuleConfig['max_image_height']) {
             $pseudo_height = $xoopsModuleConfig['max_image_height'];
-            $pseudo_width = $xoopsModuleConfig['max_image_height'] * ($imginfo[0]/$imginfo[1]);
+            $pseudo_width = $xoopsModuleConfig['max_image_height'] * ($imginfo[0] / $imginfo[1]);
             $pseudo_size = "width='".$pseudo_width."px' height='".$pseudo_height."px'";
         }
     }
@@ -112,20 +112,20 @@ 
         return false;
     }
 
-    $newWidth = (int) (min($imginfo[0],$thumb_width));
+    $newWidth = (int) (min($imginfo[0], $thumb_width));
     $newHeight = (int) ($imginfo[1] * $newWidth / $imginfo[0]);
 
     if ($xoopsModuleConfig['image_lib'] == 1 or $xoopsModuleConfig['image_lib'] == 0) {
-        if (preg_match("#[A-Z]:|\\\\#Ai",__FILE__)) {
+        if (preg_match("#[A-Z]:|\\\\#Ai", __FILE__)) {
             $cur_dir = __DIR__;
             $src_file_im = '"'.$cur_dir.'\\'.strtr($src_file, '/', '\\').'"';
             $new_file_im = '"'.$cur_dir.'\\'.strtr($new_file, '/', '\\').'"';
         } else {
-            $src_file_im =   @escapeshellarg($src_file);
-            $new_file_im =   @escapeshellarg($new_file);
+            $src_file_im = @escapeshellarg($src_file);
+            $new_file_im = @escapeshellarg($new_file);
         }
-        $path = empty($xoopsModuleConfig['path_magick'])?"":$xoopsModuleConfig['path_magick']."/";
-        $magick_command = $path . 'convert -quality 85 -antialias -sample ' . $newWidth . 'x' . $newHeight . ' ' . $src_file_im . ' +profile "*" ' . str_replace('\\', '/', $new_file_im) . '';
+        $path = empty($xoopsModuleConfig['path_magick']) ? "" : $xoopsModuleConfig['path_magick']."/";
+        $magick_command = $path.'convert -quality 85 -antialias -sample '.$newWidth.'x'.$newHeight.' '.$src_file_im.' +profile "*" '.str_replace('\\', '/', $new_file_im).'';
 
         @passthru($magick_command);
         if (file_exists($new_file)) {
@@ -134,13 +134,13 @@ 
     }
 
     if ($xoopsModuleConfig['image_lib'] == 2 or $xoopsModuleConfig['image_lib'] == 0) {
-        $path = empty($xoopsModuleConfig['path_netpbm'])?"":$xoopsModuleConfig['path_netpbm']."/";
+        $path = empty($xoopsModuleConfig['path_netpbm']) ? "" : $xoopsModuleConfig['path_netpbm']."/";
         if (eregi("\.png", $source)) {
-            $cmd = $path . "pngtopnm $src_file | ".$path . "pnmscale -xysize $newWidth $newHeight | ".$path . "pnmtopng > $new_file" ;
+            $cmd = $path."pngtopnm $src_file | ".$path."pnmscale -xysize $newWidth $newHeight | ".$path."pnmtopng > $new_file";
         } elseif (eregi("\.(jpg|jpeg)", $source)) {
-            $cmd = $path . "jpegtopnm $src_file | ".$path . "pnmscale -xysize $newWidth $newHeight | ".$path . "ppmtojpeg -quality=90 > $new_file" ;
+            $cmd = $path."jpegtopnm $src_file | ".$path."pnmscale -xysize $newWidth $newHeight | ".$path."ppmtojpeg -quality=90 > $new_file";
         } elseif (eregi("\.gif", $source)) {
-            $cmd = $path . "giftopnm $src_file | ".$path . "pnmscale -xysize $newWidth $newHeight | ppmquant 256 | ".$path . "ppmtogif > $new_file" ;
+            $cmd = $path."giftopnm $src_file | ".$path."pnmscale -xysize $newWidth $newHeight | ppmquant 256 | ".$path."ppmtogif > $new_file";
         }
 
         @exec($cmd, $output, $retval);
@@ -157,16 +157,16 @@ 
     if (function_exists('imagejpeg'))$supported_types[] = 2;
     if (function_exists('imagepng')) $supported_types[] = 3;
 
-    $imageCreateFunction = (function_exists('imagecreatetruecolor'))? "imagecreatetruecolor" : "imagecreate";
+    $imageCreateFunction = (function_exists('imagecreatetruecolor')) ? "imagecreatetruecolor" : "imagecreate";
 
-    if (in_array($type, $supported_types) ) {
+    if (in_array($type, $supported_types)) {
         switch ($type) {
             case 1 :
                 if (!function_exists('imagecreatefromgif')) return false;
-                $im = imagecreatefromgif ($src_file);
+                $im = imagecreatefromgif($src_file);
                 $new_im = imagecreate($newWidth, $newHeight);
                 imagecopyresized($new_im, $im, 0, 0, 0, 0, $newWidth, $newHeight, $imginfo[0], $imginfo[1]);
-                imagegif ($new_im, $new_file);
+                imagegif($new_im, $new_file);
                 imagedestroy($im);
                 imagedestroy($new_im);
                 break;

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;

include/comment_functions.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 function smartfaq_com_update($faq_id, $total_num)
 {
     $db = &XoopsDatabaseFactory::getDatabaseConnection();
-    $sql = 'UPDATE ' . $db->prefix('smartfaq_faq') . ' SET comments = ' . $total_num . ' WHERE faqid = ' . $faq_id;
+    $sql = 'UPDATE '.$db->prefix('smartfaq_faq').' SET comments = '.$total_num.' WHERE faqid = '.$faq_id;
     $db->query($sql);
 }
 

header.php

Spacing +2 added lines, -2 removed lines

@@ -7,9 +7,9 @@
 * Licence: GNU
 */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
 
-include_once XOOPS_ROOT_PATH . "/modules/smartfaq/include/functions.php";
+include_once XOOPS_ROOT_PATH."/modules/smartfaq/include/functions.php";
 include_once XOOPS_ROOT_PATH.'/modules/smartfaq/class/category.php';
 include_once XOOPS_ROOT_PATH.'/modules/smartfaq/class/faq.php';
 include_once XOOPS_ROOT_PATH.'/modules/smartfaq/class/answer.php';

comment_reply.php

Spacing +2 added lines, -2 removed lines

@@ -7,5 +7,5 @@
 * Licence: GNU
 */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
-include_once XOOPS_ROOT_PATH . '/include/comment_reply.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
+include_once XOOPS_ROOT_PATH.'/include/comment_reply.php';

comment_post.php

Spacing +2 added lines, -2 removed lines

@@ -7,5 +7,5 @@
 * Licence: GNU
 */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
-include_once XOOPS_ROOT_PATH . '/include/comment_post.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
+include_once XOOPS_ROOT_PATH.'/include/comment_post.php';

comment_edit.php

Spacing +2 added lines, -2 removed lines

@@ -7,5 +7,5 @@
 * Licence: GNU
 */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
-include_once XOOPS_ROOT_PATH . '/include/comment_edit.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
+include_once XOOPS_ROOT_PATH.'/include/comment_edit.php';

notification_update.php

Spacing +2 added lines, -2 removed lines

@@ -7,5 +7,5 @@
 * Licence: GNU
 */
 
-include dirname(dirname(__DIR__)) . '/mainfile.php';
-include XOOPS_ROOT_PATH . '/include/notification_update.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
+include XOOPS_ROOT_PATH.'/include/notification_update.php';

category.php

Spacing +22 added lines, -22 removed lines

@@ -7,12 +7,12 @@ 
 * Licence: GNU
 */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 $categoryid = isset($_GET['categoryid']) ? intval($_GET['categoryid']) : 0;
 
 // Creating the category handler object
-$category_handler =& sf_gethandler('category');
+$category_handler = & sf_gethandler('category');
 
 // Creating the category object for the selected category
 $categoryObj = new sfCategory($categoryid);
@@ -36,14 +36,14 @@ 
 }
 $xoopsOption['template_main'] = 'smartfaq_category.tpl';
 
-include_once(XOOPS_ROOT_PATH . "/header.php");
-include_once __DIR__ . '/footer.php';
+include_once(XOOPS_ROOT_PATH."/header.php");
+include_once __DIR__.'/footer.php';
 
 // At which record shall we start
 $start = isset($_GET['start']) ? intval($_GET['start']) : 0;
 
 // Creating the faq handler object
-$faq_handler =& sf_gethandler('faq');
+$faq_handler = & sf_gethandler('faq');
 
 // creating the FAQ objects that belong to the selected category
 if ($xoopsModuleConfig['orderbydate'] == 1) {
@@ -84,10 +84,10 @@ 
     $subcat_keys = array_keys($subcatsObj);
     foreach ($subcat_keys as $i) {
         $subcat_id = $subcatsObj[$i]->getVar('categoryid');
-        if (isset($totalQnas[$subcat_id]) && $totalQnas[$subcat_id] > 0 ) {
+        if (isset($totalQnas[$subcat_id]) && $totalQnas[$subcat_id] > 0) {
             if (isset($last_qnaObj[$subcat_id])) {
                 $subcatsObj[$i]->setVar('last_faqid', $last_qnaObj[$subcat_id]->getVar('faqid'));
-                $subcatsObj[$i]->setVar('last_question_link', "<a href='faq.php?faqid=" . $last_qnaObj[$subcat_id]->getVar('faqid') . "'>" . $last_qnaObj[$subcat_id]->question($lastfaqsize) . "</a>");
+                $subcatsObj[$i]->setVar('last_question_link', "<a href='faq.php?faqid=".$last_qnaObj[$subcat_id]->getVar('faqid')."'>".$last_qnaObj[$subcat_id]->question($lastfaqsize)."</a>");
             }
         }
         $subcatsObj[$i]->setVar('faqcount', $totalQnas[$subcat_id]);
@@ -98,15 +98,15 @@ 
     $xoopsTpl->assign('subcats', $subcats);
 }
 $thiscategory_faqcount = isset($totalQnas[$categoryid]) ? $totalQnas[$categoryid] : 0;
-$category['total'] = $thiscategory_faqcount + $total_faqs;
+$category['total'] = $thiscategory_faqcount+$total_faqs;
 
-if (count($faqsObj)>0) {
+if (count($faqsObj) > 0) {
     $userids = array();
     foreach ($faqsObj as $key => $thisfaq) {
         $faqids[] = $thisfaq->getVar('faqid');
         $userids[$thisfaq->uid()] = 1;
     }
-    $answer_handler =& sf_gethandler('answer');
+    $answer_handler = & sf_gethandler('answer');
     $allanswers = $answer_handler->getLastPublishedByFaq($faqids);
 
     foreach ($allanswers as $key => $thisanswer) {
@@ -120,7 +120,7 @@ 
         $faq = $faqsObj[$i]->toArray(null, $categoryObj);
 
         // Creating the answer object
-        $answerObj =& $allanswers[$faqsObj[$i]->faqid()];
+        $answerObj = & $allanswers[$faqsObj[$i]->faqid()];
 
         $answerObj->setVar('dohtml', $faqsObj[$i]->getVar('html'));
         $answerObj->setVar('doxcode', $faqsObj[$i]->getVar('xcodes'));
@@ -140,14 +140,14 @@ 
 
     if (isset($last_qnaObj) && $last_qnaObj) {
         $category['last_faqid'] = $last_qnaObj[$categoryObj->getVar('categoryid')]->getVar('faqid');
-        $category['last_question_link'] = "<a href='faq.php?faqid=" . $last_qnaObj[$categoryObj->getVar('categoryid')]->getVar('faqid') . "'>" . $last_qnaObj[$categoryObj->getVar('categoryid')]->question($lastfaqsize) . "</a>";
+        $category['last_question_link'] = "<a href='faq.php?faqid=".$last_qnaObj[$categoryObj->getVar('categoryid')]->getVar('faqid')."'>".$last_qnaObj[$categoryObj->getVar('categoryid')]->question($lastfaqsize)."</a>";
     }
 }
 
 $xoopsTpl->assign('whereInSection', $myts->displayTarea($xoopsModule->getVar('name')));
 $xoopsTpl->assign('displaylastfaqs', true);
 $xoopsTpl->assign('display_categoryname', true);
-$xoopsTpl->assign('displayFull', $xoopsModuleConfig['displaytype']=='full');
+$xoopsTpl->assign('displayFull', $xoopsModuleConfig['displaytype'] == 'full');
 
 // Language constants
 $xoopsTpl->assign('lang_index_faqs', _MD_SF_SMARTFAQS);
@@ -165,26 +165,26 @@ 
 $xoopsTpl->assign('lang_comments', _MD_SF_COMMENTS);
 
 // The Navigation Bar
-include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
-$pagenav = new XoopsPageNav($thiscategory_faqcount, $xoopsModuleConfig['indexperpage'], $start, 'start', 'categoryid=' . $categoryObj->getVar('categoryid'));
+include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
+$pagenav = new XoopsPageNav($thiscategory_faqcount, $xoopsModuleConfig['indexperpage'], $start, 'start', 'categoryid='.$categoryObj->getVar('categoryid'));
 if ($xoopsModuleConfig['useimagenavpage'] == 1) {
-    $xoopsTpl->assign('navbar', '<div style="text-align:right;">' . $pagenav->renderImageNav() . '</div>');
+    $xoopsTpl->assign('navbar', '<div style="text-align:right;">'.$pagenav->renderImageNav().'</div>');
 } else {
-    $xoopsTpl->assign('navbar', '<div style="text-align:right;">' . $pagenav->renderNav() . '</div>');
+    $xoopsTpl->assign('navbar', '<div style="text-align:right;">'.$pagenav->renderNav().'</div>');
 }
 
 $xoopsTpl->assign('category', $category);
 
 // Page Title Hack by marcan
 $module_name = $myts->htmlSpecialChars($xoopsModule->getVar('name'));
-$xoopsTpl->assign('xoops_pagetitle', $module_name . ' - ' . $category['name']);
+$xoopsTpl->assign('xoops_pagetitle', $module_name.' - '.$category['name']);
 // End Page Title Hack by marcan
 
 //code to include smartie
-if (file_exists(XOOPS_ROOT_PATH . '/modules/smarttie/smarttie_links.php')) {
-    include_once XOOPS_ROOT_PATH . '/modules/smarttie/smarttie_links.php';
-        $xoopsTpl->assign('smarttie',1);
+if (file_exists(XOOPS_ROOT_PATH.'/modules/smarttie/smarttie_links.php')) {
+    include_once XOOPS_ROOT_PATH.'/modules/smarttie/smarttie_links.php';
+        $xoopsTpl->assign('smarttie', 1);
 }
 //end code for smarttie
 
-include_once(XOOPS_ROOT_PATH . "/footer.php");
+include_once(XOOPS_ROOT_PATH."/footer.php");