Inspection of "updates" - mambax7/smartfaq - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (c92e39)

by Michael

created 2017-02-26 23:33 UTC

Status

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -57,8 +57,8 @@
 block discarded – undo
             extract($arrQ, EXTR_PREFIX_ALL, 'q');
 
             echo "    <qandaentry ID=\"q$q_faqid\" Revision=\"$q_datesub\" Condition=\"$q_html $q_smiley $q_xcodes\" XrefLabel=\"$q_modulelink $q_contextpage\" Vendor=\""
-                 . getUserFullName($q_uid)
-                 . "\">\r\n";
+                    . getUserFullName($q_uid)
+                    . "\">\r\n";
             echo "      <question>\r\n";
             echo '        <para>' . encodeText($q_question) . "</para>\r\n";
             if (!empty($q_howdoi)) {

Please login to merge, or discard this patch.

Spacing +13 added lines, -13 removed lines patch added patch discarded remove patch

@@ -6,16 +6,16 @@  discard block
 block discarded – undo
  * Licence: GNU
  */
 
-include_once __DIR__ . '/../../../include/cp_header.php';
+include_once __DIR__.'/../../../include/cp_header.php';
 
-$op = 'go';//'start';
+$op = 'go'; //'start';
 
 if (isset($HTTP_POST_VARS['op']) && ($HTTP_POST_VARS['op'] === 'go')) {
     $op = $HTTP_POST_VARS['op'];
 }
 
 if ($op === 'start') {
-    include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+    include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
     xoops_cp_header();
 
@@ -44,15 +44,15 @@  discard block
 block discarded – undo
 
     echo "  <title>{module title}</title>\r\n";
 
-    $resultC = $xoopsDB->query('SELECT * FROM ' . $xoopsDB->prefix('smartfaq_categories'));
+    $resultC = $xoopsDB->query('SELECT * FROM '.$xoopsDB->prefix('smartfaq_categories'));
     while ($arrC = $xoopsDB->fetchArray($resultC)) {
         extract($arrC, EXTR_PREFIX_ALL, 'c');
 
         echo "  <qandadiv ID=\"c$c_categoryid\" Revision=\"$c_created\">\r\n";
-        echo '    <title>' . encodeText($c_name) . "</title>\r\n";
-        echo '    <para>' . encodeText($c_description) . "</para>\r\n";
+        echo '    <title>'.encodeText($c_name)."</title>\r\n";
+        echo '    <para>'.encodeText($c_description)."</para>\r\n";
 
-        $resultQ = $xoopsDB->query('select * from ' . $xoopsDB->prefix('smartfaq_faq') . " where categoryid=$c_categoryid");
+        $resultQ = $xoopsDB->query('select * from '.$xoopsDB->prefix('smartfaq_faq')." where categoryid=$c_categoryid");
         while ($arrQ = $xoopsDB->fetchArray($resultQ)) {
             extract($arrQ, EXTR_PREFIX_ALL, 'q');
 
@@ -60,27 +60,27 @@  discard block
 block discarded – undo
                  . getUserFullName($q_uid)
                  . "\">\r\n";
             echo "      <question>\r\n";
-            echo '        <para>' . encodeText($q_question) . "</para>\r\n";
+            echo '        <para>'.encodeText($q_question)."</para>\r\n";
             if (!empty($q_howdoi)) {
                 echo "        <note Conformance=\"howdoi\">\r\n";
                 echo "          <title>{'How do I' from language file}</title>\r\n";
-                echo '          <para>' . encodeText($q_howdoi) . "</para>\r\n";
+                echo '          <para>'.encodeText($q_howdoi)."</para>\r\n";
                 echo "        </note>\r\n";
             }
             if (!empty($q_diduno)) {
                 echo "        <note Conformance=\"diduno\">\r\n";
                 echo "          <title>{'Did you know' from language file}</title>\r\n";
-                echo '          <para>' . encodeText($q_diduno) . "</para>\r\n";
+                echo '          <para>'.encodeText($q_diduno)."</para>\r\n";
                 echo "        </note>\r\n";
             }
             echo "      </question>\r\n";
 
-            $resultA = $xoopsDB->query('select * from ' . $xoopsDB->prefix('smartfaq_answers') . " where answerid=$q_answerid");
+            $resultA = $xoopsDB->query('select * from '.$xoopsDB->prefix('smartfaq_answers')." where answerid=$q_answerid");
             while ($arrA = $xoopsDB->fetchArray($resultA)) {
                 extract($arrA, EXTR_PREFIX_ALL, 'a');
 
-                echo "      <answer ID=\"a$a_answerid\" Revision=\"$a_datesub\" Vendor=\"" . getUserFullName($a_uid) . "\">\r\n";
-                echo '        <para>' . encodeText($a_answer) . "</para>\r\n";
+                echo "      <answer ID=\"a$a_answerid\" Revision=\"$a_datesub\" Vendor=\"".getUserFullName($a_uid)."\">\r\n";
+                echo '        <para>'.encodeText($a_answer)."</para>\r\n";
                 echo "      </answer>\r\n";
             }
             $xoopsDB->freeRecordSet($resultA);

Please login to merge, or discard this patch.

admin/permissions.php 3 patches

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

@@ -59,14 +59,14 @@
 block discarded – undo
             echo $form_view->render();
         } else {
             echo "<img id='toptableicon' src="
-                 . XOOPS_URL
-                 . '/modules/'
-                 . $xoopsModule->dirname()
-                 . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
-                 . _AM_SF_PERMISSIONSVIEWMAN
-                 . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
-                 . _AM_SF_NOPERMSSET
-                 . '</span>';
+                    . XOOPS_URL
+                    . '/modules/'
+                    . $xoopsModule->dirname()
+                    . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
+                    . _AM_SF_PERMISSIONSVIEWMAN
+                    . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
+                    . _AM_SF_NOPERMSSET
+                    . '</span>';
         }
         echo '</div>';
 

Please login to merge, or discard this patch.

Switch Indentation +42 added lines, -42 removed lines patch added patch discarded remove patch

@@ -24,53 +24,53 @@
 block discarded – undo
 }
 
 switch ($op) {
-    case 'default':
-    default:
-        global $xoopsDB, $xoopsModule;
+        case 'default':
+        default:
+            global $xoopsDB, $xoopsModule;
 
-        $adminObject  = \Xmf\Module\Admin::getInstance();
-        xoops_cp_header();
-        $adminObject->displayNavigation(basename(__FILE__));
-        // View Categories permissions
-        $item_list_view = array();
-        $block_view     = array();
-        // echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
-        sf_collapsableBar('toptable', 'toptableicon');
+            $adminObject  = \Xmf\Module\Admin::getInstance();
+            xoops_cp_header();
+            $adminObject->displayNavigation(basename(__FILE__));
+            // View Categories permissions
+            $item_list_view = array();
+            $block_view     = array();
+            // echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
+            sf_collapsableBar('toptable', 'toptableicon');
 
-        $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
-        if ($xoopsDB->getRowsNum($result_view)) {
-            while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
-                $item_list_view['cid']   = $myrow_view['categoryid'];
-                $item_list_view['title'] = $myrow_view['name'];
-                $form_view               = new XoopsGroupPermForm('', $xoopsModule->getVar('mid'), 'category_read', "<img id='toptableicon' src="
-                                                                                                                    . XOOPS_URL
-                                                                                                                    . '/modules/'
-                                                                                                                    . $xoopsModule->dirname()
-                                                                                                                    . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
-                                                                                                                    . _AM_SF_PERMISSIONSVIEWMAN
-                                                                                                                    . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
-                                                                                                                    . _AM_SF_VIEW_CATS
-                                                                                                                    . '</span>', 'admin/permissions.php');
-                $block_view[]            = $item_list_view;
-                foreach ($block_view as $itemlists) {
-                    $form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
+            $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
+            if ($xoopsDB->getRowsNum($result_view)) {
+                while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
+                    $item_list_view['cid']   = $myrow_view['categoryid'];
+                    $item_list_view['title'] = $myrow_view['name'];
+                    $form_view               = new XoopsGroupPermForm('', $xoopsModule->getVar('mid'), 'category_read', "<img id='toptableicon' src="
+                                                                                                                        . XOOPS_URL
+                                                                                                                        . '/modules/'
+                                                                                                                        . $xoopsModule->dirname()
+                                                                                                                        . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
+                                                                                                                        . _AM_SF_PERMISSIONSVIEWMAN
+                                                                                                                        . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
+                                                                                                                        . _AM_SF_VIEW_CATS
+                                                                                                                        . '</span>', 'admin/permissions.php');
+                    $block_view[]            = $item_list_view;
+                    foreach ($block_view as $itemlists) {
+                        $form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
+                    }
                 }
+                echo $form_view->render();
+            } else {
+                echo "<img id='toptableicon' src="
+                     . XOOPS_URL
+                     . '/modules/'
+                     . $xoopsModule->dirname()
+                     . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
+                     . _AM_SF_PERMISSIONSVIEWMAN
+                     . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
+                     . _AM_SF_NOPERMSSET
+                     . '</span>';
             }
-            echo $form_view->render();
-        } else {
-            echo "<img id='toptableicon' src="
-                 . XOOPS_URL
-                 . '/modules/'
-                 . $xoopsModule->dirname()
-                 . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;"
-                 . _AM_SF_PERMISSIONSVIEWMAN
-                 . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
-                 . _AM_SF_NOPERMSSET
-                 . '</span>';
-        }
-        echo '</div>';
+            echo '</div>';
 
-        echo "<br>\n";
+            echo "<br>\n";
 }
 
 require_once __DIR__ . '/admin_footer.php';

Please login to merge, or discard this patch.

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

@@ -6,8 +6,8 @@  discard block
 block discarded – undo
  * Licence: GNU
  */
 
-require_once __DIR__ . '/admin_header.php';
-include_once XOOPS_ROOT_PATH . '/class/xoopsform/grouppermform.php';
+require_once __DIR__.'/admin_header.php';
+include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
 
 if (!sf_userIsAdmin()) {
     redirect_header('javascript:history.go(-1)', 1, _NOPERM);
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
     default:
         global $xoopsDB, $xoopsModule;
 
-        $adminObject  = \Xmf\Module\Admin::getInstance();
+        $adminObject = \Xmf\Module\Admin::getInstance();
         xoops_cp_header();
         $adminObject->displayNavigation(basename(__FILE__));
         // View Categories permissions
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
         // echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
         sf_collapsableBar('toptable', 'toptableicon');
 
-        $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
+        $result_view = $xoopsDB->query('SELECT categoryid, name FROM '.$xoopsDB->prefix('smartfaq_categories').' ');
         if ($xoopsDB->getRowsNum($result_view)) {
             while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
                 $item_list_view['cid']   = $myrow_view['categoryid'];
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
                                                                                                                     . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
                                                                                                                     . _AM_SF_VIEW_CATS
                                                                                                                     . '</span>', 'admin/permissions.php');
-                $block_view[]            = $item_list_view;
+                $block_view[] = $item_list_view;
                 foreach ($block_view as $itemlists) {
                     $form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
                 }
@@ -73,4 +73,4 @@  discard block
 block discarded – undo
         echo "<br>\n";
 }
 
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

Please login to merge, or discard this patch.

admin/menu.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -18,45 +18,45 @@
 block discarded – undo
 }
 $adminObject = \Xmf\Module\Admin::getInstance();
 
-$pathIcon32    = \Xmf\Module\Admin::menuIconPath('');
+$pathIcon32 = \Xmf\Module\Admin::menuIconPath('');
 //$pathModIcon32 = $moduleHelper->getModule()->getInfo('modicons32');
 
 $moduleHelper->loadLanguage('modinfo');
 
-$adminObject              = array();
+$adminObject = array();
 $i                      = 0;
 $adminmenu[$i]['title'] = _AM_MODULEADMIN_HOME;
 $adminmenu[$i]['link']  = 'admin/index.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/home.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/home.png';
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU1;
 $adminmenu[$i]['link']  = 'admin/main.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/manage.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/manage.png';
 
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU2;
 $adminmenu[$i]['link']  = 'admin/category.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/category.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/category.png';
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU3;
 $adminmenu[$i]['link']  = 'admin/faq.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/search.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/search.png';
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU4;
 $adminmenu[$i]['link']  = 'admin/question.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/faq.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/faq.png';
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU5;
 $adminmenu[$i]['link']  = 'admin/permissions.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/permissions.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/permissions.png';
 ++$i;
 $adminmenu[$i]['title'] = _MI_SF_ADMENU8;
 $adminmenu[$i]['link']  = 'admin/import.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/download.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/download.png';
 ++$i;
 $adminmenu[$i]['title'] = _AM_MODULEADMIN_ABOUT;
 $adminmenu[$i]['link']  = 'admin/about.php';
-$adminmenu[$i]['icon']  = $pathIcon32 . '/about.png';
+$adminmenu[$i]['icon']  = $pathIcon32.'/about.png';
 //++$i;
 //$adminmenu[$i]['title'] = _AM_MODULEADMIN_ABOUT;
 //$adminmenu[$i]["link"]  = "admin/about2.php";

Please login to merge, or discard this patch.

admin/wffaq.php 2 patches

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

@@ -10,7 +10,7 @@  discard block
 block discarded – undo
  *
  */
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 
 $importFromModuleName = 'WF-FAQ';
 $scriptname           = 'wffaq.php';
@@ -22,32 +22,32 @@  discard block
 block discarded – undo
 }
 
 if ($op === 'start') {
-    include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+    include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
     xoops_cp_header();
-    $result = $xoopsDB->query('SELECT count(*) FROM ' . $xoopsDB->prefix('faqcategories'));
+    $result = $xoopsDB->query('SELECT count(*) FROM '.$xoopsDB->prefix('faqcategories'));
     list($totalCat) = $xoopsDB->fetchRow($result);
 
     sf_collapsableBar('bottomtable', 'bottomtableicon');
-    echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;" . sprintf(_AM_SF_IMPORT_FROM,
-                                                                                                                                                                $importFromModuleName) . '</h3>';
+    echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a>&nbsp;".sprintf(_AM_SF_IMPORT_FROM,
+                                                                                                                                                                $importFromModuleName).'</h3>';
     echo "<div id='bottomtable'>";
 
     if ($totalCat == 0) {
-        echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_NO_CATEGORY . '</span>';
+        echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_NO_CATEGORY.'</span>';
     } else {
-        include_once XOOPS_ROOT_PATH . '/class/xoopstree.php';
+        include_once XOOPS_ROOT_PATH.'/class/xoopstree.php';
 
-        $result = $xoopsDB->query('SELECT count(*) FROM ' . $xoopsDB->prefix('faqtopics'));
+        $result = $xoopsDB->query('SELECT count(*) FROM '.$xoopsDB->prefix('faqtopics'));
         list($totalFAQ) = $xoopsDB->fetchRow($result);
 
         if ($totalFAQ == 0) {
-            echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat) . '</span>';
+            echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'.sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat).'</span>';
         } else {
             echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'
                  . sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
                  . '</span>';
 
-            $form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL . '/modules/smartfaq/admin/' . $scriptname);
+            $form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL.'/modules/smartfaq/admin/'.$scriptname);
 
             // Categories to be imported
             $cat_cbox = new XoopsFormCheckBox(sprintf(_AM_SF_IMPORT_CATEGORIES, $importFromModuleName), 'import_category', -1);
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
             ob_end_clean();
 
             // Auto-Approve
