XoopsModules25x / xoopsfaq

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);
@@ -351,18 +351,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;
             /** @var \XoopsfaqContents $object */
@@ -373,22 +373,22 @@ 
                 ++$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>'
-              . '</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'),
@@ -299,7 +299,7 @@ 
         $ret = array();
         $sql  = 'SELECT contents_cid ';
         $sql .= 'FROM `' . $this->table . '` ';
-        $sql .= 'WHERE (contents_active =\''. XoopsfaqConstants::ACTIVE . '\') ';
+        $sql .= 'WHERE (contents_active =\'' . XoopsfaqConstants::ACTIVE . '\') ';
         $sql .= 'GROUP BY contents_cid';
         if (!$result = $this->db->query($sql)) {
             return $ret;

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 +9 added lines, -9 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
 {
     /**
@@ -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 +11 added lines, -11 removed lines

@@ -42,8 +42,8 @@ 
     {
         $currentVersion = strtolower(str_replace('XOOPS ', '', XOOPS_VERSION));
         $requiredVersion = strtolower($module->getInfo('min_xoops'));
-        $vc = version_compare ($currentVersion , $requiredVersion);
-        $success = ($vc>=0);
+        $vc = version_compare($currentVersion, $requiredVersion);
+        $success = ($vc >= 0);
         if (false === $success) {
             xoops_loadLanguage('admin', $module->dirname());
             $module->setErrors(sprintf(_AM_XOOPSFAQ_ERROR_BAD_XOOPS, $requiredVersion, $currentVersion));
@@ -159,7 +159,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
     }
 
@@ -198,7 +198,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
     }
 
@@ -232,11 +232,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;
@@ -257,19 +257,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 . '"'

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

@@ -169,32 +169,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) {
             /** @var XoopsObject $object */
             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>'
-              . '</table>';
+                . '</table>';
         return $ret;
     }
 

blocks/xoopsfaq_recent.php

Indentation +11 added lines, -11 removed lines

@@ -95,7 +95,7 @@ 
                 );
             }
             $block['faq'][] = array('title' => $faqTitle,
-                                      'ans' => $faqAns,
+                                        'ans' => $faqAns,
                                 'published' => $faqObj->getPublished(_SHORTDATESTRING)
             );
         }
@@ -136,15 +136,15 @@ 
     $nchck = !empty($ychck) ? '' : ' checked';
 
     $form = '<div class="line140">'
-          . _MB_XOOPSFAQ_NUM_FAQS . '&nbsp;'
-          . '<input type="number" name="options[0]" value="' . $options[0] . '" style="width: 5em;" min="0" class="right"><br>'
-          . _MB_XOOPSFAQ_CHARS . '&nbsp;<input type="number" name="options[1]" value="' . $options[1] . '" style="width: 5em;" min="0" class="right">&nbsp;'
-          . _MB_XOOPSFAQ_LENGTH . '<br>' . _MB_XOOPSFAQ_SHOW_DATE . '&nbsp;'
-          . '<label for="r0">' . _NO . '</label>'
-          . '<input type="radio" name="options[2]" id="r0" value="0"' . $nchck . '>&nbsp;'
-          . '<label for="r1">' . _YES . '</label>'
-          . '<input type="radio" name="options[2]" id="r1" value="1"' . $ychck . '>'
-          . '<br><br>' . _MB_XOOPSFAQ_ALL_CATS_INTRO . '&nbsp;&nbsp;' . $selectCat
-          . '</div>';
+            . _MB_XOOPSFAQ_NUM_FAQS . '&nbsp;'
+            . '<input type="number" name="options[0]" value="' . $options[0] . '" style="width: 5em;" min="0" class="right"><br>'
+            . _MB_XOOPSFAQ_CHARS . '&nbsp;<input type="number" name="options[1]" value="' . $options[1] . '" style="width: 5em;" min="0" class="right">&nbsp;'
+            . _MB_XOOPSFAQ_LENGTH . '<br>' . _MB_XOOPSFAQ_SHOW_DATE . '&nbsp;'
+            . '<label for="r0">' . _NO . '</label>'
+            . '<input type="radio" name="options[2]" id="r0" value="0"' . $nchck . '>&nbsp;'
+            . '<label for="r1">' . _YES . '</label>'
+            . '<input type="radio" name="options[2]" id="r1" value="1"' . $ychck . '>'
+            . '<br><br>' . _MB_XOOPSFAQ_ALL_CATS_INTRO . '&nbsp;&nbsp;' . $selectCat
+            . '</div>';
     return $form;
 }

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
     $moduleDirName = basename(dirname(__DIR__));
     xoops_load('constants', $moduleDirName);
 
