ggoffy / wgsitenotice

xoops_version.php

Spacing +6 added lines, -6 removed lines

@@ -38,10 +38,10 @@ 
     'help'                => 'page=help',
     'license_url'         => 'www.gnu.org/licenses/gpl-2.0.html/',
     'release_info'        => '',
-    'release_file'        => \XOOPS_URL."/modules/{$dirname}/docs/release_info file",
+    'release_file'        => \XOOPS_URL . "/modules/{$dirname}/docs/release_info file",
     'release_date'        => '2021/04/01',
     'manual'              => 'link to manual file',
-    'manual_file'         => \XOOPS_URL."/modules/{$dirname}/docs/install.txt",
+    'manual_file'         => \XOOPS_URL . "/modules/{$dirname}/docs/install.txt",
     'min_php'             => '7.4',
     'min_xoops'           => '2.5.11 Beta1',
     'min_admin'           => '1.1',
@@ -183,10 +183,10 @@ 
     'default'     => 1,
 ];
 
-$currdirname = isset($GLOBALS['xoopsModule'])&& \is_object($GLOBALS['xoopsModule']) ? $GLOBALS['xoopsModule']->getVar('dirname') : 'system';
+$currdirname = isset($GLOBALS['xoopsModule']) && \is_object($GLOBALS['xoopsModule']) ? $GLOBALS['xoopsModule']->getVar('dirname') : 'system';
 if ($dirname == $currdirname) {
-    $subcount = 1 ;
-    $pathname = \XOOPS_ROOT_PATH. '/modules/'.$dirname;
+    $subcount = 1;
+    $pathname = \XOOPS_ROOT_PATH . '/modules/' . $dirname;
     include_once $pathname . '/include/common.php';
     // Get instance of module
     $helper = Helper::getInstance();
@@ -203,7 +203,7 @@ 
         foreach (\array_keys($versions_arr) as $i)
         {
             $modversion['sub'][$subcount]['name'] = $versions_arr[$i]->getVar('version_name');
-            $modversion['sub'][$subcount++]['url'] = 'index.php?version_id=' . $versions_arr[$i]->getVar('version_id') ;
+            $modversion['sub'][$subcount++]['url'] = 'index.php?version_id=' . $versions_arr[$i]->getVar('version_id');
         }
     }
 }

class/Utility.php

Spacing +1 added lines, -1 removed lines

@@ -217,7 +217,7 @@
 //--></script>
 <!-- End Form Validation JavaScript //-->',
         ];
-        $paypalform   = [
+        $paypalform = [
             0 => '<form action="https://www.paypal.com/cgi-bin/webscr" method="post">',
             1 => '<input name="cmd" value="_s-xclick" type="hidden">',
             2 => '<input name="hosted_button_id" value="%s" type="hidden">',

checkonline.php

Braces +3 added lines, -1 removed lines

@@ -45,7 +45,9 @@
     $version_crit = new \CriteriaCompo();
     $version_crit->setSort('version_id ASC, version_name');
     $version_crit->setOrder('ASC');
-    if ($version_id > 0) $version_crit->add(new \Criteria('version_id', $version_id));
+    if ($version_id > 0) {
+        $version_crit->add(new \Criteria('version_id', $version_id));
+    }
     $version_crit->add(new \Criteria('version_online', '1'));
     $versions_rows = $versionsHandler->getCount($version_crit);
     echo '<versions_rows>' . $versions_rows . "</versions_rows>\n";

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);
 }

preloads/autoloader.php

Spacing +2 added lines, -2 removed lines