-            $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_AUTOAPPROVE, 'autoaprove', 1, ' ' . _AM_SF_YES . '', ' ' . _AM_SF_NO . ''));
+            $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_AUTOAPPROVE, 'autoaprove', 1, ' '._AM_SF_YES.'', ' '._AM_SF_NO.''));
 
             // Submitted and answered by
             $memberHandler = xoops_getHandler('member');
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
             $form->addElement($user_select);
 
             // Q&As can be commented?
-            $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_ALLOWCOMMENTS, 'cancomment', 1, ' ' . _AM_SF_YES . '', ' ' . _AM_SF_NO . ''));
+            $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_ALLOWCOMMENTS, 'cancomment', 1, ' '._AM_SF_YES.'', ' '._AM_SF_NO.''));
             $group_list      = $memberHandler->getGroupList();
             $groups_selected = array();
             $groups_checkbox = new XoopsFormCheckBox(_AM_SF_IMPORT_PERMISSIONS, 'groups_read');
@@ -111,14 +111,14 @@  discard block
 block discarded – undo
         redirect_header($scriptname, 2, _AM_SF_NOCATSELECTED);
     }
 
-    require_once __DIR__ . '/admin_header.php';
+    require_once __DIR__.'/admin_header.php';
     xoops_cp_header();
 
     sf_collapsableBar('bottomtable', 'bottomtableicon');
-    echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;" . sprintf(_AM_SF_IMPORT_FROM,
-                                                                                                                                                                $importFromModuleName) . '</h3>';
+    echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a>&nbsp;".sprintf(_AM_SF_IMPORT_FROM,
+                                                                                                                                                                $importFromModuleName).'</h3>';
     echo "<div id='bottomtable'>";
