ggoffy / wgsitenotice

class/Common/FilesManagement.php

Spacing +2 added lines, -2 removed lines

@@ -207,7 +207,7 @@ 
                 self::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
     }
 
@@ -247,7 +247,7 @@ 
                 //                \rmdir($fObj->getPath()); // now delete the directory
             }
         }
-        $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
     }
 

admin/header.php

Spacing +3 added lines, -3 removed lines

@@ -23,9 +23,9 @@ 
 
 require \dirname(__DIR__) . '/preloads/autoloader.php';
 
-require_once \dirname(__DIR__, 3). '/include/cp_header.php';
+require_once \dirname(__DIR__, 3) . '/include/cp_header.php';
 $thisPath = \dirname(__DIR__);
-include_once $thisPath.'/include/common.php';
+include_once $thisPath . '/include/common.php';
 $sysPathIcon16 = '../' . $xoopsModule->getInfo('sysicons16');
 $sysPathIcon32 = '../' . $xoopsModule->getInfo('sysicons32');
 $pathModuleAdmin = $GLOBALS['xoopsModule']->getInfo('dirmoduleadmin');
@@ -58,7 +58,7 @@ 
 $helper->loadLanguage('modinfo');
 $helper->loadLanguage('common');
 // Local admin menu class
-include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
+include_once $GLOBALS['xoops']->path($pathModuleAdmin . '/moduleadmin.php');
 
 xoops_cp_header();
 $adminMenu = new ModuleAdmin();

config/config.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
  * return object
  */
 