@@ -4,12 +4,12 @@
  * @see http://www.php-fig.org/psr/psr-4/examples/
  */
 spl_autoload_register(
-    static function ($class) {
+    static function($class) {
         // project-specific namespace prefix
         $prefix = 'XoopsModules\\' . \ucfirst(\basename(\dirname(__DIR__)));
 
         // base directory for the namespace prefix
-        $baseDir =  \dirname(__DIR__) . '/class/';
+        $baseDir = \dirname(__DIR__) . '/class/';
 
         // does the class use the namespace prefix?
         $len = \mb_strlen($prefix);

blocks/versions.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 
 use XoopsModules\Wgsitenotice\Helper;
 
-include_once \XOOPS_ROOT_PATH.'/modules/wgsitenotice/include/common.php';
+include_once \XOOPS_ROOT_PATH . '/modules/wgsitenotice/include/common.php';
 
 // Function show block
 function b_wgsitenotice_versions_show($options)
@@ -67,8 +67,8 @@ 
 {
 
     $form = \_MB_WGSITENOTICE_DISPLAY;
-    $form .= "<input name='options[0]' size='5' maxlength='255' value='".$options[0]."' type='text' />&nbsp;<br />";
-    $form .= \_MB_WGSITENOTICE_TITLELENGTH." : <input name='options[1]' size='5' maxlength='255' value='".$options[1]."' type='text' /><br /><br />";
+    $form .= "<input name='options[0]' size='5' maxlength='255' value='" . $options[0] . "' type='text' />&nbsp;<br />";
+    $form .= \_MB_WGSITENOTICE_TITLELENGTH . " : <input name='options[1]' size='5' maxlength='255' value='" . $options[1] . "' type='text' /><br /><br />";
     \array_shift($options);
     \array_shift($options);
 

admin/index.php

Spacing +2 added lines, -2 removed lines

@@ -27,8 +27,8 @@
 // InfoBox Statistics
 $adminMenu->addInfoBox(\_AM_WGSITENOTICE_STATISTICS);
 // Info elements
-$adminMenu->addInfoBoxLine(\_AM_WGSITENOTICE_STATISTICS, '<label>'.\_AM_WGSITENOTICE_THEREARE_VERSIONS.'</label>', $count_versions);
-$adminMenu->addInfoBoxLine(\_AM_WGSITENOTICE_STATISTICS, '<label>'.\_AM_WGSITENOTICE_THEREARE_CONTENTS.'</label>', $count_contents);
+$adminMenu->addInfoBoxLine(\_AM_WGSITENOTICE_STATISTICS, '<label>' . \_AM_WGSITENOTICE_THEREARE_VERSIONS . '</label>', $count_versions);
+$adminMenu->addInfoBoxLine(\_AM_WGSITENOTICE_STATISTICS, '<label>' . \_AM_WGSITENOTICE_THEREARE_CONTENTS . '</label>', $count_contents);
 // Render Index
 echo $adminMenu->addNavigation('index.php');
 echo $adminMenu->renderIndex();

index.php

Braces +6 added lines, -2 removed lines

@@ -38,7 +38,9 @@ 
 $criteriaVersions = new \CriteriaCompo();
 $criteriaVersions->setSort('version_weight');
 $criteriaVersions->setOrder('ASC');
-if ($version_id > 0) $criteriaVersions->add(new \Criteria('version_id', $version_id));
+if ($version_id > 0) {
+    $criteriaVersions->add(new \Criteria('version_id', $version_id));
+}
 $criteriaVersions->add(new \Criteria('version_current', '1'));
 $versions_count = $versionsHandler->getCount($criteriaVersions);
 $versions_arr = $versionsHandler->getAll($criteriaVersions);
@@ -50,7 +52,9 @@ 
 if ($versions_count > 0) {
     foreach (\array_keys($versions_arr) as $v) {
 
-        if ($version_id > 0) $breadcrumb_subdir = $versions_arr[$v]->getVar('version_name');
+        if ($version_id > 0) {
+            $breadcrumb_subdir = $versions_arr[$v]->getVar('version_name');
+        }
         $criteriaContents = new \CriteriaCompo();
         $criteriaContents->add(new \Criteria('cont_version_id', $versions_arr[$v]->getVar('version_id')));
         $contents_count = $contentsHandler->getCount($criteriaContents);

Spacing +7 added lines, -7 removed lines

@@ -28,12 +28,12 @@ 
 
 $GLOBALS['xoopsOption']['template_main'] = 'user/' . $helper->getConfig('wgsitenotice_template') . '/wgsitenotice_index_'
                                            . $helper->getConfig('wgsitenotice_template') . '.tpl';
-include_once \XOOPS_ROOT_PATH.'/header.php';
+include_once \XOOPS_ROOT_PATH . '/header.php';
 
 // Define Stylesheet
-$xoTheme->addStylesheet( $style );
+$xoTheme->addStylesheet($style);
 
-$breadcrumb ='<a href="' . \XOOPS_URL . '">' . \_YOURHOME . '</a>  &raquo; ' . $xoopsModule->name();
+$breadcrumb = '<a href="' . \XOOPS_URL . '">' . \_YOURHOME . '</a>  &raquo; ' . $xoopsModule->name();
 
 $version_id = Request::getInt('version_id');
 
@@ -83,21 +83,21 @@ 
 }
 
 /* create breadcrumb */
-$breadcrumb ='<a href="' . \XOOPS_URL . '">' . \_YOURHOME . '</a>  &raquo; ';
+$breadcrumb = '<a href="' . \XOOPS_URL . '">' . \_YOURHOME . '</a>  &raquo; ';
 if ('' == $breadcrumb_subdir) {
     $breadcrumb .= $xoopsModule->name();
 } else {
-    $breadcrumb .='<a href="' . \WGSITENOTICE_URL . '">' . $xoopsModule->name() . '</a>  &raquo; ' . $breadcrumb_subdir;
+    $breadcrumb .= '<a href="' . \WGSITENOTICE_URL . '">' . $xoopsModule->name() . '</a>  &raquo; ' . $breadcrumb_subdir;
 }
 $GLOBALS['xoopsTpl']->assign('breadcrumb', $breadcrumb);
 
 
 // keywords
-wgsitenotice_meta_keywords($helper->getConfig('keywords').', '. \implode(', ', $keywords));
+wgsitenotice_meta_keywords($helper->getConfig('keywords') . ', ' . \implode(', ', $keywords));
 unset($keywords);
 // description
 wgsitenotice_meta_description(\_MA_WGSITENOTICE_DESC);
 //
-$GLOBALS['xoopsTpl']->assign('xoops_mpageurl', \WGSITENOTICE_URL.'/contents.php');
+$GLOBALS['xoopsTpl']->assign('xoops_mpageurl', \WGSITENOTICE_URL . '/contents.php');
 //
 include_once __DIR__ . '/footer.php';

class/Checkonline.php

Spacing +2 added lines, -2 removed lines

@@ -74,9 +74,9 @@
         \xoops_load('XoopsFormLoader');
         $form = new \XoopsThemeForm($title, 'form', $action, 'post', true);
         $form->setExtra('enctype="multipart/form-data"');
-        $oc_server = $helper->getConfig('wgsitenotice_oc_server').'checkonline.php';
+        $oc_server = $helper->getConfig('wgsitenotice_oc_server') . 'checkonline.php';
         // Form Text oc_server
-        $form->addElement( new \XoopsFormText(\_MI_WGSITENOTICE_OC_SERVER, 'oc_server', 50, 255, $oc_server), true );
+        $form->addElement(new \XoopsFormText(\_MI_WGSITENOTICE_OC_SERVER, 'oc_server', 50, 255, $oc_server), true);
         // Send
         $form->addElement(new \XoopsFormHidden('op', 'checkonline'));
         $form->addElement(new \XoopsFormButton('', 'submit', \_AM_WGSITENOTICE_OC_START, 'submit'));

admin/checkonline.php

Spacing +11 added lines, -11 removed lines

@@ -27,7 +27,7 @@ 
 
 $oc_server = Request::getString('oc_server', 'default');
 if ('default' === $oc_server) {
-  $oc_server = $helper->getConfig('wgsitenotice_oc_server').'checkonline.php';
+  $oc_server = $helper->getConfig('wgsitenotice_oc_server') . 'checkonline.php';
 }
 
 $template_main = 'wgsitenotice_admin_checkonline.tpl';
@@ -46,19 +46,19 @@ 
         // Request version_id
         $version_id = Request::getInt('version_id');
         if (0 == $version_id) {
-            $GLOBALS['xoopsTpl']->assign('error',\_AM_WGSITENOTICE_OC_ERR_INVALID_PARAM);
+            $GLOBALS['xoopsTpl']->assign('error', \_AM_WGSITENOTICE_OC_ERR_INVALID_PARAM);
             break;
         }
         //$xml_text = \file_get_contents($oc_server.'?version_id='.$version_id);
-        $xml_text = $checkonlineHandler->getData($oc_server . '?version_id='.$version_id);
+        $xml_text = $checkonlineHandler->getData($oc_server . '?version_id=' . $version_id);
         // echo $xml_text;
         $xml_arr = $checkonlineHandler->readXML($xml_text);
         // var_dump($xml_arr);
         if (!$xml_arr) {
             //error when loading xml
-            $GLOBALS['xoopsTpl']->assign('error',\str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_READ_XML));
+            $GLOBALS['xoopsTpl']->assign('error', \str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_READ_XML));
         } else {
-            echo $oc_server.'?version_id='.$version_id;
+            echo $oc_server . '?version_id=' . $version_id;
             echo $xml_arr->status_access;
 
             if ('allowed' == $xml_arr->status_access) {
@@ -98,9 +98,9 @@ 
                     }
                 }
             } else if ('forbidden' == $xml_arr->status_access) {
-                $GLOBALS['xoopsTpl']->assign('error',\_AM_WGSITENOTICE_OC_ERR_FORBIDDEN);
+                $GLOBALS['xoopsTpl']->assign('error', \_AM_WGSITENOTICE_OC_ERR_FORBIDDEN);
             } else {
-                $GLOBALS['xoopsTpl']->assign('error',\str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_CONNECT));
+                $GLOBALS['xoopsTpl']->assign('error', \str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_CONNECT));
             }
         }
         break;