-    echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_RESULT . '</span>';
+    echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_RESULT.'</span>';
 
     $cnt_imported_cat = 0;
     $cnt_imported_faq = 0;
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
     echo "Can Comment: $cancomment<br>";
     echo "Auto aprove: $autoaprove<br>";*/
 
-    $resultCat = $xoopsDB->query('select * from ' . $xoopsDB->prefix('faqcategories') . " where catID in ($import_category_list) order by weight");
+    $resultCat = $xoopsDB->query('select * from '.$xoopsDB->prefix('faqcategories')." where catID in ($import_category_list) order by weight");
 
     while ($arrCat = $xoopsDB->fetchArray($resultCat)) {
         extract($arrCat, EXTR_PREFIX_ALL, 'wfc');
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
         $categoryObj->setVar('description', $wfc_description);
 
         if (!$categoryObj->store(false)) {
-            echo sprintf(_AM_SF_IMPORT_CATEGORY_ERROR, $xcat_name) . '<br>';
+            echo sprintf(_AM_SF_IMPORT_CATEGORY_ERROR, $xcat_name).'<br>';
             continue;
         }
 
@@ -169,9 +169,9 @@  discard block
 block discarded – undo
 
         ++$cnt_imported_cat;
 
-        echo sprintf(_AM_SF_IMPORT_CATEGORY_SUCCESS, $wfc_name) . "<br\>";
+        echo sprintf(_AM_SF_IMPORT_CATEGORY_SUCCESS, $wfc_name)."<br\>";
 
-        $resultFAQ = $xoopsDB->query('select * from ' . $xoopsDB->prefix('faqtopics') . " where catID=$wfc_catID order by weight");
+        $resultFAQ = $xoopsDB->query('select * from '.$xoopsDB->prefix('faqtopics')." where catID=$wfc_catID order by weight");
         while ($arrFAQ = $xoopsDB->fetchArray($resultFAQ)) {
             extract($arrFAQ, EXTR_PREFIX_ALL, 'wft');
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
             $faqObj->setVar('exacturl', 0);
 
             if (!$faqObj->store(false)) {
-                echo sprintf('  ' . _AM_SF_IMPORT_FAQ_ERROR, $wft_question) . '<br>';
+                echo sprintf('  '._AM_SF_IMPORT_FAQ_ERROR, $wft_question).'<br>';
                 continue;
             } else {
                 $answerObj->setVar('faqid', $faqObj->faqid());
@@ -212,10 +212,10 @@  discard block
 block discarded – undo
                 $answerObj->setVar('status', _SF_AN_STATUS_APPROVED);
 
                 if (!$answerObj->store()) {
-                    echo sprintf('  ' . _AM_SF_IMPORT_FAQ_ERROR) . '<br>';
+                    echo sprintf('  '._AM_SF_IMPORT_FAQ_ERROR).'<br>';
                     continue;
                 } else {
-                    echo '&nbsp;&nbsp;' . sprintf(_AM_SF_IMPORTED_QUESTION, $faqObj->question(50)) . '<br>';
+                    echo '&nbsp;&nbsp;'.sprintf(_AM_SF_IMPORTED_QUESTION, $faqObj->question(50)).'<br>';
                     ++$cnt_imported_faq;
                 }
             }
@@ -225,8 +225,8 @@  discard block
 block discarded – undo
     }
 
     echo 'Done.<br>';
-    echo sprintf(_AM_SF_IMPORTED_CATEGORIES, $cnt_imported_cat) . '<br>';
-    echo sprintf(_AM_SF_IMPORTED_QUESTIONS, $cnt_imported_faq) . '<br>';
+    echo sprintf(_AM_SF_IMPORTED_CATEGORIES, $cnt_imported_cat).'<br>';
+    echo sprintf(_AM_SF_IMPORTED_QUESTIONS, $cnt_imported_faq).'<br>';
 
     exit();
 }

Please login to merge, or discard this patch.

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -46,8 +46,8 @@
 block discarded – undo
             echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat) . '</span>';
         } else {
             echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'
-                 . sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
-                 . '</span>';
+                    . sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
+                    . '</span>';
 
             $form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL . '/modules/smartfaq/admin/' . $scriptname);
 

Please login to merge, or discard this patch.

admin/admin_footer.php 2 patches

Indentation +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -19,12 +19,12 @@
 block discarded – undo
 
 $pathIcon32      = \Xmf\Module\Admin::iconUrl('', 32);
 echo "<div class='adminfooter'>\n"
-     . "  <div style='text-align: center;'>\n"
-     . "    <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
-     . "  </div>\n"
-     . '  '
-     . _AM_MODULEADMIN_ADMIN_FOOTER
-     . "\n"
-     . '</div>';
+        . "  <div style='text-align: center;'>\n"
+        . "    <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
+        . "  </div>\n"
+        . '  '
+        . _AM_MODULEADMIN_ADMIN_FOOTER
+        . "\n"
+        . '</div>';
 
 xoops_cp_footer();

Please login to merge, or discard this patch.

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -17,7 +17,7 @@
 block discarded – undo
  * @author       XOOPS Development Team
  */
 
-$pathIcon32      = \Xmf\Module\Admin::iconUrl('', 32);
+$pathIcon32 = \Xmf\Module\Admin::iconUrl('', 32);
 echo "<div class='adminfooter'>\n"
      . "  <div style='text-align: center;'>\n"
      . "    <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"

Please login to merge, or discard this patch.

admin/admin_header.php 1 patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

@@ -17,7 +17,7 @@  discard block
 block discarded – undo
  * @author       XOOPS Development Team
  */
 
-include_once __DIR__ . '/../../../include/cp_header.php';
+include_once __DIR__.'/../../../include/cp_header.php';
 //include_once $GLOBALS['xoops']->path('www/class/xoopsformloader.php');
 
 //require __DIR__ . '/../class/utility.php';
@@ -28,16 +28,16 @@  discard block
 block discarded – undo
 }
 
 $path = dirname(dirname(dirname(__DIR__)));
-include_once $path . '/kernel/module.php';
-include_once $path . '/class/xoopstree.php';
-include_once $path . '/class/xoopslists.php';
-include_once $path . '/class/pagenav.php';
-include_once $path . '/class/xoopsformloader.php';
+include_once $path.'/kernel/module.php';
+include_once $path.'/class/xoopstree.php';
+include_once $path.'/class/xoopslists.php';
+include_once $path.'/class/pagenav.php';
+include_once $path.'/class/xoopsformloader.php';
 
-include_once __DIR__ . '/../include/functions.php';
-include_once __DIR__ . '/../class/category.php';
-include_once __DIR__ . '/../class/faq.php';
-include_once __DIR__ . '/../class/answer.php';
+include_once __DIR__.'/../include/functions.php';
+include_once __DIR__.'/../class/category.php';
+include_once __DIR__.'/../class/faq.php';
+include_once __DIR__.'/../class/answer.php';
 //require __DIR__ . '/../class/utility.php';
 //require_once __DIR__ . '/../include/common.php';
 

Please login to merge, or discard this patch.

admin/import.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -6,7 +6,7 @@  discard block
 block discarded – undo
  * Licence: GNU
  */
 
-require_once __DIR__ . '/admin_header.php';
+require_once __DIR__.'/admin_header.php';
 
 $op = 'none';
 
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
     case 'importExecute':
 
         $importfile      = isset($_POST['importfile']) ? $_POST['importfile'] : 'nonselected';
-        $importfile_path = XOOPS_ROOT_PATH . '/modules/smartfaq/admin/' . $importfile . '.php';
+        $importfile_path = XOOPS_ROOT_PATH.'/modules/smartfaq/admin/'.$importfile.'.php';
         if (!file_exists($importfile_path)) {
             $errs[] = sprintf(_AM_SF_IMPORT_FILE_NOT_FOUND, $importfile_path);
             $error  = true;
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
             include_once $importfile_path;
         }
         foreach ($msgs as $m) {
-            echo $m . '<br>';
+            echo $m.'<br>';
         }
         echo '<br>';
         $endMsg = _AM_SF_IMPORT_SUCCESS;
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
 
         echo $endMsg;
         echo '<br><br>';
-        echo "<a href='import.php'>" . _AM_SF_IMPORT_BACK . '</a>';
+        echo "<a href='import.php'>"._AM_SF_IMPORT_BACK.'</a>';
         echo '<br><br>';
         break;
 
@@ -53,13 +53,13 @@  discard block
 block discarded – undo
         xoops_cp_header();
 
         sf_collapsableBar('bottomtable', 'bottomtableicon');
