mambax7 / smartfaq

comment_delete.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_delete.php';
+include dirname(dirname(__DIR__)).'/mainfile.php';
+include_once XOOPS_ROOT_PATH.'/include/comment_delete.php';

open_index.php

Spacing +20 added lines, -20 removed lines

@@ -7,28 +7,28 @@ 
 * Licence: GNU
 */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 global $xoopsConfig, $xoopsModuleConfig, $xoopsModule;
 
 // At which record shall we start for the Categories
-$catstart = isset($_GET['catstart'])? intval($_GET['catstart']) : 0;
+$catstart = isset($_GET['catstart']) ? intval($_GET['catstart']) : 0;
 
 // At which record shall we start for the FAQs
-$start = isset($_GET['start'])? intval($_GET['start']) : 0;
+$start = isset($_GET['start']) ? intval($_GET['start']) : 0;
 
 // Creating the category handler object
-$category_handler =& sf_gethandler('category');
+$category_handler = & sf_gethandler('category');
 
 // Get the total number of categories
 $totalCategories = count($category_handler->getCategories());
 
 // Creating the faq handler object
-$faq_handler =& sf_gethandler('faq');
+$faq_handler = & sf_gethandler('faq');
 
 // Total number of published FAQ in the module
 $totalFaqs = $faq_handler->getFaqsCount(-1, _SF_STATUS_OPENED);
