mambax7 / xoopsfaq

class/category.php

Doc Comments +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
      *
      * @param string sort order ('id', order', or 'title') - default: id
      *
-     * @return mixed XoopsfaqCategory | false on failure
+     * @return boolean XoopsfaqCategory | false on failure
      */
     public function getObj($sort = 'id')
     {

Indentation +15 added lines, -15 removed lines

@@ -170,32 +170,32 @@
         $buttons = array('edit', 'delete');
 
         $ret = '<table class="outer width100 bnone pad3 marg5">'
-             . '  <thead>'
-             . '  <tr class="xoopsCenter">'
-             . '    <th class="width5">' . _AM_XOOPSFAQ_CATEGORY_ORDER . '</th>'
-             . '    <th class="width5">' . _AM_XOOPSFAQ_CATEGORY_ID . '</th>'
-             . '    <th class="txtleft">' . _AM_XOOPSFAQ_CATEGORY_TITLE . '</th>'
-             . '    <th class="width20">' . _AM_XOOPSFAQ_ACTIONS . '</th>'
-             . '  </tr>'
-             . '  </thead>'
-             . '  <tbody>';
+                . '  <thead>'
+                . '  <tr class="xoopsCenter">'
+                . '    <th class="width5">' . _AM_XOOPSFAQ_CATEGORY_ORDER . '</th>'
+                . '    <th class="width5">' . _AM_XOOPSFAQ_CATEGORY_ID . '</th>'
+                . '    <th class="txtleft">' . _AM_XOOPSFAQ_CATEGORY_TITLE . '</th>'
+                . '    <th class="width20">' . _AM_XOOPSFAQ_ACTIONS . '</th>'
+                . '  </tr>'
+                . '  </thead>'
+                . '  <tbody>';
         if ($objects['count'] > 0) {
             foreach ($objects['list'] as $object) {
                 $ret .= '  <tr class="xoopsCenter">'
-                      . '    <td class="even txtcenter">' . $object->getVar('category_order') . '</td>'
-                      . '    <td class="even txtcenter">' . $object->getVar('category_id') . '</td>'
-                      . '    <td class="even txtleft">' . $object->getVar('category_title') . '</td>'
-                      . '    <td class="even txtcenter">';
+                        . '    <td class="even txtcenter">' . $object->getVar('category_order') . '</td>'
+                        . '    <td class="even txtcenter">' . $object->getVar('category_id') . '</td>'
+                        . '    <td class="even txtleft">' . $object->getVar('category_title') . '</td>'
+                        . '    <td class="even txtcenter">';
                 $ret .= XoopsfaqUtility::renderIconLinks($buttons, 'category_id', $object->getVar('category_id'));
                 $ret .= '    </td>'
-                      . '  </tr>';
+                        . '  </tr>';
             }
         } else {
             $ret .= '  <tr class="txtcenter"><td colspan="4" class="even">' . _AM_XOOPSFAQ_NOLISTING . '</td></tr>';
         }
         $ret .= '  </tbody>'
 //              . '  <tfoot><tr class="txtcenter"><td colspan="4" class="head">&nbsp;</td></tr></tfoot>'
-              . '</table>';
+                . '</table>';
         return $ret;
     }
 

class/contents.php

Doc Comments +2 added lines, -2 removed lines

@@ -240,7 +240,7 @@ 
      *
      * @param CriteriaElement|string sort order ('id', 'cid', 'title', 'publish', or 'weight') default: 'id'
      *
-     * @return mixed XoopsfaqContents object | false on failure
+     * @return boolean XoopsfaqContents object | false on failure
      */
     public function getObj($sort = 'id')
     {
@@ -265,7 +265,7 @@ 
      * XoopsfaqContentsHandler::getPublished()
      *
      * @param string $id
-     * @return mixed array of XoopsfaqContent objects | false on failure
+     * @return boolean array of XoopsfaqContent objects | false on failure
      */
     public function getPublished($id = '')
     {

Indentation +29 added lines, -29 removed lines

@@ -30,9 +30,9 @@ 
  */
 class XoopsfaqContents extends XoopsObject
 {
-  /**
-   * @var string contains this modules directory name
-   */
+    /**
+     * @var string contains this modules directory name
+     */
     protected $dirname;
 
     /**
@@ -131,11 +131,11 @@ 
         if (class_exists('XoopsFormEditor')) {
 //            $editorConfigs = array('editor' => $GLOBALS['xoopsConfig']['general_editor'],
             $editorConfigs = array('editor' => $xfHelper->getConfig('use_wysiwyg', 'dhtmltextarea'),
-                                     'rows' => 25,
-                                     'cols' => '100%',
+                                        'rows' => 25,
+                                        'cols' => '100%',
                                     'width' => '100%',
-                                   'height' => '600px',
-                                     'name' => 'contents_contents',
+                                    'height' => '600px',
+                                        'name' => 'contents_contents',
                                     'value' => $this->getVar('contents_contents', 'e')
             );
             $contents_contents = new XoopsFormEditor('', 'contents_contents', $editorConfigs);
@@ -331,18 +331,18 @@ 
         $buttons      = array('edit', 'delete');
 
         $ret = '<table class="outer width100 bnone pad3 marg5">'
-             . '  <thead>'
-             . '  <tr class="center">'
-             . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_ID . '</th>'
-             . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_ACTIVE . '</th>'
-             . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_WEIGHT . '</th>'
-             . '    <th class="left">' . _AM_XOOPSFAQ_CONTENTS_TITLE . '</th>'
-             . '    <th class="left">' . _AM_XOOPSFAQ_CATEGORY_TITLE . '</th>'
-             . '    <th>' . _AM_XOOPSFAQ_CONTENTS_PUBLISH . '</th>'
-             . '    <th class="width20">' . _AM_XOOPSFAQ_ACTIONS . '</th>'
-             . '  </tr>'
-             . '  </thead>'
-             . '  <tbody>';
+                . '  <thead>'
+                . '  <tr class="center">'
+                . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_ID . '</th>'
+                . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_ACTIVE . '</th>'
+                . '    <th class="width5">' . _AM_XOOPSFAQ_CONTENTS_WEIGHT . '</th>'
+                . '    <th class="left">' . _AM_XOOPSFAQ_CONTENTS_TITLE . '</th>'
+                . '    <th class="left">' . _AM_XOOPSFAQ_CATEGORY_TITLE . '</th>'
+                . '    <th>' . _AM_XOOPSFAQ_CONTENTS_PUBLISH . '</th>'
+                . '    <th class="width20">' . _AM_XOOPSFAQ_ACTIONS . '</th>'
+                . '  </tr>'
+                . '  </thead>'
+                . '  <tbody>';
         if ($objects['count'] > 0) {
             $tdClass = 0;
             foreach ($objects['list'] as $object) {
@@ -352,23 +352,23 @@ 
                 ++$tdClass;
                 $dispClass = ($tdClass % 1) ? 'even' : 'odd';
                 $ret .= '  <tr class="center middle">'
-                      . '    <td class="' . $dispClass . '">' . $object->getVar('contents_id') . '</td>'
-                      . '    <td class="' . $dispClass . '">' . $object->getActiveIcon() . '</td>'
-                      . '    <td class="' . $dispClass . '">' . $object->getVar('contents_weight') . '</td>'
-                      . '    <td class="' . $dispClass . ' left">' . $thisContentTitle . '</td>'
-                      . '    <td class="' . $dispClass . ' left">' . $thisCatTitle . '</td>'
-                      . '    <td class="' . $dispClass . '">' . $object->getPublished(_SHORTDATESTRING) . '</td>'
-                      . '    <td class="' . $dispClass . '">';
+                        . '    <td class="' . $dispClass . '">' . $object->getVar('contents_id') . '</td>'
+                        . '    <td class="' . $dispClass . '">' . $object->getActiveIcon() . '</td>'
+                        . '    <td class="' . $dispClass . '">' . $object->getVar('contents_weight') . '</td>'
+                        . '    <td class="' . $dispClass . ' left">' . $thisContentTitle . '</td>'
+                        . '    <td class="' . $dispClass . ' left">' . $thisCatTitle . '</td>'
+                        . '    <td class="' . $dispClass . '">' . $object->getPublished(_SHORTDATESTRING) . '</td>'
+                        . '    <td class="' . $dispClass . '">';
                 $ret .= XoopsfaqUtility::renderIconLinks($buttons, 'contents_id', $object->getVar('contents_id'))
-                      . '</td>'
-                      . '  </tr>';
+                        . '</td>'
+                        . '  </tr>';
             }
         } else {
             $ret .= '  <tr class="center"><td colspan="7" class="even">' . _AM_XOOPSFAQ_NOLISTING . '</td></tr>';
         }
         $ret .= '  </tbody>'
 //             . '  <tr class="center"><td colspan="7" class="foot">&nbsp;</td></tr>'
-              . '</table>';
+                . '</table>';
         return $ret;
     }
 

Spacing +3 added lines, -3 removed lines

@@ -102,7 +102,7 @@ 
         $form->addElement(new xoopsFormHidden('contents_id', $this->getVar('contents_id', 'e')));
 
         // Active
-        $contents_active = new XoopsFormRadioYN(_AM_XOOPSFAQ_E_CONTENTS_ACTIVE, 'contents_active', $this->getVar('contents_active', 'e'),   ' ' . _YES . '', ' ' . _NO . '' );
+        $contents_active = new XoopsFormRadioYN(_AM_XOOPSFAQ_E_CONTENTS_ACTIVE, 'contents_active', $this->getVar('contents_active', 'e'), ' ' . _YES . '', ' ' . _NO . '');
         $contents_active->setDescription(_AM_XOOPSFAQ_E_CONTENTS_ACTIVE_DESC);
         $form->addElement($contents_active, false);
 
@@ -127,7 +127,7 @@ 
         $form->addElement($contents_weight, false);
 
         // Editor
-        $options_tray  = new XoopsFormElementTray(_AM_XOOPSFAQ_E_CONTENTS_CONTENT, '<br>');
+        $options_tray = new XoopsFormElementTray(_AM_XOOPSFAQ_E_CONTENTS_CONTENT, '<br>');
         if (class_exists('XoopsFormEditor')) {
 //            $editorConfigs = array('editor' => $GLOBALS['xoopsConfig']['general_editor'],
             $editorConfigs = array('editor' => $xfHelper->getConfig('use_wysiwyg', 'dhtmltextarea'),
@@ -254,7 +254,7 @@ 
             $criteria->setStart(0);
             $criteria->setLimit(0);
         } else {
-            $obj['list']  = $this->getObjects($sort, false);
+            $obj['list'] = $this->getObjects($sort, false);
         }
         $obj['list'] = $this->getObjects($criteria, false);
         $obj['count'] = (false !== $obj['list']) ? count($obj['list']) : 0;

class/utility.php

Doc Comments +2 added lines, -2 removed lines

@@ -268,8 +268,8 @@
     /**
      * Render the icon links
      *
-     * @param array $icon_array contains operation=>icon_name as key=>value
-     * @param mixed $param      HTML parameter
+     * @param string[] $icon_array contains operation=>icon_name as key=>value
+     * @param string $param      HTML parameter
      * @param mixed $value      HTML parameter value to set
      * @param mixed $extra      are any additional HTML attributes desired for the <a> tag
      * @return string

Indentation +10 added lines, -10 removed lines

@@ -22,12 +22,12 @@ 
  * @since::      File available since version 4.10
  */
 
- /**
-  * XoopsfaqUtility
-  *
-  * Static utility class to provide common functionality
-  *
-  */
+    /**
+     * XoopsfaqUtility
+     *
+     * Static utility class to provide common functionality
+     *
+     */
 class XoopsfaqUtility
 {
     /**
@@ -170,7 +170,7 @@ 
 
         // Open the source directory to read in files
         $iterator = new DirectoryIterator($src);
-       foreach ($iterator as $fObj) {
+        foreach ($iterator as $fObj) {
             if ($fObj->isFile()) {
                 $filename = $fObj->getPathname();
                 $fObj = null; // clear this iterator object to close the file
@@ -294,9 +294,9 @@ 
                 }
                 $title = constant (htmlspecialchars(mb_strtoupper('_XO_LA_' . $iconName)));
                 $img = '<img src="' . Xmf\Module\Admin::iconUrl($iconName . '.' . $iconExt, '16') . '"'
-                     . ' title ="' . $title . '"'
-                     . ' alt = "' . $title . '"'
-                     . ' class="bnone middle">';
+                        . ' title ="' . $title . '"'
+                        . ' alt = "' . $title . '"'
+                        . ' class="bnone middle">';
                 $ret .= '<a href="' . $url . '"' . $extra . '>' . $img . '</a>';
             }
         }

Spacing +9 added lines, -9 removed lines

@@ -182,7 +182,7 @@ 
                 static::rrmdir($fObj->getPathname());
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -221,7 +221,7 @@ 
                 static::rmove($fObj->getPathname(), $dest . '/' . $fObj->getFilename());
             }
         }
-        $iterator = null;   // clear iterator Obj to close file/directory
+        $iterator = null; // clear iterator Obj to close file/directory
         return rmdir($src); // remove the directory & return results
     }
 
@@ -255,11 +255,11 @@ 
 
         // Open the source directory to read in files
         $iterator = new DirectoryIterator($src);
-        foreach($iterator as $fObj) {
-            if($fObj->isFile()) {
+        foreach ($iterator as $fObj) {
+            if ($fObj->isFile()) {
                 copy($fObj->getPathname(), $dest . '/' . $fObj->getFilename());
-            } else if(!$fObj->isDot() && $fObj->isDir()) {
-                static::rcopy($fObj->getPathname(), $dest . '/' . $fObj-getFilename());
+            } else if (!$fObj->isDot() && $fObj->isDir()) {
+                static::rcopy($fObj->getPathname(), $dest . '/' . $fObj - getFilename());
             }
         }
         return true;
@@ -280,19 +280,19 @@ 
         xoops_loadLanguage('admin', $moduleDirName);
         $ret = '';
         if (null !== $value) {
-            foreach($icon_array as $_op => $icon) {
+            foreach ($icon_array as $_op => $icon) {
                 if (false === strpos($icon, '.')) {
                     $iconName = $icon;
                     $iconExt = 'png';
                 } else {
-                    $iconName = substr($icon, 0, strlen($icon)-strrchr($icon, '.'));
+                    $iconName = substr($icon, 0, strlen($icon) - strrchr($icon, '.'));
                     $iconExt  = substr(strrchr($icon, '.'), 1);
                 }
                 $url = (!is_numeric($_op)) ? $_op . '?' . $param . '=' . $value : xoops_getenv('PHP_SELF') . '?op=' . $iconName . '&' . $param . '=' . $value;
                 if (null !== $extra) {
                     $url .= ' ' . $extra;
                 }
-                $title = constant (htmlspecialchars(mb_strtoupper('_XO_LA_' . $iconName)));
+                $title = constant(htmlspecialchars(mb_strtoupper('_XO_LA_' . $iconName)));
                 $img = '<img src="' . Xmf\Module\Admin::iconUrl($iconName . '.' . $iconExt, '16') . '"'
                      . ' title ="' . $title . '"'
                      . ' alt = "' . $title . '"'

include/onupdate.inc.php

Unused Use Statements -2 removed lines

@@ -21,9 +21,7 @@
  * @since     File available since version 1.25
  */
 //use Xmf\Debug;
-use Xmf\Database\Tables;
 use Xmf\Module\Helper;
-use Xmf\Request;
 
 /**
  * @internal {Make sure you PROTECT THIS FILE}

Indentation +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
    || !($GLOBALS['xoopsUser'] instanceof XoopsUser)
    || !($GLOBALS['xoopsUser']->isAdmin()))
 {
-     exit("Restricted access" . PHP_EOL);
+        exit("Restricted access" . PHP_EOL);
 }
 
 /**
@@ -93,8 +93,8 @@ 
         // been relocated to ./assets
         //----------------------------------------------------------------
         $old_directories = array($xfHelper->path('css/'),
-                                 $xfHelper->path('js/'),
-                                 $xfHelper->path('images/')
+                                    $xfHelper->path('js/'),
+                                    $xfHelper->path('images/')
         );
         foreach ($old_directories as $old_dir) {
             $dirInfo = new SplFileInfo($old_dir);

Spacing +1 added lines, -1 removed lines

@@ -130,7 +130,7 @@
         $oldFiles = array($xfHelper->path('include/functions.php'),
                             $xsHelper->path('class/utilities.php')
         );
-        foreach($oldFiles as $file) {
+        foreach ($oldFiles as $file) {
             if (is_file($file)) {
                 if (false === ($delOk = unlink($file))) {
                     $module->setErrors(sprintf(_AM_XOOPSFAQ_ERROR_BAD_REMOVE, $file));

include/search.inc.php

Indentation +3 added lines, -3 removed lines

@@ -70,7 +70,7 @@ 
     $totalLimit = (int)$limit - $catCount;
     foreach ($catArray as $cId => $cat) {
         $ret[] = array ('image' => 'assets/images/folder.png',
-                         'link' => $xfHelper->url('index.php?cat_id=' . $cId),
+                            'link' => $xfHelper->url('index.php?cat_id=' . $cId),
                         'title' => $cat['category_title'],
         );
     }
@@ -100,9 +100,9 @@ 
     $contentArray = $xfFaqHandler->getAll($criteria, $contentFields, false);
     foreach ($contentArray as $content) {
         $ret[] = array ('image' => 'assets/images/question2.gif',
-                         'link' => $xfHelper->url('index.php?cat_id=' . $content['contents_cid'] . '#q' . $content['contents_id']),
+                            'link' => $xfHelper->url('index.php?cat_id=' . $content['contents_cid'] . '#q' . $content['contents_id']),
                         'title' => $content['contents_title'],
-                         'time' => $content['contents_publish'],
+                            'time' => $content['contents_publish'],
         );
     }
     unset($contentArray);

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
         return $ret;
     }
 
-    $moduleDirName = basename(dirname(__DIR__)) ;
+    $moduleDirName = basename(dirname(__DIR__));
     xoops_load('constants', $moduleDirName);
 
     $xfHelper = Xmf\Module\Helper::getHelper($moduleDirName);
@@ -69,7 +69,7 @@ 
     $catCount   = !empty($catArray) ? count($catArray) : 0;
     $totalLimit = (int)$limit - $catCount;
     foreach ($catArray as $cId => $cat) {
-        $ret[] = array ('image' => 'assets/images/folder.png',
+        $ret[] = array('image' => 'assets/images/folder.png',
                          'link' => $xfHelper->url('index.php?cat_id=' . $cId),
                         'title' => $cat['category_title'],
         );
@@ -99,7 +99,7 @@ 
     }
     $contentArray = $xfFaqHandler->getAll($criteria, $contentFields, false);
     foreach ($contentArray as $content) {
-        $ret[] = array ('image' => 'assets/images/question2.gif',
+        $ret[] = array('image' => 'assets/images/question2.gif',
                          'link' => $xfHelper->url('index.php?cat_id=' . $content['contents_cid'] . '#q' . $content['contents_id']),
                         'title' => $content['contents_title'],
                          'time' => $content['contents_publish'],

include/sitemap.plugin.php

Indentation +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
     $retVal = array() ;
     foreach ($catList as $id=>$title){
         $retVal['parent'][] = array('id' => $id,
-                                 'title' => $myts->htmlSpecialChars($title),
-                                   'url' => $xfHelper->url('index.php?cat_id=' . $id)
+                                    'title' => $myts->htmlSpecialChars($title),
+                                    'url' => $xfHelper->url('index.php?cat_id=' . $id)
         );
     }
     return $retVal;

Spacing +3 added lines, -3 removed lines

@@ -25,13 +25,13 @@
 
     $myts = MyTextSanitizer::getInstance();
 
-    $moduleDirName = basename(dirname(__DIR__)) ;
+    $moduleDirName = basename(dirname(__DIR__));
     $xfHelper      = Xmf\Module\Helper::getHelper($moduleDirName);
     $xfCatHandler  = $xfHelper->getHandler('category');
     $catList       = $xfCatHandler->getList();
 
-    $retVal = array() ;
-    foreach ($catList as $id=>$title){
+    $retVal = array();
+    foreach ($catList as $id=>$title) {
         $retVal['parent'][] = array('id' => $id,
                                  'title' => $myts->htmlSpecialChars($title),
                                    'url' => $xfHelper->url('index.php?cat_id=' . $id)

include/oninstall.inc.php

Indentation +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
    || !($GLOBALS['xoopsUser'] instanceof XoopsUser)
    || !($GLOBALS['xoopsUser']->isAdmin()))
 {
-     exit("Restricted access" . PHP_EOL);
+        exit("Restricted access" . PHP_EOL);
 }
 
 /**

include/rss.inc.php

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 
     foreach ($contentObjs as $contentObj) {
         $retVal[] = array ('image' => '',
-                           'title' => $contentObj->getVar('contents_title'),
+                            'title' => $contentObj->getVar('contents_title'),
                             'link' => $contentObj->getVar('contents_contents'),
                             'time' => $contentObj->getVar('contents_publish'),
                             'desc' => '',

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     $retVal = array();
 
     foreach ($contentObjs as $contentObj) {
-        $retVal[] = array ('image' => '',
+        $retVal[] = array('image' => '',
                            'title' => $contentObj->getVar('contents_title'),
                             'link' => $contentObj->getVar('contents_contents'),
                             'time' => $contentObj->getVar('contents_publish'),

admin/main.php

Indentation +7 added lines, -7 removed lines

@@ -74,15 +74,15 @@
         if (is_object($obj) && $obj instanceof XoopsfaqContents) {
             $contents_publish = strtotime(Request::getString('contents_publish', time(), 'POST'));
             $obj->setVars(array('contents_cid' => Request::getInt('contents_cid', XoopsfaqConstants::DEFAULT_CATEGORY, 'POST'),
-                              'contents_title' => Request::getString('contents_title', '', 'POST'),
-                           'contents_contents' => $contents_contents,
-                             'contents_weight' => Request::getInt('contents_weight', XoopsfaqConstants::DEFAULT_WEIGHT, 'POST'),
-                             'contents_active' => Request::getInt('contents_active', XoopsfaqConstants::ACTIVE, 'POST'),
+                                'contents_title' => Request::getString('contents_title', '', 'POST'),
+                            'contents_contents' => $contents_contents,
+                                'contents_weight' => Request::getInt('contents_weight', XoopsfaqConstants::DEFAULT_WEIGHT, 'POST'),
+                                'contents_active' => Request::getInt('contents_active', XoopsfaqConstants::ACTIVE, 'POST'),
                             'contents_publish' => $contents_publish,
-                                      'dohtml' => isset($_POST['dohtml']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
+                                        'dohtml' => isset($_POST['dohtml']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
                                     'dosmiley' => isset($_POST['dosmiley']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
-                                     'doxcode' => isset($_POST['doxcode']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
-                                     'doimage' => isset($_POST['doimage']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
+                                        'doxcode' => isset($_POST['doxcode']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
+                                        'doimage' => isset($_POST['doimage']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET,
                                         'dobr' => isset($_POST['dobr']) ? XoopsfaqConstants::SET : XoopsfaqConstants::NOTSET)
             );
             $ret = $xfFaqHandler->insert($obj, true);

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
         $catCount = $xfCatHandler->getCount();
         $adminObject->displayNavigation(basename(__FILE__));
         if (!empty($catCount)) {
-            $adminObject->addItemButton(_AM_XOOPSFAQ_CREATE_NEW, basename(__FILE__) . '?op=edit', 'add' , '');
+            $adminObject->addItemButton(_AM_XOOPSFAQ_CREATE_NEW, basename(__FILE__) . '?op=edit', 'add', '');
             $adminObject->displayButton('left');
         }
         $xfFaqHandler->displayAdminListing('weight');