-        echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a>&nbsp;" . _AM_SF_IMPORT_TITLE . '</h3>';
+        echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a>&nbsp;"._AM_SF_IMPORT_TITLE.'</h3>';
         echo "<div id='bottomtable'>";
-        echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_INFO . '</span>';
+        echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_INFO.'</span>';
 
         global $xoopsUser, $xoopsUser, $xoopsConfig, $xoopsDB, $modify, $xoopsModuleConfig, $xoopsModule, $XOOPS_URL, $myts;
 
-        include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+        include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
 
         /** @var XoopsModuleHandler $moduleHandler */
         $moduleHandler = xoops_getHandler('module');
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
             $sform->display();
             unset($hidden);
         } else {
-            echo '<span style="color: #567; margin: 3px 0 12px 0; font-weight: bold; font-size: small; display: block; ">' . _AM_SF_IMPORT_NO_MODULE . '</span>';
+            echo '<span style="color: #567; margin: 3px 0 12px 0; font-weight: bold; font-size: small; display: block; ">'._AM_SF_IMPORT_NO_MODULE.'</span>';
         }
 
         // End of collapsable bar
@@ -108,4 +108,4 @@  discard block
 block discarded – undo
         break;
 }
 
-require_once __DIR__ . '/admin_footer.php';
+require_once __DIR__.'/admin_footer.php';

Please login to merge, or discard this patch.

