XoopsModules25x / xoopsfaq

class/contents.php

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

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 . '"'