-if ($totalFaqs  == 0) {
+if ($totalFaqs == 0) {
     redirect_header("request.php", 2, _MD_SF_NO_OPEN_QUESTION);
     exit;
 }
@@ -38,15 +38,15 @@ 
 
 // If no categories are found, exit
 $totalCategoriesOnPage = count($categoriesObj);
-if ($totalCategoriesOnPage  == 0) {
+if ($totalCategoriesOnPage == 0) {
     redirect_header("javascript:history.go(-1)", 2, _AM_SF_NO_CAT_EXISTS);
     exit;
 }
 
 $xoopsOption['template_main'] = 'smartfaq_index.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';
 
 //get all categories for future reference
 $allcategories = $category_handler->getObjects(null, true);
@@ -57,18 +57,18 @@ 
 $subcats = $category_handler->getSubCats($categoriesObj);
 $totalQnas = $category_handler->faqsCount(0, array(_SF_STATUS_OPENED));
 
-$faq_handler =& sf_gethandler('faq');
+$faq_handler = & sf_gethandler('faq');
 $last_qnaObj = $faq_handler->getLastPublishedByCat(array(_SF_STATUS_OPENED));
 
 foreach ($categoriesObj as $cat_id => $category) {
     $total = 0;
-    if (isset($subcats[$cat_id]) && count($subcats[$cat_id])>0) {
+    if (isset($subcats[$cat_id]) && count($subcats[$cat_id]) > 0) {
         foreach ($subcats[$cat_id] as $key => $subcat) {
             $subcat_id = $subcat->getVar('categoryid');
             if (isset($totalQnas[$subcat_id]) && $totalQnas[$subcat_id] > 0) {
                 if (isset($last_qnaObj[$subcat_id])) {
                     $subcat->setVar('last_faqid', $last_qnaObj[$subcat_id]->getVar('faqid'));
-                    $subcat->setVar('last_question_link', "<a href='faq.php?faqid=" . $last_qnaObj[$subcat_id]->getVar('faqid') . "'>" . $last_qnaObj[$subcat_id]->question(50) . "</a>");
+                    $subcat->setVar('last_question_link', "<a href='faq.php?faqid=".$last_qnaObj[$subcat_id]->getVar('faqid')."'>".$last_qnaObj[$subcat_id]->question(50)."</a>");
                 }
                 $subcat->setVar('faqcount', $totalQnas[$subcat_id]);
                 $categories[$cat_id]['subcats'][$subcat_id] = $subcat->toArray(null, true);
@@ -129,7 +129,7 @@ 
 $xoopsTpl->assign('lang_description', _MD_SF_DESCRIPTION);
 $xoopsTpl->assign('lang_category', _MD_SF_CATEGORY);
 $xoopsTpl->assign('sectionname', $moduleName);
-$xoopsTpl->assign('whereInSection', "<a href='index.php'>" . $moduleName . "</a> > " . _MD_SF_OPEN_SECTION);
+$xoopsTpl->assign('whereInSection', "<a href='index.php'>".$moduleName."</a> > "._MD_SF_OPEN_SECTION);
 
 $xoopsTpl->assign('displayFull', false);
 $xoopsTpl->assign('displaylastfaqs', $xoopsModuleConfig['displaylastfaqs']);
@@ -145,25 +145,25 @@ 
 $xoopsTpl->assign('lang_category', _MD_SF_CATEGORY);
 
 // Category Navigation Bar
-include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
+include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
 $pagenav = new XoopsPageNav($totalCategories, $xoopsModuleConfig['catperpage'], $catstart, 'catstart', '');
 if ($xoopsModuleConfig['useimagenavpage'] == 1) {
-    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">' . $pagenav->renderImageNav() . '</div>');
+    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">'.$pagenav->renderImageNav().'</div>');
 } else {
-    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">' . $pagenav->renderNav() . '</div>');
+    $xoopsTpl->assign('catnavbar', '<div style="text-align:right;">'.$pagenav->renderNav().'</div>');
 }
 
 // FAQ Navigation Bar
 $pagenav = new XoopsPageNav($totalFaqs, $xoopsModuleConfig['indexperpage'], $start, 'start', '');
 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>');
 }
 
 // Page Title Hack by marcan
 $module_name = $myts->htmlSpecialChars($xoopsModule->getVar('name'));
-$xoopsTpl->assign('xoops_pagetitle', $module_name . ' - ' . $category->getVar('name'));
+$xoopsTpl->assign('xoops_pagetitle', $module_name.' - '.$category->getVar('name'));
 // End Page Title Hack by marcan
 
-include_once(XOOPS_ROOT_PATH . "/footer.php");
+include_once(XOOPS_ROOT_PATH."/footer.php");

answer.php

Spacing +21 added lines, -21 removed lines

@@ -7,7 +7,7 @@ 
 * Licence: GNU
 */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 global $xoopsUser, $xoopsConfig, $xoopsModuleConfig, $xoopsModule;
 
@@ -24,8 +24,8 @@ 
 if (isset($_POST['op'])) $op = $_POST['op'];
 
 // Getting the faqid
-$faqid = isset($_GET['faqid'])? intval($_GET['faqid']) : 0;
-$faqid = isset($_POST['faqid'])? intval($_POST['faqid']) : $faqid;
+$faqid = isset($_GET['faqid']) ? intval($_GET['faqid']) : 0;
+$faqid = isset($_POST['faqid']) ? intval($_POST['faqid']) : $faqid;
 
 // If no FAQ is selected, exit
 if ($faqid == 0) {
@@ -34,10 +34,10 @@ 
 }
 
 // Creating the FAQ handler object
-$faq_handler =& sf_gethandler('faq');
+$faq_handler = & sf_gethandler('faq');
 
 // Creating the answer handler object
-$answer_handler =& sf_gethandler('answer');
+$answer_handler = & sf_gethandler('answer');
 
 switch ($op) {
     // The answer is posted
@@ -67,14 +67,14 @@ 
     }
 
     // Get the category object related to that FAQ
-    $categoryObj =& $faqObj->category();
+    $categoryObj = & $faqObj->category();
 
     // Create the answer object
     $newAnswerObj = $answer_handler->create();
 
     // Putting the values in the answer object
     $newAnswerObj->setVar('faqid', $faqObj->faqid());
-    $newAnswerObj->setVar('answer',$_POST['answer']);
+    $newAnswerObj->setVar('answer', $_POST['answer']);
     $newAnswerObj->setVar('uid', $uid);
 
     // Depending of the status of the FAQ, some values need to be set
@@ -127,14 +127,14 @@ 
     }
 
     // Storing the FAQ object in the database
-    if ( !$faqObj->store() ) {
-        redirect_header("javascript:history.go(-1)", 3, _MD_SF_SUBMIT_ERROR . sf_formatErrors($faqObj->getErrors()));
+    if (!$faqObj->store()) {
+        redirect_header("javascript:history.go(-1)", 3, _MD_SF_SUBMIT_ERROR.sf_formatErrors($faqObj->getErrors()));
         exit();
     }
 
     // Storing the answer object in the database
-    if ( !$newAnswerObj->store() ) {
-        redirect_header("javascript:history.go(-1)", 3, _MD_SF_SUBMIT_ERROR . sf_formatErrors($newAnswerObj->getErrors()));
+    if (!$newAnswerObj->store()) {
+        redirect_header("javascript:history.go(-1)", 3, _MD_SF_SUBMIT_ERROR.sf_formatErrors($newAnswerObj->getErrors()));
         exit();
     }
 
@@ -151,7 +151,7 @@ 
         case 2 :
         // Answer for an open question submitted, auto-approved; became Q&A, need approbation
         if (isset($_POST['notifypub']) && $_POST['notifypub'] == 1) {
-            include_once XOOPS_ROOT_PATH . '/include/notification_constants.php';
+            include_once XOOPS_ROOT_PATH.'/include/notification_constants.php';
             $notification_handler->subscribe('faq', $faqObj->faqid(), 'approved', XOOPS_NOTIFICATION_MODE_SENDONCETHENDELETE);
         }
         // Send notifications
@@ -161,7 +161,7 @@ 
         case 3 :
         // Answer submitted, needs approbation
         if (isset($_POST['notifypub']) && $_POST['notifypub'] == 1) {
-            include_once XOOPS_ROOT_PATH . '/include/notification_constants.php';
+            include_once XOOPS_ROOT_PATH.'/include/notification_constants.php';
             $notification_handler->subscribe('question', $newAnswerObj->answerid(), 'approved', XOOPS_NOTIFICATION_MODE_SENDONCETHENDELETE);
         }
         // Send notifications
@@ -175,7 +175,7 @@ 
         // New answer submitted for a published Q&A, need approbation
         // Send notifications
         if (isset($_POST['notifypub']) && $_POST['notifypub'] == 1) {
-            include_once XOOPS_ROOT_PATH . '/include/notification_constants.php';
+            include_once XOOPS_ROOT_PATH.'/include/notification_constants.php';
             $notification_handler->subscribe('faq', $newAnswerObj->answerid(), 'answer_approved', XOOPS_NOTIFICATION_MODE_SENDONCETHENDELETE);
         }
 
@@ -204,10 +204,10 @@ 
     }
 
     // Creating the category object that holds the selected FAQ
-    $categoryObj =& $faqObj->category();
+    $categoryObj = & $faqObj->category();
 
     // Creating the answer object
-    $answerObj =& $faqObj->answer();
+    $answerObj = & $faqObj->answer();
 
     // Check user permissions to access that category of the selected FAQ
     if (faqAccessGranted($faqObj) < 0) {
@@ -216,20 +216,20 @@ 
     }
 
     $xoopsOption['template_main'] = 'smartfaq_submit.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';
 
-    $name = ($xoopsUser)? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
+    $name = ($xoopsUser) ? (ucwords($xoopsUser->getVar("uname"))) : 'Anonymous';
 
     $moduleName = $myts->displayTarea($xoopsModule->getVar('name'));
     $xoopsTpl->assign('whereInSection', $moduleName);
     $xoopsTpl->assign('lang_submit', _MD_SF_SUBMITANSWER);
 
     $xoopsTpl->assign('lang_intro_title', sprintf(_MD_SF_SUBMITANSWERTO, ucwords($xoopsModule->name())));
-    $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY . "<b>$name</b>, " . _MD_SF_SUBMITANSWER_INTRO);
+    $xoopsTpl->assign('lang_intro_text', _MD_SF_GOODDAY."<b>$name</b>, "._MD_SF_SUBMITANSWER_INTRO);
 
     include_once 'include/answer.inc.php';
 
-    include_once XOOPS_ROOT_PATH . '/footer.php';
+    include_once XOOPS_ROOT_PATH.'/footer.php';
     break;
 }

faq.php

Spacing +11 added lines, -11 removed lines

@@ -7,7 +7,7 @@ 
 * Licence: GNU
 */
 
-include_once __DIR__ . '/header.php';
+include_once __DIR__.'/header.php';
 
 $faqid = isset($_GET['faqid']) ? intval($_GET['faqid']) : 0;
 
@@ -17,7 +17,7 @@ 
 }
 
 // Creating the FAQ handler object
-$faq_handler =& sf_gethandler('faq');
+$faq_handler = & sf_gethandler('faq');
 
 // Creating the FAQ object for the selected FAQ
 $faqObj = new sfFaq($faqid);
@@ -46,8 +46,8 @@ 
     $faqObj->updateCounter();
 }
 $xoopsOption['template_main'] = 'smartfaq_faq.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';
 
 $faq = $faqObj->toArray(null, $categoryObj, false);
 
@@ -82,7 +82,7 @@ 
 $xoopsTpl->assign('display_categoryname', false);
 
 $xoopsTpl->assign('xcodes', $faqObj->getVar('xcodes'));
-$xoopsTpl->assign('mail_link', 'mailto:?subject=' . sprintf(_MD_SF_INTARTICLE, $xoopsConfig['sitename']) . '&body=' . sprintf(_MD_SF_INTARTFOUND, $xoopsConfig['sitename']) . ':  ' . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . '/faq.php?faqid=' . $faqObj->getVar('faqid'));
+$xoopsTpl->assign('mail_link', 'mailto:?subject='.sprintf(_MD_SF_INTARTICLE, $xoopsConfig['sitename']).'&body='.sprintf(_MD_SF_INTARTFOUND, $xoopsConfig['sitename']).':  '.XOOPS_URL.'/modules/'.$xoopsModule->dirname().'/faq.php?faqid='.$faqObj->getVar('faqid'));
 $xoopsTpl->assign('lang_printerpage', _MD_SF_PRINTERFRIENDLY);
 $xoopsTpl->assign('lang_sendstory', _MD_SF_SENDSTORY);
 $xoopsTpl->assign('faqid', $faqObj->getVar('faqid'));
@@ -98,19 +98,19 @@ 
 
 // Page Title Hack by marcan
 $module_name = $myts->htmlSpecialChars($xoopsModule->getVar('name'));
-$xoopsTpl->assign('xoops_pagetitle', $module_name . ' - ' . $categoryObj->name() . ' - ' . $faq['question']);
+$xoopsTpl->assign('xoops_pagetitle', $module_name.' - '.$categoryObj->name().' - '.$faq['question']);
 // End Page Title Hack by marcan
 
 // Include the comments if the selected FAQ supports comments
 if ($faqObj->cancomment() == 1) {
-    include_once XOOPS_ROOT_PATH . "/include/comment_view.php";
+    include_once XOOPS_ROOT_PATH."/include/comment_view.php";
 }
 
 //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';

include/functions.php

Spacing +62 added lines, -62 removed lines

@@ -18,7 +18,7 @@ 
     if (!isset($smartModule)) {
         global $xoopsModule;
         if (isset($xoopsModule) && is_object($xoopsModule) && $xoopsModule->getVar('dirname') == 'smartfaq') {
-            $smartModule =& $xoopsModule;
+            $smartModule = & $xoopsModule;
         } else {
             $hModule = &xoops_gethandler('module');
             $smartModule = $hModule->getByDirname('smartfaq');
@@ -35,9 +35,9 @@ 
         global $xoopsModule;
         if (isset($xoopsModule) && is_object($xoopsModule) && $xoopsModule->getVar('dirname') == 'smartfaq') {
             global $xoopsModuleConfig;
-            $smartConfig =& $xoopsModuleConfig;
+            $smartConfig = & $xoopsModuleConfig;
         } else {
-            $smartModule =& sf_getModuleInfo();
+            $smartModule = & sf_getModuleInfo();
             $hModConfig = &xoops_gethandler('config');
             $smartConfig = $hModConfig->getConfigsByCat(0, $smartModule->getVar('mid'));
         }
@@ -48,14 +48,14 @@ 
 
 function sf_getHelpPath()
 {
-    $smartConfig =& sf_getModuleConfig();
+    $smartConfig = & sf_getModuleConfig();
     switch ($smartConfig['helppath_select']) {
         case 'docs.xoops.org' :
             return 'http://docs.xoops.org/help/sfaqh/index.htm';
         break;
 
         case 'inside' :
-            return XOOPS_URL . "/modules/smartfaq/doc/";
+            return XOOPS_URL."/modules/smartfaq/doc/";
         break;
 
         case 'custom' :
@@ -64,31 +64,31 @@ 
     }
 }
 
-function sf_formatErrors($errors=array())
+function sf_formatErrors($errors = array())
 {
     $ret = '';
     foreach ($errors as $key=>$value) {
-        $ret .= "<br /> - " . $value;
+        $ret .= "<br /> - ".$value;
     }
 
     return $ret;
 }
 
-function sf_addCategoryOption($categoryObj, $selectedid=0, $level = 0, $ret='')
+function sf_addCategoryOption($categoryObj, $selectedid = 0, $level = 0, $ret = '')
 {
     // Creating the category handler object
-    $category_handler =& sf_gethandler('category');
+    $category_handler = & sf_gethandler('category');
 
     $spaces = '';
     for ($j = 0; $j < $level; ++$j) {
         $spaces .= '--';
     }
 
-    $ret .= "<option value='" . $categoryObj->categoryid() . "'";
+    $ret .= "<option value='".$categoryObj->categoryid()."'";
     if ($selectedid == $categoryObj->categoryid()) {
         $ret .= " selected='selected'";
     }
-    $ret .= ">" . $spaces . $categoryObj->name() . "</option>\n";
+    $ret .= ">".$spaces.$categoryObj->name()."</option>\n";
 
     $subCategoriesObj = $category_handler->getCategories(0, 0, $categoryObj->categoryid());
     if (count($subCategoriesObj) > 0) {
@@ -101,16 +101,16 @@ 
     return $ret;
 }
 
-function sf_createCategorySelect($selectedid=0, $parentcategory=0, $allCatOption=true)
+function sf_createCategorySelect($selectedid = 0, $parentcategory = 0, $allCatOption = true)
 {
-    $ret = "" . _MB_SF_SELECTCAT . "&nbsp;<select name='options[]'>";
+    $ret = ""._MB_SF_SELECTCAT."&nbsp;<select name='options[]'>";
     if ($allCatOption) {
         $ret .= "<option value='0'";
-        $ret .= ">" . _MB_SF_ALLCAT . "</option>\n";
+        $ret .= ">"._MB_SF_ALLCAT."</option>\n";
     }
 
     // Creating the category handler object
-    $category_handler =& sf_gethandler('category');
+    $category_handler = & sf_gethandler('category');
 
     // Creating category objects
     $categoriesObj = $category_handler->getCategories(0, 0, $parentcategory);
@@ -125,7 +125,7 @@ 
     return $ret;
 }
 
-function sf_getStatusArray ()
+function sf_getStatusArray()
 {
     $result = array("1" => _AM_SF_STATUS1,
     "2" => _AM_SF_STATUS2,
@@ -139,14 +139,14 @@ 
     return $result;
 }
 
-function sf_moderator ()
+function sf_moderator()
 {
     global $xoopsUser;
 
     if (!$xoopsUser) {
         $result = false;
     } else {
-        $smartPermHandler =& xoops_getmodulehandler('permission', 'smartfaq');
+        $smartPermHandler = & xoops_getmodulehandler('permission', 'smartfaq');
 
         $categories = $smartPermHandler->getPermissions('moderation');
         if (count($categories) == 0) {
@@ -159,13 +159,13 @@ 
     return $result;
 }
 
-function sf_modFooter ()
+function sf_modFooter()
 {
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
 
-    $modfootertxt = "Module " . $smartModule->getInfo('name') . " - Version " . $smartModule->getInfo('version') . "";
+    $modfootertxt = "Module ".$smartModule->getInfo('name')." - Version ".$smartModule->getInfo('version')."";
 
-    $modfooter = "<a href='" . $smartModule->getInfo('support_site_url') . "' target='_blank'><img src='" . XOOPS_URL . "/modules/smartfaq/assets/images/sfcssbutton.gif' title='" . $modfootertxt . "' alt='" . $modfootertxt . "'/></a>";
+    $modfooter = "<a href='".$smartModule->getInfo('support_site_url')."' target='_blank'><img src='".XOOPS_URL."/modules/smartfaq/assets/images/sfcssbutton.gif' title='".$modfootertxt."' alt='".$modfootertxt."'/></a>";
 
     return $modfooter;
 }
@@ -183,7 +183,7 @@ 
 
     $result = false;
 
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
     $module_id = $smartModule->getVar('mid');
 
     if (!empty($xoopsUser)) {
@@ -216,10 +216,10 @@ 
     } else {
         $result = -1;
 
-        $groups = ($xoopsUser)? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
+        $groups = ($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
 
         $gperm_handler = &xoops_gethandler('groupperm');
-        $smartModule =& sf_getModuleInfo();
+        $smartModule = & sf_getModuleInfo();
         $module_id = $smartModule->getVar('mid');
 
         // Do we have access to the parent category
@@ -255,12 +255,12 @@ 
     Global $xoopsDB;
 
     $result = true;
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
     $module_id = $smartModule->getVar('mid');
 
     $gperm_handler = &xoops_gethandler('groupperm');
 
-    $sql = "SELECT faqid FROM " . $xoopsDB->prefix("smartfaq_faq") . " WHERE categoryid = '$categoryid' ";
+    $sql = "SELECT faqid FROM ".$xoopsDB->prefix("smartfaq_faq")." WHERE categoryid = '$categoryid' ";
     $result = $xoopsDB->query($sql);
 
     if (count($result) > 0) {
@@ -292,7 +292,7 @@ 
 function sf_saveItemPermissions($groups, $itemID)
 {
     $result = true;
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
     $module_id = $smartModule->getVar('mid');
 
     $gperm_handler = &xoops_gethandler('groupperm');
@@ -322,7 +322,7 @@ 
 function sf_saveCategory_Permissions($groups, $categoryid, $perm_name)
 {
     $result = true;
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
     $module_id = $smartModule->getVar('mid');
 
     $gperm_handler = &xoops_gethandler('groupperm');
@@ -351,7 +351,7 @@ 
 function sf_saveModerators($moderators, $categoryid)
 {
     $result = true;
-    $smartModule =& sf_getModuleInfo();
+    $smartModule = & sf_getModuleInfo();
     $module_id = $smartModule->getVar('mid');
 
     $gperm_handler = &xoops_gethandler('groupperm');
@@ -372,7 +372,7 @@ 
     $ret = array();
     global $xoopsDB;
 
-    $result = $xoopsDB->query("SELECT * FROM " . $xoopsDB->prefix("smartfaq_faq") . " WHERE faqid = '$faqid'");
+    $result = $xoopsDB->query("SELECT * FROM ".$xoopsDB->prefix("smartfaq_faq")." WHERE faqid = '$faqid'");
     $ret = $xoopsDB->fetcharray($result);
 
     return $ret;
@@ -386,30 +386,30 @@ 
 */
 
 // TODO : Move this to the sfFaq class
-function sf_getAdminLinks($faqid = 0, $open=false)
+function sf_getAdminLinks($faqid = 0, $open = false)
 {
     global $xoopsUser, $xoopsModule, $xoopsModuleConfig, $xoopsConfig;
     $adminLinks = '';
-    $modulePath = XOOPS_URL . "/modules/" . $xoopsModule->dirname() . "/";
-    $page = $open? 'question.php' : 'faq.php';
+    $modulePath = XOOPS_URL."/modules/".$xoopsModule->dirname()."/";
+    $page = $open ? 'question.php' : 'faq.php';
     if ($xoopsUser && $xoopsUser->isAdmin($xoopsModule->getVar('mid'))) {
         // Edit button
-        $adminLinks .= "<a href='" . $modulePath . "admin/$page?op=mod&amp;faqid=" . $faqid . "'><img src='" . $modulePath . "assets/images/links/edit.gif'" . " title='" . _MD_SF_EDIT . "' alt='" . _MD_SF_EDIT . "'/></a>";
+        $adminLinks .= "<a href='".$modulePath."admin/$page?op=mod&amp;faqid=".$faqid."'><img src='".$modulePath."assets/images/links/edit.gif'"." title='"._MD_SF_EDIT."' alt='"._MD_SF_EDIT."'/></a>";
         $adminLinks .= " ";
         // Delete button
-        $adminLinks .= "<a href='" . $modulePath . "admin/$page?op=del&amp;faqid=" . $faqid . "'><img src='" . $modulePath . "assets/images/links/delete.gif'" . " title='" . _MD_SF_DELETE . "' alt='" . _MD_SF_DELETE . "'/></a>";
+        $adminLinks .= "<a href='".$modulePath."admin/$page?op=del&amp;faqid=".$faqid."'><img src='".$modulePath."assets/images/links/delete.gif'"." title='"._MD_SF_DELETE."' alt='"._MD_SF_DELETE."'/></a>";
         $adminLinks .= " ";
     }
     // Print button
-    $adminLinks .= "<a href='" . $modulePath . "print.php?faqid=" . $faqid . "'><img src='" . $modulePath . "assets/images/links/print.gif' title='" . _MD_SF_PRINT . "' alt='" . _MD_SF_PRINT . "'/></a>";
+    $adminLinks .= "<a href='".$modulePath."print.php?faqid=".$faqid."'><img src='".$modulePath."assets/images/links/print.gif' title='"._MD_SF_PRINT."' alt='"._MD_SF_PRINT."'/></a>";
     $adminLinks .= " ";
     // Email button
-    $maillink = 'mailto:?subject=' . sprintf(_MD_SF_INTARTICLE, $xoopsConfig['sitename']) . '&amp;body=' . sprintf(_MD_SF_INTARTFOUND, $xoopsConfig['sitename']) . ':  ' . $modulePath . 'faq.php?faqid=' . $faqid;
-    $adminLinks .= "<a href=\"" . $maillink . "\"><img src='" . $modulePath . "assets/images/links/friend.gif' title='" . _MD_SF_MAIL . "' alt='" . _MD_SF_MAIL . "'/></a>";
+    $maillink = 'mailto:?subject='.sprintf(_MD_SF_INTARTICLE, $xoopsConfig['sitename']).'&amp;body='.sprintf(_MD_SF_INTARTFOUND, $xoopsConfig['sitename']).':  '.$modulePath.'faq.php?faqid='.$faqid;
+    $adminLinks .= "<a href=\"".$maillink."\"><img src='".$modulePath."assets/images/links/friend.gif' title='"._MD_SF_MAIL."' alt='"._MD_SF_MAIL."'/></a>";
     $adminLinks .= " ";
     // Submit New Answer button
     if (($xoopsModuleConfig['allownewanswer']) && (is_object($xoopsUser) || $xoopsModuleConfig['anonpost'])) {
-        $adminLinks .= "<a href='" . $modulePath . "answer.php?faqid=" . $faqid . "'><img src='" . $modulePath . "assets/images/links/newanswer.gif' title='" . _MD_SF_SUBMITANSWER . "' alt='" . _MD_SF_SUBMITANSWER . "'/></a>";
+        $adminLinks .= "<a href='".$modulePath."answer.php?faqid=".$faqid."'><img src='".$modulePath."assets/images/links/newanswer.gif' title='"._MD_SF_SUBMITANSWER."' alt='"._MD_SF_SUBMITANSWER."'/></a>";
         $adminLinks .= " ";
     }
 
@@ -439,7 +439,7 @@ 
             if (!isset($users[$userid])) {
                 return $GLOBALS['xoopsConfig']['anonymous'];
             }
-            $user =& $users[$userid];
+            $user = & $users[$userid];
         }
 
         if (is_object($user)) {
@@ -453,9 +453,9 @@ 
                 $fullname = $user->getVar('name');
             }
             if (!empty($fullname)) {
-                $linkeduser = "$fullname [<a href='" . XOOPS_URL . "/userinfo.php?uid=" . $userid . "'>" . $ts->htmlSpecialChars($username) . "</a>]";
+                $linkeduser = "$fullname [<a href='".XOOPS_URL."/userinfo.php?uid=".$userid."'>".$ts->htmlSpecialChars($username)."</a>]";
             } else {
-                $linkeduser = "<a href='" . XOOPS_URL . "/userinfo.php?uid=" . $userid . "'>" . ucwords($ts->htmlSpecialChars($username)) . "</a>";
+                $linkeduser = "<a href='".XOOPS_URL."/userinfo.php?uid=".$userid."'>".ucwords($ts->htmlSpecialChars($username))."</a>";
             }
 
             return $linkeduser;
@@ -479,18 +479,18 @@ 
     return $xurl;
 }
 
-function sf_adminMenu ($currentoption = 0, $breadcrumb = '')
+function sf_adminMenu($currentoption = 0, $breadcrumb = '')
 {
 
     /* Nice buttons styles */
     echo "
         <style type='text/css'>
         #buttontop { float:left; width:100%; background: #e7e7e7; font-size:93%; line-height:normal; border-top: 1px solid black; border-left: 1px solid black; border-right: 1px solid black; margin: 0; }
-        #buttonbar { float:left; width:100%; background: #e7e7e7 url('" . XOOPS_URL . "/modules/smartfaq/assets/images/bg.gif') repeat-x left bottom; font-size:93%; line-height:normal; border-left: 1px solid black; border-right: 1px solid black; margin-bottom: 12px; }
+        #buttonbar { float:left; width:100%; background: #e7e7e7 url('" . XOOPS_URL."/modules/smartfaq/assets/images/bg.gif') repeat-x left bottom; font-size:93%; line-height:normal; border-left: 1px solid black; border-right: 1px solid black; margin-bottom: 12px; }
         #buttonbar ul { margin:0; margin-top: 15px; padding:10px 10px 0; list-style:none; }
         #buttonbar li { display:inline; margin:0; padding:0; }
-        #buttonbar a { float:left; background:url('" . XOOPS_URL . "/modules/smartfaq/assets/images/left_both.gif') no-repeat left top; margin:0; padding:0 0 0 9px; border-bottom:1px solid #000; text-decoration:none; }
-        #buttonbar a span { float:left; display:block; background:url('" . XOOPS_URL . "/modules/smartfaq/assets/images/right_both.gif') no-repeat right top; padding:5px 15px 4px 6px; font-weight:bold; color:#765; }
+        #buttonbar a { float:left; background:url('" . XOOPS_URL."/modules/smartfaq/assets/images/left_both.gif') no-repeat left top; margin:0; padding:0 0 0 9px; border-bottom:1px solid #000; text-decoration:none; }
+        #buttonbar a span { float:left; display:block; background:url('" . XOOPS_URL."/modules/smartfaq/assets/images/right_both.gif') no-repeat right top; padding:5px 15px 4px 6px; font-weight:bold; color:#765; }
         /* Commented Backslash Hack hides rule from IE5-Mac \*/
         #buttonbar a span {float:none;}
         /* End IE5-Mac hack */
@@ -509,27 +509,27 @@ 
     $tblColors = Array();
     $tblColors[0] = $tblColors[1] = $tblColors[2] = $tblColors[3] = $tblColors[4] = $tblColors[5] = $tblColors[6] = $tblColors[7] = $tblColors[8] = '';
     $tblColors[$currentoption] = 'current';
-    if (file_exists(XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/language/' . $xoopsConfig['language'] . '/modinfo.php')) {
-        include_once XOOPS_ROOT_PATH . '/modules/smartfaq/language/' . $xoopsConfig['language'] . '/modinfo.php';
+    if (file_exists(XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/language/'.$xoopsConfig['language'].'/modinfo.php')) {
+        include_once XOOPS_ROOT_PATH.'/modules/smartfaq/language/'.$xoopsConfig['language'].'/modinfo.php';
     } else {
-        include_once XOOPS_ROOT_PATH . '/modules/smartfaq/english/modinfo.php';
+        include_once XOOPS_ROOT_PATH.'/modules/smartfaq/english/modinfo.php';
     }
 
     echo "<div id='buttontop'>";
     echo "<table style=\"width: 100%; padding: 0; \" cellspacing=\"0\"><tr>";
     //echo "<td style=\"width: 45%; font-size: 10px; text-align: left; color: #2F5376; padding: 0 6px; line-height: 18px;\"><a class=\"nobutton\" href=\"../../system/admin.php?fct=preferences&amp;op=showmod&amp;mod=" . $xoopsModule->getVar('mid') . "\">" . _AM_SF_OPTS . "</a> | <a href=\"import.php\">" . _AM_SF_IMPORT . "</a> | <a href=\"../index.php\">" . _AM_SF_GOMOD . "</a> | <a href=\"../help/index.html\" target=\"_blank\">" . _AM_SF_HELP . "</a> | <a href=\"about.php\">" . _AM_SF_ABOUT . "</a></td>";
-    echo "<td style='font-size: 10px; text-align: left; color: #2F5376; padding: 0 6px; line-height: 18px;'><a class='nobutton' href='" . XOOPS_URL . "/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=" . $xoopsModule->getVar('mid') . "'>" . _AM_SF_OPTS . "</a> | <a href='" . XOOPS_URL . "/modules/smartfaq/admin/import.php'>" . _AM_SF_IMPORT . "</a> | <a href='" . XOOPS_URL . "/modules/smartfaq/index.php'>" . _AM_SF_GOMOD . "</a> | <a href='" . XOOPS_URL . "/modules/system/admin.php?fct=modulesadmin&op=update&module=" . $xoopsModule->getVar('dirname') . "'>" . _AM_SF_UPDATE . "</a> | <a href='" . sf_getHelpPath() . "' target='_blank'>" . _AM_SF_HELP . "</a> | <a href='" . XOOPS_URL . "/modules/smartfaq/admin/about.php'>" . _AM_SF_ABOUT . "</a></td>";
-    echo "<td style='font-size: 10px; text-align: right; color: #2F5376; padding: 0 6px; line-height: 18px;'><b>" . $myts->displayTarea($xoopsModule->name()) . " " . _AM_SF_MODADMIN . "</b> " . $breadcrumb . "</td>";
+    echo "<td style='font-size: 10px; text-align: left; color: #2F5376; padding: 0 6px; line-height: 18px;'><a class='nobutton' href='".XOOPS_URL."/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=".$xoopsModule->getVar('mid')."'>"._AM_SF_OPTS."</a> | <a href='".XOOPS_URL."/modules/smartfaq/admin/import.php'>"._AM_SF_IMPORT."</a> | <a href='".XOOPS_URL."/modules/smartfaq/index.php'>"._AM_SF_GOMOD."</a> | <a href='".XOOPS_URL."/modules/system/admin.php?fct=modulesadmin&op=update&module=".$xoopsModule->getVar('dirname')."'>"._AM_SF_UPDATE."</a> | <a href='".sf_getHelpPath()."' target='_blank'>"._AM_SF_HELP."</a> | <a href='".XOOPS_URL."/modules/smartfaq/admin/about.php'>"._AM_SF_ABOUT."</a></td>";
+    echo "<td style='font-size: 10px; text-align: right; color: #2F5376; padding: 0 6px; line-height: 18px;'><b>".$myts->displayTarea($xoopsModule->name())." "._AM_SF_MODADMIN."</b> ".$breadcrumb."</td>";
     echo "</tr></table>";
     echo "</div>";
 
     echo "<div id='buttonbar'>";
     echo "<ul>";
-    echo "<li id='" . $tblColors[0] . "'><a href=\"" . XOOPS_URL . "/modules/smartfaq/admin/index.php\"><span>" . _AM_SF_INDEX . "</span></a></li>";
-    echo "<li id='" . $tblColors[1] . "'><a href=\"" . XOOPS_URL . "/modules/smartfaq/admin/category.php\"><span>" . _AM_SF_CATEGORIES . "</span></a></li>";
-    echo "<li id='" . $tblColors[2] . "'><a href=\"" . XOOPS_URL . "/modules/smartfaq/admin/faq.php\"><span>" . _AM_SF_SMARTFAQS . "</span></a></li>";
-    echo "<li id='" . $tblColors[3] . "'><a href=\"" . XOOPS_URL . "/modules/smartfaq/admin/question.php\"><span>" . _AM_SF_OPEN_QUESTIONS . "</span></a></li>";
-    echo "<li id='" . $tblColors[4] . "'><a href=\"" . XOOPS_URL . "/modules/smartfaq/admin/permissions.php\"><span>" . _AM_SF_PERMISSIONS . "</span></a></li>";
+    echo "<li id='".$tblColors[0]."'><a href=\"".XOOPS_URL."/modules/smartfaq/admin/index.php\"><span>"._AM_SF_INDEX."</span></a></li>";
+    echo "<li id='".$tblColors[1]."'><a href=\"".XOOPS_URL."/modules/smartfaq/admin/category.php\"><span>"._AM_SF_CATEGORIES."</span></a></li>";
+    echo "<li id='".$tblColors[2]."'><a href=\"".XOOPS_URL."/modules/smartfaq/admin/faq.php\"><span>"._AM_SF_SMARTFAQS."</span></a></li>";
+    echo "<li id='".$tblColors[3]."'><a href=\"".XOOPS_URL."/modules/smartfaq/admin/question.php\"><span>"._AM_SF_OPEN_QUESTIONS."</span></a></li>";
+    echo "<li id='".$tblColors[4]."'><a href=\"".XOOPS_URL."/modules/smartfaq/admin/permissions.php\"><span>"._AM_SF_PERMISSIONS."</span></a></li>";
     echo "</ul></div>";
 }
 
@@ -578,15 +578,15 @@ 
     //-->
     </script>
     <?php
-    echo "<h3 style=\"color: #2F5376; margin: 6px 0 0 0; \"><a href='#' onClick=\"toggle('" . $tablename . "'); toggleIcon('" . $iconname . "');\">";
+    echo "<h3 style=\"color: #2F5376; margin: 6px 0 0 0; \"><a href='#' onClick=\"toggle('".$tablename."'); toggleIcon('".$iconname."');\">";
 }
 
-function sf_gethandler($name, $optional = false )
+function sf_gethandler($name, $optional = false)
 {
     static $handlers;
     $name = strtolower(trim($name));
     if (!isset($handlers[$name])) {
-        if ( file_exists( $hnd_file = XOOPS_ROOT_PATH.'/modules/smartfaq/class/'.$name.'.php' ) ) {
+        if (file_exists($hnd_file = XOOPS_ROOT_PATH.'/modules/smartfaq/class/'.$name.'.php')) {
             require_once $hnd_file;
         }
         $class = 'sf'.ucfirst($name).'Handler';
@@ -594,10 +594,10 @@ 
             $handlers[$name] = new $class($GLOBALS['xoopsDB']);
         }
     }
-    if (!isset($handlers[$name]) && !$optional ) {
+    if (!isset($handlers[$name]) && !$optional) {
         trigger_error('Class <b>'.$class.'</b> does not exist<br />Handler Name: '.$name, E_USER_ERROR);
     }
     $false = false;
 
-    return isset($handlers[$name])? $handlers[$name] : $false;
+    return isset($handlers[$name]) ? $handlers[$name] : $false;
 }

include/searchform.php

Spacing +6 added lines, -6 removed lines

@@ -7,9 +7,9 @@ 
 * Licence: GNU
 */
 
-$categoryID = isset($categoryID)? intval($categoryID) : 0;
-$type = isset($type)? intval($type) : 3;
-$term = isset($term)? ($type) : '';
+$categoryID = isset($categoryID) ? intval($categoryID) : 0;
+$type = isset($type) ? intval($type) : 3;
+$term = isset($term) ? ($type) : '';
 
 $sform = new XoopsThemeForm(_MD_WB_SEARCHFORM, "searchform", "search.php");
 $sform->setExtra('enctype="multipart/form-data"');
@@ -20,12 +20,12 @@ 
 
 if ($xoopsModuleConfig['multicats'] == 1) {
     $searchcat = new XoopsFormSelect(_MD_WB_CATEGORY, 'categoryID', $categoryID);
-    $searchcat->addOption ("0", _MD_WB_ALLOFTHEM);
+    $searchcat->addOption("0", _MD_WB_ALLOFTHEM);
 
-    $resultcat = $xoopsDB->query ("SELECT categoryID, name FROM " . $xoopsDB->prefix ("wbcategories") . " ORDER BY categoryID");
+    $resultcat = $xoopsDB->query("SELECT categoryID, name FROM ".$xoopsDB->prefix("wbcategories")." ORDER BY categoryID");
 
     while (list($categoryID, $name) = $xoopsDB->fetchRow($resultcat)) {
-        $searchcat->addOption ("categoryID", "$categoryID : $name");
+        $searchcat->addOption("categoryID", "$categoryID : $name");
     }
     $sform->addElement($searchcat, true);
 }

include/answer.inc.php

Spacing +7 added lines, -7 removed lines

@@ -10,9 +10,9 @@ 
 
 global $_POST;
 
-include_once XOOPS_ROOT_PATH . "/class/xoopstree.php";
-include_once XOOPS_ROOT_PATH . "/class/xoopslists.php";
-include_once XOOPS_ROOT_PATH . "/class/xoopsformloader.php";
+include_once XOOPS_ROOT_PATH."/class/xoopstree.php";
+include_once XOOPS_ROOT_PATH."/class/xoopslists.php";
+include_once XOOPS_ROOT_PATH."/class/xoopsformloader.php";
 //include_once 'functions.php';
 
 $mytree = new XoopsTree($xoopsDB->prefix("smartfaq_categories"), "categoryid", "parentid");
@@ -32,10 +32,10 @@ 
            $options['width'] = '100%';
            $options['height'] = '200px';
        $answerEditor  = new XoopsFormEditor('', $xoopsModuleConfig['form_editorOptionsUser'], $options, $nohtml = false, $onfailure = 'textarea');
-           $editorTray->addElement($answerEditor,true );
+           $editorTray->addElement($answerEditor, true);
        } else {
        $answerEditor  = new XoopsFormDhtmlTextArea(_MD_SF_ANSWER_FAQ, 'answer', '', '100%', '100%');
-       $editorTray->addElement($answerEditor, true );
+       $editorTray->addElement($answerEditor, true);
    }
 $form->addElement($editorTray);
 
@@ -46,8 +46,8 @@ 
     $form->addElement($notify_checkbox);
 }
 
-if (($faqObj->status() == _SF_STATUS_PUBLISHED) || ($faqObj->status() == _SF_STATUS_NEW_ANSWER) ) {
-    $answerObj =& $faqObj->answer();
+if (($faqObj->status() == _SF_STATUS_PUBLISHED) || ($faqObj->status() == _SF_STATUS_NEW_ANSWER)) {
+    $answerObj = & $faqObj->answer();
     $form->addElement(new XoopsFormLabel(_MD_SF_ORIGINAL_ANSWER, $answerObj->answer()), false);
 }
 

include/functions.render.php

Spacing +5 added lines, -5 removed lines

@@ -53,7 +53,7 @@ 
     if ($br != 0) {
         $text = $myts->nl2Br($text);
     }
-    $text = $myts->codeConv($text, $xcode, $image);    // Ryuji_edit(2003-11-18)
+    $text = $myts->codeConv($text, $xcode, $image); // Ryuji_edit(2003-11-18)
 
     return $text;
 }
@@ -100,10 +100,10 @@ 
     // START hacked by irmtfan
     // to show text links instead of buttons - func_num_args()==2 => only when $image, $alt is set and optional $display not set
     global $xoopsModuleConfig;
-    if (func_num_args()== 2) {
+    if (func_num_args() == 2) {
         // overall setting
-        if ( !empty($xoopsModuleConfig['display_text_links']) ) {
-            $display=false;
+        if (!empty($xoopsModuleConfig['display_text_links'])) {
+            $display = false;
         }
         // if set for each link => overwrite $display
         if (isset($xoopsModuleConfig['display_text_each_link'][$image])) {
@@ -130,7 +130,7 @@ 
     }
 
     $icon_handler = NewbbIconHandler::instance();
-    $icon_handler->template =& $xoTheme->template;
+    $icon_handler->template = & $xoTheme->template;
     $icon_handler->init($xoopsConfig["language"]);
 
     return $icon_handler;

include/request.inc.php

Spacing +4 added lines, -4 removed lines

@@ -10,13 +10,13 @@
 
 global $_POST;
 
-include_once XOOPS_ROOT_PATH . "/class/xoopstree.php";
-include_once XOOPS_ROOT_PATH . "/class/xoopslists.php";
-include_once XOOPS_ROOT_PATH . "/class/xoopsformloader.php";
+include_once XOOPS_ROOT_PATH."/class/xoopstree.php";
+include_once XOOPS_ROOT_PATH."/class/xoopslists.php";
+include_once XOOPS_ROOT_PATH."/class/xoopsformloader.php";
 
 $form = new XoopsThemeForm(_MD_SF_REQUEST, "form", xoops_getenv('PHP_SELF'));
 // CATEGORY
-$mytree = new XoopsTree($xoopsDB->prefix("smartfaq_categories"), "categoryid" , "parentid");
+$mytree = new XoopsTree($xoopsDB->prefix("smartfaq_categories"), "categoryid", "parentid");
 ob_start();
 $form->addElement(new XoopsFormHidden('categoryid', ''));
 $mytree->makeMySelBox("name", "weight", '');