blocks/faqs_recent_questions.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@  discard block
 block discarded – undo
 
 function b_faqs_recent_questions_show($options)
 {
-    include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
+    include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
 
     $block = array();
 
@@ -60,44 +60,44 @@  discard block
 block discarded – undo
  */
 function b_faqs_recent_questions_edit($options)
 {
-    include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
+    include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
 
     $form = sf_createCategorySelect($options[0]);
 
-    $form .= '&nbsp;<br>' . _MB_SF_ORDER . "&nbsp;<select name='options[]'>";
+    $form .= '&nbsp;<br>'._MB_SF_ORDER."&nbsp;<select name='options[]'>";
 
     $form .= "<option value='datesub'";
     if ($options[1] === 'datesub') {
         $form .= ' selected';
     }
-    $form .= '>' . _MB_SF_DATE . "</option>\n";
+    $form .= '>'._MB_SF_DATE."</option>\n";
 
     $form .= "<option value='counter'";
     if ($options[1] === 'counter') {
         $form .= ' selected';
     }
-    $form .= '>' . _MB_SF_HITS . "</option>\n";
+    $form .= '>'._MB_SF_HITS."</option>\n";
 
     $form .= "<option value='weight'";
     if ($options[1] === 'weight') {
         $form .= ' selected';
     }
-    $form .= '>' . _MB_SF_WEIGHT . "</option>\n";
+    $form .= '>'._MB_SF_WEIGHT."</option>\n";
 
     $form .= "</select>\n";
 
-    $form .= '&nbsp;' . _MB_SF_DISP . "&nbsp;<input type='text' name='options[]' value='" . $options[2] . "' />&nbsp;" . _MB_SF_QUESTIONS . '';
-    $form .= '&nbsp;<br>' . _MB_SF_CHARS . "&nbsp;<input type='text' name='options[]' value='" . $options[3] . "' />&nbsp;" . _MB_SF_LENGTH . '';
+    $form .= '&nbsp;'._MB_SF_DISP."&nbsp;<input type='text' name='options[]' value='".$options[2]."' />&nbsp;"._MB_SF_QUESTIONS.'';
+    $form .= '&nbsp;<br>'._MB_SF_CHARS."&nbsp;<input type='text' name='options[]' value='".$options[3]."' />&nbsp;"._MB_SF_LENGTH.'';
 
-    $form .= '<br>' . _MB_SF_SHOW_DATE . "&nbsp;<input type='radio' id='options[]' name='options[]' value='1'";
+    $form .= '<br>'._MB_SF_SHOW_DATE."&nbsp;<input type='radio' id='options[]' name='options[]' value='1'";
     if ($options[4] == 1) {
         $form .= ' checked';
     }
-    $form .= ' />&nbsp;' . _YES . "<input type='radio' id='options[]' name='options[]' value='0'";
+    $form .= ' />&nbsp;'._YES."<input type='radio' id='options[]' name='options[]' value='0'";
     if ($options[4] == 0) {
         $form .= ' checked';
     }
-    $form .= ' />&nbsp;' . _NO . '';
+    $form .= ' />&nbsp;'._NO.'';
 
     return $form;
 }

Please login to merge, or discard this patch.

blocks/faqs_context.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -11,7 +11,7 @@  discard block
 block discarded – undo
 
 function b_faqs_context_show($options)
 {
-    include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
+    include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
 
     $block = array();
 
@@ -48,8 +48,8 @@  discard block
 block discarded – undo
  */
 function b_faqs_context_edit($options)
 {
-    $form = '' . _MB_SF_DISP . '&nbsp;';
-    $form .= "<input type='text' name='options[]' value='" . $options[0] . "' />&nbsp;" . _MB_SF_FAQS . '';
+    $form = ''._MB_SF_DISP.'&nbsp;';
+    $form .= "<input type='text' name='options[]' value='".$options[0]."' />&nbsp;"._MB_SF_FAQS.'';
 
     return $form;
 }

Please login to merge, or discard this patch.

		@@ -11,7 +11,7 @@ discard block
		block discarded – undo
11	11
12	12	function b_faqs_recent_questions_show($options)
13	13	{
14		- include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
	14	+ include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
15	15
16	16	$block = array();
17	17
		@@ -60,44 +60,44 @@ discard block
		block discarded – undo
60	60	*/
61	61	function b_faqs_recent_questions_edit($options)
62	62	{
63		- include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
	63	+ include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
64	64
65	65	$form = sf_createCategorySelect($options[0]);
66	66
67		- $form .= ' <br>' . _MB_SF_ORDER . " <select name='options[]'>";
	67	+ $form .= ' <br>'._MB_SF_ORDER." <select name='options[]'>";
68	68
69	69	$form .= "<option value='datesub'";
70	70	if ($options[1] === 'datesub') {
71	71	$form .= ' selected';
72	72	}
73		- $form .= '>' . _MB_SF_DATE . "</option>\n";
	73	+ $form .= '>'._MB_SF_DATE."</option>\n";
74	74
75	75	$form .= "<option value='counter'";
76	76	if ($options[1] === 'counter') {
77	77	$form .= ' selected';
78	78	}
79		- $form .= '>' . _MB_SF_HITS . "</option>\n";
	79	+ $form .= '>'._MB_SF_HITS."</option>\n";
80	80
81	81	$form .= "<option value='weight'";
82	82	if ($options[1] === 'weight') {
83	83	$form .= ' selected';
84	84	}
85		- $form .= '>' . _MB_SF_WEIGHT . "</option>\n";
	85	+ $form .= '>'._MB_SF_WEIGHT."</option>\n";
86	86
87	87	$form .= "</select>\n";
88	88
89		- $form .= ' ' . _MB_SF_DISP . " <input type='text' name='options[]' value='" . $options[2] . "' /> " . _MB_SF_QUESTIONS . '';
90		- $form .= ' <br>' . _MB_SF_CHARS . " <input type='text' name='options[]' value='" . $options[3] . "' /> " . _MB_SF_LENGTH . '';
	89	+ $form .= ' '._MB_SF_DISP." <input type='text' name='options[]' value='".$options[2]."' /> "._MB_SF_QUESTIONS.'';
	90	+ $form .= ' <br>'._MB_SF_CHARS." <input type='text' name='options[]' value='".$options[3]."' /> "._MB_SF_LENGTH.'';
91	91
92		- $form .= '<br>' . _MB_SF_SHOW_DATE . " <input type='radio' id='options[]' name='options[]' value='1'";
	92	+ $form .= '<br>'._MB_SF_SHOW_DATE." <input type='radio' id='options[]' name='options[]' value='1'";
93	93	if ($options[4] == 1) {
94	94	$form .= ' checked';
95	95	}
96		- $form .= ' /> ' . _YES . "<input type='radio' id='options[]' name='options[]' value='0'";
	96	+ $form .= ' /> '._YES."<input type='radio' id='options[]' name='options[]' value='0'";
97	97	if ($options[4] == 0) {
98	98	$form .= ' checked';
99	99	}
100		- $form .= ' /> ' . _NO . '';
	100	+ $form .= ' /> '._NO.'';
101	101
102	102	return $form;
103	103	}

		@@ -57,8 +57,8 @@
		block discarded – undo
57	57	extract($arrQ, EXTR_PREFIX_ALL, 'q');
58	58
59	59	echo " <qandaentry ID=\"q$q_faqid\" Revision=\"$q_datesub\" Condition=\"$q_html $q_smiley $q_xcodes\" XrefLabel=\"$q_modulelink $q_contextpage\" Vendor=\""
60		- . getUserFullName($q_uid)
61		- . "\">\r\n";
	60	+ . getUserFullName($q_uid)
	61	+ . "\">\r\n";
62	62	echo " <question>\r\n";
63	63	echo ' <para>' . encodeText($q_question) . "</para>\r\n";
64	64	if (!empty($q_howdoi)) {

		@@ -6,16 +6,16 @@ discard block
		block discarded – undo
6	6	* Licence: GNU
7	7	*/
8	8
9		-include_once __DIR__ . '/../../../include/cp_header.php';
	9	+include_once __DIR__.'/../../../include/cp_header.php';
10	10
11		-$op = 'go';//'start';
	11	+$op = 'go'; //'start';
12	12
13	13	if (isset($HTTP_POST_VARS['op']) && ($HTTP_POST_VARS['op'] === 'go')) {
14	14	$op = $HTTP_POST_VARS['op'];
15	15	}
16	16
17	17	if ($op === 'start') {
18		- include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
	18	+ include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
19	19
20	20	xoops_cp_header();
21	21
		@@ -44,15 +44,15 @@ discard block
		block discarded – undo
44	44
45	45	echo " <title>{module title}</title>\r\n";
46	46
47		- $resultC = $xoopsDB->query('SELECT * FROM ' . $xoopsDB->prefix('smartfaq_categories'));
	47	+ $resultC = $xoopsDB->query('SELECT * FROM '.$xoopsDB->prefix('smartfaq_categories'));
48	48	while ($arrC = $xoopsDB->fetchArray($resultC)) {
49	49	extract($arrC, EXTR_PREFIX_ALL, 'c');
50	50
51	51	echo " <qandadiv ID=\"c$c_categoryid\" Revision=\"$c_created\">\r\n";
52		- echo ' <title>' . encodeText($c_name) . "</title>\r\n";
53		- echo ' <para>' . encodeText($c_description) . "</para>\r\n";
	52	+ echo ' <title>'.encodeText($c_name)."</title>\r\n";
	53	+ echo ' <para>'.encodeText($c_description)."</para>\r\n";
54	54
55		- $resultQ = $xoopsDB->query('select * from ' . $xoopsDB->prefix('smartfaq_faq') . " where categoryid=$c_categoryid");
	55	+ $resultQ = $xoopsDB->query('select * from '.$xoopsDB->prefix('smartfaq_faq')." where categoryid=$c_categoryid");
56	56	while ($arrQ = $xoopsDB->fetchArray($resultQ)) {
57	57	extract($arrQ, EXTR_PREFIX_ALL, 'q');
58	58
		@@ -60,27 +60,27 @@ discard block
		block discarded – undo
60	60	. getUserFullName($q_uid)
61	61	. "\">\r\n";
62	62	echo " <question>\r\n";
63		- echo ' <para>' . encodeText($q_question) . "</para>\r\n";
	63	+ echo ' <para>'.encodeText($q_question)."</para>\r\n";
64	64	if (!empty($q_howdoi)) {
65	65	echo " <note Conformance=\"howdoi\">\r\n";
66	66	echo " <title>{'How do I' from language file}</title>\r\n";
67		- echo ' <para>' . encodeText($q_howdoi) . "</para>\r\n";
	67	+ echo ' <para>'.encodeText($q_howdoi)."</para>\r\n";
68	68	echo " </note>\r\n";
69	69	}
70	70	if (!empty($q_diduno)) {
71	71	echo " <note Conformance=\"diduno\">\r\n";
72	72	echo " <title>{'Did you know' from language file}</title>\r\n";
73		- echo ' <para>' . encodeText($q_diduno) . "</para>\r\n";
	73	+ echo ' <para>'.encodeText($q_diduno)."</para>\r\n";
74	74	echo " </note>\r\n";
75	75	}
76	76	echo " </question>\r\n";
77	77
78		- $resultA = $xoopsDB->query('select * from ' . $xoopsDB->prefix('smartfaq_answers') . " where answerid=$q_answerid");
	78	+ $resultA = $xoopsDB->query('select * from '.$xoopsDB->prefix('smartfaq_answers')." where answerid=$q_answerid");
79	79	while ($arrA = $xoopsDB->fetchArray($resultA)) {
80	80	extract($arrA, EXTR_PREFIX_ALL, 'a');
81	81
82		- echo " <answer ID=\"a$a_answerid\" Revision=\"$a_datesub\" Vendor=\"" . getUserFullName($a_uid) . "\">\r\n";
83		- echo ' <para>' . encodeText($a_answer) . "</para>\r\n";
	82	+ echo " <answer ID=\"a$a_answerid\" Revision=\"$a_datesub\" Vendor=\"".getUserFullName($a_uid)."\">\r\n";
	83	+ echo ' <para>'.encodeText($a_answer)."</para>\r\n";
84	84	echo " </answer>\r\n";
85	85	}
86	86	$xoopsDB->freeRecordSet($resultA);

		@@ -59,14 +59,14 @@
		block discarded – undo
59	59	echo $form_view->render();
60	60	} else {
61	61	echo "<img id='toptableicon' src="
62		- . XOOPS_URL
63		- . '/modules/'
64		- . $xoopsModule->dirname()
65		- . "/assets/images/icon/close12.gif alt='' /></a> "
66		- . _AM_SF_PERMISSIONSVIEWMAN
67		- . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
68		- . _AM_SF_NOPERMSSET
69		- . '</span>';
	62	+ . XOOPS_URL
	63	+ . '/modules/'
	64	+ . $xoopsModule->dirname()
	65	+ . "/assets/images/icon/close12.gif alt='' /></a> "
	66	+ . _AM_SF_PERMISSIONSVIEWMAN
	67	+ . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
	68	+ . _AM_SF_NOPERMSSET
	69	+ . '</span>';
70	70	}
71	71	echo '</div>';
72	72

		@@ -24,53 +24,53 @@
		block discarded – undo
24	24	}
25	25
26	26	switch ($op) {
27		- case 'default':
28		- default:
29		- global $xoopsDB, $xoopsModule;
	27	+ case 'default':
	28	+ default:
	29	+ global $xoopsDB, $xoopsModule;
30	30
31		- $adminObject = \Xmf\Module\Admin::getInstance();
32		- xoops_cp_header();
33		- $adminObject->displayNavigation(basename(__FILE__));
34		- // View Categories permissions
35		- $item_list_view = array();
36		- $block_view = array();
37		- // echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
38		- sf_collapsableBar('toptable', 'toptableicon');
	31	+ $adminObject = \Xmf\Module\Admin::getInstance();
	32	+ xoops_cp_header();
	33	+ $adminObject->displayNavigation(basename(__FILE__));
	34	+ // View Categories permissions
	35	+ $item_list_view = array();
	36	+ $block_view = array();
	37	+ // echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
	38	+ sf_collapsableBar('toptable', 'toptableicon');
39	39
40		- $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
41		- if ($xoopsDB->getRowsNum($result_view)) {
42		- while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
43		- $item_list_view['cid'] = $myrow_view['categoryid'];
44		- $item_list_view['title'] = $myrow_view['name'];
45		- $form_view = new XoopsGroupPermForm('', $xoopsModule->getVar('mid'), 'category_read', "<img id='toptableicon' src="
46		- . XOOPS_URL
47		- . '/modules/'
48		- . $xoopsModule->dirname()
49		- . "/assets/images/icon/close12.gif alt='' /></a> "
50		- . _AM_SF_PERMISSIONSVIEWMAN
51		- . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
52		- . _AM_SF_VIEW_CATS
53		- . '</span>', 'admin/permissions.php');
54		- $block_view[] = $item_list_view;
55		- foreach ($block_view as $itemlists) {
56		- $form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
	40	+ $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
	41	+ if ($xoopsDB->getRowsNum($result_view)) {
	42	+ while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
	43	+ $item_list_view['cid'] = $myrow_view['categoryid'];
	44	+ $item_list_view['title'] = $myrow_view['name'];
	45	+ $form_view = new XoopsGroupPermForm('', $xoopsModule->getVar('mid'), 'category_read', "<img id='toptableicon' src="
	46	+ . XOOPS_URL
	47	+ . '/modules/'
	48	+ . $xoopsModule->dirname()
	49	+ . "/assets/images/icon/close12.gif alt='' /></a> "
	50	+ . _AM_SF_PERMISSIONSVIEWMAN
	51	+ . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
	52	+ . _AM_SF_VIEW_CATS
	53	+ . '</span>', 'admin/permissions.php');
	54	+ $block_view[] = $item_list_view;
	55	+ foreach ($block_view as $itemlists) {
	56	+ $form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
	57	+ }
57	58	}
	59	+ echo $form_view->render();
	60	+ } else {
	61	+ echo "<img id='toptableicon' src="
	62	+ . XOOPS_URL
	63	+ . '/modules/'
	64	+ . $xoopsModule->dirname()
	65	+ . "/assets/images/icon/close12.gif alt='' /></a> "
	66	+ . _AM_SF_PERMISSIONSVIEWMAN
	67	+ . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
	68	+ . _AM_SF_NOPERMSSET
	69	+ . '</span>';
58	70	}
59		- echo $form_view->render();
60		- } else {
61		- echo "<img id='toptableicon' src="
62		- . XOOPS_URL
63		- . '/modules/'
64		- . $xoopsModule->dirname()
65		- . "/assets/images/icon/close12.gif alt='' /></a> "
66		- . _AM_SF_PERMISSIONSVIEWMAN
67		- . "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
68		- . _AM_SF_NOPERMSSET
69		- . '</span>';
70		- }
71		- echo '</div>';
	71	+ echo '</div>';
72	72
73		- echo "<br>\n";
	73	+ echo "<br>\n";
74	74	}
75	75
76	76	require_once __DIR__ . '/admin_footer.php';

		@@ -6,8 +6,8 @@ discard block
		block discarded – undo
6	6	* Licence: GNU
7	7	*/
8	8
9		-require_once __DIR__ . '/admin_header.php';
10		-include_once XOOPS_ROOT_PATH . '/class/xoopsform/grouppermform.php';
	9	+require_once __DIR__.'/admin_header.php';
	10	+include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
11	11
12	12	if (!sf_userIsAdmin()) {
13	13	redirect_header('javascript:history.go(-1)', 1, _NOPERM);
		@@ -28,7 +28,7 @@ discard block
		block discarded – undo
28	28	default:
29	29	global $xoopsDB, $xoopsModule;
30	30
31		- $adminObject = \Xmf\Module\Admin::getInstance();
	31	+ $adminObject = \Xmf\Module\Admin::getInstance();
32	32	xoops_cp_header();
33	33	$adminObject->displayNavigation(basename(__FILE__));
34	34	// View Categories permissions
		@@ -37,7 +37,7 @@ discard block
		block discarded – undo
37	37	// echo "<h3 style='color: #2F5376; '>"._AM_SF_PERMISSIONSADMIN."</h3>\n" ;
38	38	sf_collapsableBar('toptable', 'toptableicon');
39	39
40		- $result_view = $xoopsDB->query('SELECT categoryid, name FROM ' . $xoopsDB->prefix('smartfaq_categories') . ' ');
	40	+ $result_view = $xoopsDB->query('SELECT categoryid, name FROM '.$xoopsDB->prefix('smartfaq_categories').' ');
41	41	if ($xoopsDB->getRowsNum($result_view)) {
42	42	while ($myrow_view = $xoopsDB->fetcharray($result_view)) {
43	43	$item_list_view['cid'] = $myrow_view['categoryid'];
		@@ -51,7 +51,7 @@ discard block
		block discarded – undo
51	51	. "</h3><div id='toptable'><span style=\"color: #567; margin: 3px 0 0 0; font-size: small; display: block; \">"
52	52	. _AM_SF_VIEW_CATS
53	53	. '</span>', 'admin/permissions.php');
54		- $block_view[] = $item_list_view;
	54	+ $block_view[] = $item_list_view;
55	55	foreach ($block_view as $itemlists) {
56	56	$form_view->addItem($itemlists['cid'], $myts->displayTarea($itemlists['title']));
57	57	}
		@@ -73,4 +73,4 @@ discard block
		block discarded – undo
73	73	echo "<br>\n";
74	74	}
75	75
76		-require_once __DIR__ . '/admin_footer.php';
	76	+require_once __DIR__.'/admin_footer.php';

		@@ -18,45 +18,45 @@
		block discarded – undo
18	18	}
19	19	$adminObject = \Xmf\Module\Admin::getInstance();
20	20
21		-$pathIcon32 = \Xmf\Module\Admin::menuIconPath('');
	21	+$pathIcon32 = \Xmf\Module\Admin::menuIconPath('');
22	22	//$pathModIcon32 = $moduleHelper->getModule()->getInfo('modicons32');
23	23
24	24	$moduleHelper->loadLanguage('modinfo');
25	25
26		-$adminObject = array();
	26	+$adminObject = array();
27	27	$i = 0;
28	28	$adminmenu[$i]['title'] = _AM_MODULEADMIN_HOME;
29	29	$adminmenu[$i]['link'] = 'admin/index.php';
30		-$adminmenu[$i]['icon'] = $pathIcon32 . '/home.png';
	30	+$adminmenu[$i]['icon'] = $pathIcon32.'/home.png';
31	31	++$i;
32	32	$adminmenu[$i]['title'] = _MI_SF_ADMENU1;
33	33	$adminmenu[$i]['link'] = 'admin/main.php';
34		-$adminmenu[$i]['icon'] = $pathIcon32 . '/manage.png';
	34	+$adminmenu[$i]['icon'] = $pathIcon32.'/manage.png';
35	35
36	36	++$i;
37	37	$adminmenu[$i]['title'] = _MI_SF_ADMENU2;
38	38	$adminmenu[$i]['link'] = 'admin/category.php';
39		-$adminmenu[$i]['icon'] = $pathIcon32 . '/category.png';
	39	+$adminmenu[$i]['icon'] = $pathIcon32.'/category.png';
40	40	++$i;
41	41	$adminmenu[$i]['title'] = _MI_SF_ADMENU3;
42	42	$adminmenu[$i]['link'] = 'admin/faq.php';
43		-$adminmenu[$i]['icon'] = $pathIcon32 . '/search.png';
	43	+$adminmenu[$i]['icon'] = $pathIcon32.'/search.png';
44	44	++$i;
45	45	$adminmenu[$i]['title'] = _MI_SF_ADMENU4;
46	46	$adminmenu[$i]['link'] = 'admin/question.php';
47		-$adminmenu[$i]['icon'] = $pathIcon32 . '/faq.png';
	47	+$adminmenu[$i]['icon'] = $pathIcon32.'/faq.png';
48	48	++$i;
49	49	$adminmenu[$i]['title'] = _MI_SF_ADMENU5;
50	50	$adminmenu[$i]['link'] = 'admin/permissions.php';
51		-$adminmenu[$i]['icon'] = $pathIcon32 . '/permissions.png';
	51	+$adminmenu[$i]['icon'] = $pathIcon32.'/permissions.png';
52	52	++$i;
53	53	$adminmenu[$i]['title'] = _MI_SF_ADMENU8;
54	54	$adminmenu[$i]['link'] = 'admin/import.php';
55		-$adminmenu[$i]['icon'] = $pathIcon32 . '/download.png';
	55	+$adminmenu[$i]['icon'] = $pathIcon32.'/download.png';
56	56	++$i;
57	57	$adminmenu[$i]['title'] = _AM_MODULEADMIN_ABOUT;
58	58	$adminmenu[$i]['link'] = 'admin/about.php';
59		-$adminmenu[$i]['icon'] = $pathIcon32 . '/about.png';
	59	+$adminmenu[$i]['icon'] = $pathIcon32.'/about.png';
60	60	//++$i;
61	61	//$adminmenu[$i]['title'] = _AM_MODULEADMIN_ABOUT;
62	62	//$adminmenu[$i]["link"] = "admin/about2.php";

		@@ -10,7 +10,7 @@ discard block
		block discarded – undo
10	10	*
11	11	*/
12	12
13		-require_once __DIR__ . '/admin_header.php';
	13	+require_once __DIR__.'/admin_header.php';
14	14
15	15	$importFromModuleName = 'WF-FAQ';
16	16	$scriptname = 'wffaq.php';
		@@ -22,32 +22,32 @@ discard block
		block discarded – undo
22	22	}
23	23
24	24	if ($op === 'start') {
25		- include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
	25	+ include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
26	26	xoops_cp_header();
27		- $result = $xoopsDB->query('SELECT count(*) FROM ' . $xoopsDB->prefix('faqcategories'));
	27	+ $result = $xoopsDB->query('SELECT count(*) FROM '.$xoopsDB->prefix('faqcategories'));
28	28	list($totalCat) = $xoopsDB->fetchRow($result);
29	29
30	30	sf_collapsableBar('bottomtable', 'bottomtableicon');
31		- echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a> " . sprintf(_AM_SF_IMPORT_FROM,
32		- $importFromModuleName) . '</h3>';
	31	+ echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a> ".sprintf(_AM_SF_IMPORT_FROM,
	32	+ $importFromModuleName).'</h3>';
33	33	echo "<div id='bottomtable'>";
34	34
35	35	if ($totalCat == 0) {
36		- echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_NO_CATEGORY . '</span>';
	36	+ echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_NO_CATEGORY.'</span>';
37	37	} else {
38		- include_once XOOPS_ROOT_PATH . '/class/xoopstree.php';
	38	+ include_once XOOPS_ROOT_PATH.'/class/xoopstree.php';
39	39
40		- $result = $xoopsDB->query('SELECT count(*) FROM ' . $xoopsDB->prefix('faqtopics'));
	40	+ $result = $xoopsDB->query('SELECT count(*) FROM '.$xoopsDB->prefix('faqtopics'));
41	41	list($totalFAQ) = $xoopsDB->fetchRow($result);
42	42
43	43	if ($totalFAQ == 0) {
44		- echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat) . '</span>';
	44	+ echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'.sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat).'</span>';
45	45	} else {
46	46	echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'
47	47	. sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
48	48	. '</span>';
49	49
50		- $form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL . '/modules/smartfaq/admin/' . $scriptname);
	50	+ $form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL.'/modules/smartfaq/admin/'.$scriptname);
51	51
52	52	// Categories to be imported
53	53	$cat_cbox = new XoopsFormCheckBox(sprintf(_AM_SF_IMPORT_CATEGORIES, $importFromModuleName), 'import_category', -1);
		@@ -69,7 +69,7 @@ discard block
		block discarded – undo
69	69	ob_end_clean();
70	70
71	71	// Auto-Approve
72		- $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_AUTOAPPROVE, 'autoaprove', 1, ' ' . _AM_SF_YES . '', ' ' . _AM_SF_NO . ''));
	72	+ $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_AUTOAPPROVE, 'autoaprove', 1, ' '._AM_SF_YES.'', ' '._AM_SF_NO.''));