-    $myts         = MyTextSanitizer::getInstance();
+    $myts = MyTextSanitizer::getInstance();
 
     /** @var XoopsfaqCategoryHandler $xfCatHandler */
     /** @var XoopsfaqContentsHandler $xfFaqHandler */
@@ -84,7 +84,7 @@ 
             $faqTitle = $myts->displayTarea($faqObj->getVar('contents_title'));
             if (!empty($options[1])) {
                 $txtAns   = strip_tags($faqObj->getVar('contents_contents')); // get rid of html for block
-                $faqAns   = $myts->displayTarea(xoops_substr($txtAns, 0, $options[1]), 0, 0 ,0 ,0 ,0);
+                $faqAns   = $myts->displayTarea(xoops_substr($txtAns, 0, $options[1]), 0, 0, 0, 0, 0);
             } else {
                 $faqAns   = $myts->displayTarea($faqObj->getVar('contents_contents'),
                             (int)$faqObj->getVar('dohtml'),
@@ -130,7 +130,7 @@ 
     $formSelect->addOptionArray($optionArray);
     $selOptions = (false === strpos($options[3], ',')) ? $options[3] : explode(',', $options[3]);
     $formSelect->setValue($selOptions);
-    $selectCat   = $formSelect->render();
+    $selectCat = $formSelect->render();
 
     $ychck = (isset($options[2]) && ($options[2] > 0)) ? ' checked' : '';
     $nchck = !empty($ychck) ? '' : ' checked';

admin/menu.php

Indentation +12 added lines, -12 removed lines

@@ -28,23 +28,23 @@
 
 $adminmenu = array(
     array('title' => _MI_XOOPSFAQ_MENU_ADMIN_INDEX,
-           'link' => 'admin/index.php',
-           'desc' => _MI_XOOPSFAQ_ADMIN_INDEX_DESC,
-           'icon' => Admin::menuIconPath('home.png')
+            'link' => 'admin/index.php',
+            'desc' => _MI_XOOPSFAQ_ADMIN_INDEX_DESC,
+            'icon' => Admin::menuIconPath('home.png')
     ),
     array('title' => _MI_XOOPSFAQ_MENU_ADMIN_CATEGORY,
-           'link' => 'admin/category.php',
-           'desc' => _MI_XOOPSFAQ_ADMIN_CATEGORY_DESC,
-           'icon' => Admin::menuIconPath('category.png')
+            'link' => 'admin/category.php',
+            'desc' => _MI_XOOPSFAQ_ADMIN_CATEGORY_DESC,
+            'icon' => Admin::menuIconPath('category.png')
     ),
     array('title' => _MI_XOOPSFAQ_MENU_ADMIN_FAQ,
-           'link' => 'admin/main.php',
-           'desc' => _MI_XOOPSFAQ_ADMIN_FAQ_DESC,
-           'icon' => Admin::menuIconPath('faq.png')
+            'link' => 'admin/main.php',
+            'desc' => _MI_XOOPSFAQ_ADMIN_FAQ_DESC,
+            'icon' => Admin::menuIconPath('faq.png')
     ),
     array('title' => _MI_XOOPSFAQ_MENU_ADMIN_ABOUT,
-           'link' => 'admin/about.php',
-           'desc' => _MI_XOOPSFAQ_ADMIN_ABOUT_DESC,
-           'icon' => Admin::menuIconPath('about.png')
+            'link' => 'admin/about.php',
+            'desc' => _MI_XOOPSFAQ_ADMIN_ABOUT_DESC,
+            'icon' => Admin::menuIconPath('about.png')
     )
 );

include/rss.inc.php

Indentation +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     /** @var XoopsObject $contentObj */
     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

@@ -58,7 +58,7 @@
 
     /** @var XoopsObject $contentObj */
     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'),

index.php

Indentation +2 added lines, -2 removed lines

@@ -71,8 +71,8 @@
             /** @var XoopsObject $obj */
             foreach ($contentsObj['list'] as $obj) {
                 $question = array('id' => $obj->getVar('contents_id'),
-                               'title' => $obj->getVar('contents_title'),
-                              'answer' => $obj->getVar('contents_contents')
+                                'title' => $obj->getVar('contents_title'),
+                                'answer' => $obj->getVar('contents_contents')
                 );
                 $GLOBALS['xoopsTpl']->append('questions', $question);
                 $bodyWords .= ' ' . $obj->getVar('contents_title') . ' ' . $obj->getVar('contents_contents');