-$moduleDirName  = \basename(\dirname(__DIR__));
-$moduleDirNameUpper  = \mb_strtoupper($moduleDirName);
+$moduleDirName = \basename(\dirname(__DIR__));
+$moduleDirNameUpper = \mb_strtoupper($moduleDirName);
 return (object)[
     'name'           => \mb_strtoupper($moduleDirName) . ' Module Configurator',
     'paths'          => [

include/functions.php

Spacing +9 added lines, -9 removed lines

@@ -27,13 +27,13 @@ 
  */
 function wgsitenotice_block_addCatSelect($cats) {
     $cat_sql = '';
-    if(\is_array($cats))
+    if (\is_array($cats))
     {
-        $cat_sql = '('.current($cats);
+        $cat_sql = '(' . current($cats);
         \array_shift($cats);
-        foreach($cats as $cat)
+        foreach ($cats as $cat)
         {
-            $cat_sql .= ','.$cat;
+            $cat_sql .= ',' . $cat;
         }
         $cat_sql .= ')';
     }
@@ -44,9 +44,9 @@ 
 {
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
-    $content= $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'keywords', \strip_tags($content));
+    $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'keywords', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_keywords', \strip_tags($content));
     }
@@ -57,8 +57,8 @@ 
     global $xoopsTpl, $xoTheme;
     $myts = MyTextSanitizer::getInstance();
     $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
-    if(isset($xoTheme) && \is_object($xoTheme)) {
-        $xoTheme->addMeta( 'meta', 'description', \strip_tags($content));
+    if (isset($xoTheme) && \is_object($xoTheme)) {
+        $xoTheme->addMeta('meta', 'description', \strip_tags($content));
     } else {    // Compatibility for old Xoops versions
         $xoopsTpl->assign('xoops_meta_description', \strip_tags($content));
     }

admin/versions.php

Spacing +9 added lines, -9 removed lines

@@ -28,8 +28,8 @@ 
 $version_id = Request::getInt('version_id');
 $start      = Request::getInt('start');
 
-$img_yes = "<img src='../".$modPathIcon16."/on.png' >";
-$img_no  = "<img src='../".$modPathIcon16."/off.png' >";
+$img_yes = "<img src='../" . $modPathIcon16 . "/on.png' >";
+$img_no  = "<img src='../" . $modPathIcon16 . "/off.png' >";
 // Switch options
 switch ($op)
 {
@@ -76,7 +76,7 @@ 
                 $GLOBALS['xoopsTpl']->append('versions_list', $version);
                 unset($version);
             }
-            if ( $versions_rows > $limit ) {
+            if ($versions_rows > $limit) {
                 include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
                 $pagenav = new \XoopsPageNav($versions_rows, $limit, $start, 'start', 'op=list&limit=' . $limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
@@ -99,7 +99,7 @@ 
         $GLOBALS['xoopsTpl']->assign('form', $form->render());
     break;
     case 'save':
-        if ( !$GLOBALS['xoopsSecurity']->check() ) {
+        if (!$GLOBALS['xoopsSecurity']->check()) {
            \redirect_header('versions.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         if (isset($version_id)) {
@@ -142,7 +142,7 @@ 
             $versionsObj->setVar('version_current', $version_current);
             // Insert Data
             if ($versionsHandler->insert($versionsObj)) {
-               \redirect_header('versions.php?op=list&start='.$start, 2, \_AM_WGSITENOTICE_FORMOK);
+               \redirect_header('versions.php?op=list&start=' . $start, 2, \_AM_WGSITENOTICE_FORMOK);
             }
         }
     break;
@@ -155,7 +155,7 @@ 
             $versionsObj->setVar('version_online', $version_online);
             // Insert Data
             if ($versionsHandler->insert($versionsObj)) {
-               \redirect_header('versions.php?op=list&start='.$start, 2, \_AM_WGSITENOTICE_FORMOK);
+               \redirect_header('versions.php?op=list&start=' . $start, 2, \_AM_WGSITENOTICE_FORMOK);
             }
         }
     break;
@@ -173,7 +173,7 @@ 
     case 'delete':
         $versionsObj = $versionsHandler->get($version_id);
         if (isset($_REQUEST['ok']) && 1 == (int)$_REQUEST['ok']) {
-            if ( !$GLOBALS['xoopsSecurity']->check() ) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
                 \redirect_header('versions.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
             }
 
@@ -195,9 +195,9 @@ 
 
     case 'order':
         $vorder = Request::getArray('vorder');
-        for ($i = 0, $iMax = \count($vorder); $i < $iMax; $i++){
+        for ($i = 0, $iMax = \count($vorder); $i < $iMax; $i++) {
             $versionsObj = $versionsHandler->get($vorder[$i]);
-            $versionsObj->setVar('version_weight',$i+1);
+            $versionsObj->setVar('version_weight', $i + 1);
             $versionsHandler->insert($versionsObj);
         }
         break;

admin/menu.php

Spacing +2 added lines, -2 removed lines

@@ -19,7 +19,7 @@ 
  * @author          Goffy (xoops.wedega.com) - Email:<webmaster@wedega.com> - Website:<https://xoops.wedega.com>
  */
 
-$dirname = \basename( \dirname(__DIR__) ) ;
+$dirname = \basename(\dirname(__DIR__));
 $moduleDirNameUpper = \mb_strtoupper($dirname);
 $moduleHandler = \xoops_getHandler('module');
 $xoopsModule = \XoopsModule::getByDirname($dirname);
@@ -29,7 +29,7 @@ 
 $adminmenu[] = [
     'title' => \_MI_WGSITENOTICE_ADMENU1,
     'link'  => 'admin/index.php',
-    'icon'  => $sysPathIcon32.'/dashboard.png'
+    'icon'  => $sysPathIcon32 . '/dashboard.png'
 ];
 $adminmenu[] = [
     'title' => \_MI_WGSITENOTICE_ADMENU2,

footer.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
 $GLOBALS['xoopsTpl']->assign('wgsitenotice_url', \WGSITENOTICE_URL);
 //
 $GLOBALS['xoopsTpl']->assign('admin', \WGSITENOTICE_ADMIN);
-if ( $helper->getConfig('show_copyright') ) {
+if ($helper->getConfig('show_copyright')) {
     $GLOBALS['xoopsTpl']->assign('copyright', $copyright);
 }
 // User footer
-include_once \XOOPS_ROOT_PATH.'/footer.php';
\ No newline at end of file
+include_once \XOOPS_ROOT_PATH . '/footer.php';
\ No newline at end of file

checkonline.php

Spacing +14 added lines, -14 removed lines

@@ -57,12 +57,12 @@ 
         echo "<versions>\n";
         foreach (\array_keys($versions_arr) as $i) {
             echo "\t<version>\n";
-            echo "\t<version_id>".$versions_arr[$i]->getVar('version_id')."</version_id>\n";
-            echo "\t<version_name>".$versions_arr[$i]->getVar('version_name')."</version_name>\n";
-            echo "\t<version_lang>".$versions_arr[$i]->getVar('version_lang')."</version_lang>\n";
-            echo "\t<version_descr>".$versions_arr[$i]->getVar('version_descr')."</version_descr>\n";
-            echo "\t<version_author>".$versions_arr[$i]->getVar('version_author')."</version_author>\n";
-            echo "\t<version_date>".$versions_arr[$i]->getVar('version_date')."</version_date>\n";
+            echo "\t<version_id>" . $versions_arr[$i]->getVar('version_id') . "</version_id>\n";
+            echo "\t<version_name>" . $versions_arr[$i]->getVar('version_name') . "</version_name>\n";
+            echo "\t<version_lang>" . $versions_arr[$i]->getVar('version_lang') . "</version_lang>\n";
+            echo "\t<version_descr>" . $versions_arr[$i]->getVar('version_descr') . "</version_descr>\n";
+            echo "\t<version_author>" . $versions_arr[$i]->getVar('version_author') . "</version_author>\n";
+            echo "\t<version_date>" . $versions_arr[$i]->getVar('version_date') . "</version_date>\n";
 
             if ($version_id > 0) {
                 echo "\t\t<contents>\n";
@@ -76,14 +76,14 @@ 
                 if ($contents_rows > 0) {
                     foreach (\array_keys($contents_arr) as $j) {
                         echo "\t\t\t<content>\n";
-                        echo "\t\t\t\t<cont_id>".$contents_arr[$j]->getVar('cont_id')."</cont_id>\n";
-                        echo "\t\t\t\t<cont_version_id>".$contents_arr[$j]->getVar('cont_version_id')."</cont_version_id>\n";
+                        echo "\t\t\t\t<cont_id>" . $contents_arr[$j]->getVar('cont_id') . "</cont_id>\n";
+                        echo "\t\t\t\t<cont_version_id>" . $contents_arr[$j]->getVar('cont_version_id') . "</cont_version_id>\n";
                         $xml = $contents_arr[$j]->getVar('cont_header');
-                        echo "\t\t\t\t<cont_header>".text2xml($xml)."</cont_header>\n";
+                        echo "\t\t\t\t<cont_header>" . text2xml($xml) . "</cont_header>\n";
                         $xml = $contents_arr[$j]->getVar('cont_text', 'n');
-                        echo "\t\t\t\t<cont_text>".text2xml($xml)."</cont_text>\n";
-                        echo "\t\t\t\t<cont_weight>".$contents_arr[$j]->getVar('cont_weight')."</cont_weight>\n";
-                        echo "\t\t\t\t<cont_date>".$contents_arr[$j]->getVar('cont_date')."</cont_date>\n";
+                        echo "\t\t\t\t<cont_text>" . text2xml($xml) . "</cont_text>\n";
+                        echo "\t\t\t\t<cont_weight>" . $contents_arr[$j]->getVar('cont_weight') . "</cont_weight>\n";
+                        echo "\t\t\t\t<cont_date>" . $contents_arr[$j]->getVar('cont_date') . "</cont_date>\n";
                         echo "\t\t\t</content>\n";
                     }
                 }
@@ -98,10 +98,10 @@ 
 }
 echo "</document>\n";
 
-function text2xml ($xml) {
+function text2xml($xml) {
     // replace html tags e.g. in case of links in the text
     $search = ['<', '>', '"'];
-    $replace  = ['&lt;', '&gt;', '&quot;'];
+    $replace = ['&lt;', '&gt;', '&quot;'];
 
     return \str_replace($search, $replace, (string)$xml);
 }

class/Versions.php

Spacing +7 added lines, -7 removed lines

@@ -80,9 +80,9 @@ 
         $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
         $form->setExtra('enctype="multipart/form-data"');
         // Form Text version_name
-        $form->addElement( new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_NAME, 'version_name', 50, 255, $this->getVar('version_name')), true );
+        $form->addElement(new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_NAME, 'version_name', 50, 255, $this->getVar('version_name')), true);
         // Form Text version_lang
-        $form->addElement( new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_LANG, 'version_lang', 50, 255, $this->getVar('version_lang')) );
+        $form->addElement(new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_LANG, 'version_lang', 50, 255, $this->getVar('version_lang')));
         // Form Text Area
         // Form Dhtml Text Area
         $editor_configs = [];
@@ -93,18 +93,18 @@ 
         $editor_configs['width'] = '100%';
         $editor_configs['height'] = '400px';
         $editor_configs['editor'] = $helper->getConfig('wgsitenotice_editor');
-        $form->addElement( new \XoopsFormEditor(\_AM_WGSITENOTICE_VERSION_DESCR, 'version_descr', $editor_configs) );
+        $form->addElement(new \XoopsFormEditor(\_AM_WGSITENOTICE_VERSION_DESCR, 'version_descr', $editor_configs));
         // Form Text version_author
         $version_author = $this->isNew() ? $GLOBALS['xoopsUser']->getVar('uname') : $this->getVar('version_author');
-        $form->addElement( new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_AUTHOR, 'version_author', 50, 255, $version_author) );
+        $form->addElement(new \XoopsFormText(\_AM_WGSITENOTICE_VERSION_AUTHOR, 'version_author', 50, 255, $version_author));
         // Form Text version_weight
-        $form->addElement( new \XoopsFormHidden('version_weight', $this->getVar('version_weight')) );
+        $form->addElement(new \XoopsFormHidden('version_weight', $this->getVar('version_weight')));
         // Form Radio Yes/No version_current
         $version_current = $this->isNew() ? 0 : $this->getVar('version_current');
-        $form->addElement( new \XoopsFormRadioYN(\_AM_WGSITENOTICE_VERSION_CURRENT, 'version_current', $version_current), true );
+        $form->addElement(new \XoopsFormRadioYN(\_AM_WGSITENOTICE_VERSION_CURRENT, 'version_current', $version_current), true);
         // Form Radio Yes/No
         $version_online = $this->isNew() ? 0 : $this->getVar('version_online');
-        $form->addElement( new \XoopsFormRadioYN(\_AM_WGSITENOTICE_VERSION_ONLINE, 'version_online', $version_online), true );
+        $form->addElement(new \XoopsFormRadioYN(\_AM_WGSITENOTICE_VERSION_ONLINE, 'version_online', $version_online), true);
         // Form Text Date Select
         //$form->addElement( new \XoopsFormTextDateSelect(\_AM_WGSITENOTICE_VERSION_DATE, 'version_date', '', $this->getVar('version_date')), true );
         // Send