73	73
74	74	// Submitted and answered by
75	75	$memberHandler = xoops_getHandler('member');
		@@ -82,7 +82,7 @@ discard block
		block discarded – undo
82	82	$form->addElement($user_select);
83	83
84	84	// Q&As can be commented?
85		- $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_ALLOWCOMMENTS, 'cancomment', 1, ' ' . _AM_SF_YES . '', ' ' . _AM_SF_NO . ''));
	85	+ $form->addElement(new XoopsFormRadioYN(_AM_SF_IMPORT_ALLOWCOMMENTS, 'cancomment', 1, ' '._AM_SF_YES.'', ' '._AM_SF_NO.''));
86	86	$group_list = $memberHandler->getGroupList();
87	87	$groups_selected = array();
88	88	$groups_checkbox = new XoopsFormCheckBox(_AM_SF_IMPORT_PERMISSIONS, 'groups_read');
		@@ -111,14 +111,14 @@ discard block
		block discarded – undo
111	111	redirect_header($scriptname, 2, _AM_SF_NOCATSELECTED);
112	112	}
113	113
114		- require_once __DIR__ . '/admin_header.php';
	114	+ require_once __DIR__.'/admin_header.php';
115	115	xoops_cp_header();
116	116
117	117	sf_collapsableBar('bottomtable', 'bottomtableicon');
118		- echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a> " . sprintf(_AM_SF_IMPORT_FROM,
119		- $importFromModuleName) . '</h3>';
	118	+ echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a> ".sprintf(_AM_SF_IMPORT_FROM,
	119	+ $importFromModuleName).'</h3>';