@@ -116,7 +116,7 @@ 
         // echo "<br/><br/>var_dump xml_arr:<br/>".var_dump($xml_arr);
 
         if (!$xml_arr) {
-            $GLOBALS['xoopsTpl']->assign('error',\str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_READ_XML));
+            $GLOBALS['xoopsTpl']->assign('error', \str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_READ_XML));
         } else {
             // echo "<br/>status_access:".$xml_arr->status_access;
             if ('allowed' == $xml_arr->status_access) {
@@ -132,12 +132,12 @@ 
                         $GLOBALS['xoopsTpl']->append('versions_list', $version);
                     }
                 } else {
-                    $GLOBALS['xoopsTpl']->assign('error',\_AM_WGSITENOTICE_OC_ERR_NO_VERSIONS);
+                    $GLOBALS['xoopsTpl']->assign('error', \_AM_WGSITENOTICE_OC_ERR_NO_VERSIONS);
                 }
             } else if ('forbidden' == $xml_arr->status_access) {
-                $GLOBALS['xoopsTpl']->assign('error',\_AM_WGSITENOTICE_OC_ERR_FORBIDDEN);
+                $GLOBALS['xoopsTpl']->assign('error', \_AM_WGSITENOTICE_OC_ERR_FORBIDDEN);
             } else {
-                $GLOBALS['xoopsTpl']->assign('error',\str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_CONNECT));
+                $GLOBALS['xoopsTpl']->assign('error', \str_replace('%s', $oc_server, \_AM_WGSITENOTICE_OC_ERR_CONNECT));
             }
         }
         unset($version);