120	120	echo "<div id='bottomtable'>";
121		- echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_RESULT . '</span>';
	121	+ echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_RESULT.'</span>';
122	122
123	123	$cnt_imported_cat = 0;
124	124	$cnt_imported_faq = 0;
		@@ -146,7 +146,7 @@ discard block
		block discarded – undo
146	146	echo "Can Comment: $cancomment<br>";
147	147	echo "Auto aprove: $autoaprove<br>";*/
148	148
149		- $resultCat = $xoopsDB->query('select * from ' . $xoopsDB->prefix('faqcategories') . " where catID in ($import_category_list) order by weight");
	149	+ $resultCat = $xoopsDB->query('select * from '.$xoopsDB->prefix('faqcategories')." where catID in ($import_category_list) order by weight");
150	150
151	151	while ($arrCat = $xoopsDB->fetchArray($resultCat)) {
152	152	extract($arrCat, EXTR_PREFIX_ALL, 'wfc');
		@@ -161,7 +161,7 @@ discard block
		block discarded – undo
161	161	$categoryObj->setVar('description', $wfc_description);
162	162
163	163	if (!$categoryObj->store(false)) {
164		- echo sprintf(_AM_SF_IMPORT_CATEGORY_ERROR, $xcat_name) . '<br>';
	164	+ echo sprintf(_AM_SF_IMPORT_CATEGORY_ERROR, $xcat_name).'<br>';
165	165	continue;
166	166	}
167	167
		@@ -169,9 +169,9 @@ discard block
		block discarded – undo
169	169
170	170	++$cnt_imported_cat;
171	171
172		- echo sprintf(_AM_SF_IMPORT_CATEGORY_SUCCESS, $wfc_name) . "<br\>";
	172	+ echo sprintf(_AM_SF_IMPORT_CATEGORY_SUCCESS, $wfc_name)."<br\>";
173	173
174		- $resultFAQ = $xoopsDB->query('select * from ' . $xoopsDB->prefix('faqtopics') . " where catID=$wfc_catID order by weight");
	174	+ $resultFAQ = $xoopsDB->query('select * from '.$xoopsDB->prefix('faqtopics')." where catID=$wfc_catID order by weight");
175	175	while ($arrFAQ = $xoopsDB->fetchArray($resultFAQ)) {
176	176	extract($arrFAQ, EXTR_PREFIX_ALL, 'wft');
177	177
		@@ -203,7 +203,7 @@ discard block
		block discarded – undo
203	203	$faqObj->setVar('exacturl', 0);
204	204
205	205	if (!$faqObj->store(false)) {
206		- echo sprintf(' ' . _AM_SF_IMPORT_FAQ_ERROR, $wft_question) . '<br>';
	206	+ echo sprintf(' '._AM_SF_IMPORT_FAQ_ERROR, $wft_question).'<br>';
207	207	continue;
208	208	} else {
209	209	$answerObj->setVar('faqid', $faqObj->faqid());
		@@ -212,10 +212,10 @@ discard block
		block discarded – undo
212	212	$answerObj->setVar('status', _SF_AN_STATUS_APPROVED);
213	213
214	214	if (!$answerObj->store()) {
215		- echo sprintf(' ' . _AM_SF_IMPORT_FAQ_ERROR) . '<br>';
	215	+ echo sprintf(' '._AM_SF_IMPORT_FAQ_ERROR).'<br>';
216	216	continue;
217	217	} else {
218		- echo '  ' . sprintf(_AM_SF_IMPORTED_QUESTION, $faqObj->question(50)) . '<br>';
	218	+ echo '  '.sprintf(_AM_SF_IMPORTED_QUESTION, $faqObj->question(50)).'<br>';
219	219	++$cnt_imported_faq;
220	220	}
221	221	}
		@@ -225,8 +225,8 @@ discard block
		block discarded – undo
225	225	}
226	226
227	227	echo 'Done.<br>';
228		- echo sprintf(_AM_SF_IMPORTED_CATEGORIES, $cnt_imported_cat) . '<br>';
229		- echo sprintf(_AM_SF_IMPORTED_QUESTIONS, $cnt_imported_faq) . '<br>';
	228	+ echo sprintf(_AM_SF_IMPORTED_CATEGORIES, $cnt_imported_cat).'<br>';
	229	+ echo sprintf(_AM_SF_IMPORTED_QUESTIONS, $cnt_imported_faq).'<br>';
230	230
231	231	exit();
232	232	}

		@@ -46,8 +46,8 @@
		block discarded – undo
46	46	echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . sprintf(_AM_SF_IMPORT_MODULE_FOUND_NO_FAQ, $importFromModuleName, $totalCat) . '</span>';
47	47	} else {
48	48	echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'
49		- . sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
50		- . '</span>';
	49	+ . sprintf(_AM_SF_IMPORT_MODULE_FOUND, $importFromModuleName, $totalCat, $totalFAQ)
	50	+ . '</span>';
51	51
52	52	$form = new XoopsThemeForm(_AM_SF_IMPORT_SETTINGS, 'import_form', XOOPS_URL . '/modules/smartfaq/admin/' . $scriptname);
53	53

		@@ -19,12 +19,12 @@
		block discarded – undo
19	19
20	20	$pathIcon32 = \Xmf\Module\Admin::iconUrl('', 32);
21	21	echo "<div class='adminfooter'>\n"
22		- . " <div style='text-align: center;'>\n"
23		- . " <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
24		- . " </div>\n"
25		- . ' '
26		- . _AM_MODULEADMIN_ADMIN_FOOTER
27		- . "\n"
28		- . '</div>';
	22	+ . " <div style='text-align: center;'>\n"
	23	+ . " <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
	24	+ . " </div>\n"
	25	+ . ' '
	26	+ . _AM_MODULEADMIN_ADMIN_FOOTER
	27	+ . "\n"
	28	+ . '</div>';
29	29
30	30	xoops_cp_footer();

		@@ -17,7 +17,7 @@
		block discarded – undo
17	17	* @author XOOPS Development Team
18	18	*/
19	19
20		-$pathIcon32 = \Xmf\Module\Admin::iconUrl('', 32);
	20	+$pathIcon32 = \Xmf\Module\Admin::iconUrl('', 32);
21	21	echo "<div class='adminfooter'>\n"
22	22	. " <div style='text-align: center;'>\n"
23	23	. " <a href='http://www.xoops.org' rel='external'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"

		@@ -17,7 +17,7 @@ discard block
		block discarded – undo
17	17	* @author XOOPS Development Team
18	18	*/
19	19
20		-include_once __DIR__ . '/../../../include/cp_header.php';
	20	+include_once __DIR__.'/../../../include/cp_header.php';
21	21	//include_once $GLOBALS['xoops']->path('www/class/xoopsformloader.php');
22	22
23	23	//require __DIR__ . '/../class/utility.php';
		@@ -28,16 +28,16 @@ discard block
		block discarded – undo
28	28	}
29	29
30	30	$path = dirname(dirname(dirname(__DIR__)));
31		-include_once $path . '/kernel/module.php';
32		-include_once $path . '/class/xoopstree.php';
33		-include_once $path . '/class/xoopslists.php';
34		-include_once $path . '/class/pagenav.php';
35		-include_once $path . '/class/xoopsformloader.php';
	31	+include_once $path.'/kernel/module.php';
	32	+include_once $path.'/class/xoopstree.php';
	33	+include_once $path.'/class/xoopslists.php';
	34	+include_once $path.'/class/pagenav.php';
	35	+include_once $path.'/class/xoopsformloader.php';
36	36
37		-include_once __DIR__ . '/../include/functions.php';
38		-include_once __DIR__ . '/../class/category.php';
39		-include_once __DIR__ . '/../class/faq.php';
40		-include_once __DIR__ . '/../class/answer.php';
	37	+include_once __DIR__.'/../include/functions.php';
	38	+include_once __DIR__.'/../class/category.php';
	39	+include_once __DIR__.'/../class/faq.php';
	40	+include_once __DIR__.'/../class/answer.php';
41	41	//require __DIR__ . '/../class/utility.php';
42	42	//require_once __DIR__ . '/../include/common.php';
43	43

		@@ -6,7 +6,7 @@ discard block
		block discarded – undo
6	6	* Licence: GNU
7	7	*/
8	8
9		-require_once __DIR__ . '/admin_header.php';
	9	+require_once __DIR__.'/admin_header.php';
10	10
11	11	$op = 'none';
12	12
		@@ -23,7 +23,7 @@ discard block
		block discarded – undo
23	23	case 'importExecute':
24	24
25	25	$importfile = isset($_POST['importfile']) ? $_POST['importfile'] : 'nonselected';
26		- $importfile_path = XOOPS_ROOT_PATH . '/modules/smartfaq/admin/' . $importfile . '.php';
	26	+ $importfile_path = XOOPS_ROOT_PATH.'/modules/smartfaq/admin/'.$importfile.'.php';
27	27	if (!file_exists($importfile_path)) {
28	28	$errs[] = sprintf(_AM_SF_IMPORT_FILE_NOT_FOUND, $importfile_path);
29	29	$error = true;
		@@ -31,7 +31,7 @@ discard block
		block discarded – undo
31	31	include_once $importfile_path;
32	32	}
33	33	foreach ($msgs as $m) {
34		- echo $m . '<br>';
	34	+ echo $m.'<br>';
35	35	}
36	36	echo '<br>';
37	37	$endMsg = _AM_SF_IMPORT_SUCCESS;
		@@ -41,7 +41,7 @@ discard block
		block discarded – undo
41	41
42	42	echo $endMsg;
43	43	echo '<br><br>';
44		- echo "<a href='import.php'>" . _AM_SF_IMPORT_BACK . '</a>';
	44	+ echo "<a href='import.php'>"._AM_SF_IMPORT_BACK.'</a>';
45	45	echo '<br><br>';
46	46	break;
47	47
		@@ -53,13 +53,13 @@ discard block
		block discarded – undo
53	53	xoops_cp_header();
54	54
55	55	sf_collapsableBar('bottomtable', 'bottomtableicon');
56		- echo "<img id='bottomtableicon' src=" . XOOPS_URL . '/modules/' . $xoopsModule->dirname() . "/assets/images/icon/close12.gif alt='' /></a> " . _AM_SF_IMPORT_TITLE . '</h3>';
	56	+ echo "<img id='bottomtableicon' src=".XOOPS_URL.'/modules/'.$xoopsModule->dirname()."/assets/images/icon/close12.gif alt='' /></a> "._AM_SF_IMPORT_TITLE.'</h3>';
57	57	echo "<div id='bottomtable'>";
58		- echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">' . _AM_SF_IMPORT_INFO . '</span>';
	58	+ echo '<span style="color: #567; margin: 3px 0 12px 0; font-size: small; display: block; ">'._AM_SF_IMPORT_INFO.'</span>';
59	59
60	60	global $xoopsUser, $xoopsUser, $xoopsConfig, $xoopsDB, $modify, $xoopsModuleConfig, $xoopsModule, $XOOPS_URL, $myts;
61	61
62		- include_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
	62	+ include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
63	63
64	64	/** @var XoopsModuleHandler $moduleHandler */
65	65	$moduleHandler = xoops_getHandler('module');
		@@ -99,7 +99,7 @@ discard block
		block discarded – undo
99	99	$sform->display();
100	100	unset($hidden);
101	101	} else {
102		- echo '<span style="color: #567; margin: 3px 0 12px 0; font-weight: bold; font-size: small; display: block; ">' . _AM_SF_IMPORT_NO_MODULE . '</span>';
	102	+ echo '<span style="color: #567; margin: 3px 0 12px 0; font-weight: bold; font-size: small; display: block; ">'._AM_SF_IMPORT_NO_MODULE.'</span>';
103	103	}
104	104
105	105	// End of collapsable bar
		@@ -108,4 +108,4 @@ discard block
		block discarded – undo
108	108	break;
109	109	}
110	110
111		-require_once __DIR__ . '/admin_footer.php';
	111	+require_once __DIR__.'/admin_footer.php';

		@@ -11,7 +11,7 @@ discard block
		block discarded – undo
11	11
12	12	function b_faqs_context_show($options)
13	13	{
14		- include_once XOOPS_ROOT_PATH . '/modules/smartfaq/include/functions.php';
	14	+ include_once XOOPS_ROOT_PATH.'/modules/smartfaq/include/functions.php';
15	15
16	16	$block = array();
17	17
		@@ -48,8 +48,8 @@ discard block
		block discarded – undo
48	48	*/
49	49	function b_faqs_context_edit($options)
50	50	{
51		- $form = '' . _MB_SF_DISP . ' ';
52		- $form .= "<input type='text' name='options[]' value='" . $options[0] . "' /> " . _MB_SF_FAQS . '';
	51	+ $form = ''._MB_SF_DISP.' ';
	52	+ $form .= "<input type='text' name='options[]' value='".$options[0]."' /> "._MB_SF_FAQS.'';
53	53
54	54	return $form;
55	55	}

mambax7 / smartfaq

Branch — master (c92e39)

Status

Category

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +13 added lines, -13 removed lines patch added patch discarded remove patch

Indentation +8 added lines, -8 removed lines patch added patch discarded remove patch

Switch Indentation +42 added lines, -42 removed lines patch added patch discarded remove patch

Spacing +6 added lines, -6 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +25 added lines, -25 removed lines patch added patch discarded remove patch

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

Indentation +7 added lines, -7 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +10 added lines, -10 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch