ggoffy / wgsitenotice

class/CheckonlineHandler.php

Spacing +6 added lines, -6 removed lines

@@ -71,9 +71,9 @@ 
             //execute post
             $result = \curl_exec($ch);
             // print_r(\curl_getinfo($ch));
-            if (false == $result)  {
+            if (false == $result) {
                 //echo '<br>unexpected curl_error:' . \curl_error($ch) . '<br>';
-                $GLOBALS['xoopsTpl']->assign('error',\curl_error($ch));
+                $GLOBALS['xoopsTpl']->assign('error', \curl_error($ch));
             }
 
             \curl_close($ch);
@@ -86,7 +86,7 @@ 
                 ]
             ];
 
-            $context  = \stream_context_create($opts);
+            $context = \stream_context_create($opts);
             $result = \file_get_contents($oc_server, false, $context);
         }
         return $result;
@@ -98,7 +98,7 @@ 
      * @param string $xml_string
      * @return array|\SimpleXMLElement|string
      */
-    public function readXML(string $xml_string){
+    public function readXML(string $xml_string) {
         // creating temporary string for avoiding entitiy errors
         $xml_string = \str_replace('&', '[[avoid_entity_error]]', $xml_string);
         $xml_arr = \simplexml_load_string($xml_string);
@@ -108,7 +108,7 @@ 
             $errors = \libxml_get_errors();
             foreach ($errors as $error) {
                 //echo $this->display_xml_error($error, $xml);
-                $GLOBALS['xoopsTpl']->assign('error',$this->display_xml_error($error, $xml));
+                $GLOBALS['xoopsTpl']->assign('error', $this->display_xml_error($error, $xml));
             }
             \libxml_clear_errors();
         }
@@ -122,7 +122,7 @@ 
      * @param string $xml
      * @return string
      */
-    public function xml2str(string $xml){
+    public function xml2str(string $xml) {
         // replace temporary string for avoiding entitiy errors
         $str = \str_replace('[[avoid_entity_error]]', '&', (string)$xml);
         // rebuild html tags

class/Common/MigrateHelper.php

Spacing +11 added lines, -11 removed lines

@@ -94,7 +94,7 @@ 
                 $skip = false;
                 $options = '';
                 // start table definition
-                $tableName = $this->getTableName ($line);
+                $tableName = $this->getTableName($line);
                 $tables[$tableName] = [];
                 $tables[$tableName]['options'] = '';
                 $tables[$tableName]['columns'] = [];
@@ -183,10 +183,10 @@ 
      * @param  string $line
      * @return string|bool
      */
-    private function getTableName (string $line)
+    private function getTableName(string $line)
     {
 
-        $arrLine = \explode( '`', $line);
+        $arrLine = \explode('`', $line);
         if (\count($arrLine) > 0) {
             return $arrLine[1];
         }
@@ -201,12 +201,12 @@ 
      * @param string $line
      * @return array|bool
      */
-    private function getColumns (string $line)
+    private function getColumns(string $line)
     {
 
         $columns = [];
 
-        $arrCol = \explode( ' ', \trim($line));
+        $arrCol = \explode(' ', \trim($line));
         if (\count($arrCol) > 0) {
             $name = \str_replace(['`'], '', $arrCol[0]);
         } else {
@@ -217,9 +217,9 @@ 
         $columns['name'] = $name;
         // update quotes
         if (\strpos($attributes, "''") > 0) {
-            $attributes = \trim(\str_replace("''", "''''''''" , $attributes));
+            $attributes = \trim(\str_replace("''", "''''''''", $attributes));
         } elseif (\strpos($attributes, "'") > 0) {
-            $attributes = \trim(\str_replace("'", "''" , $attributes));
+            $attributes = \trim(\str_replace("'", "''", $attributes));
         }
         $columns['attributes'] = "' " . $attributes . " '";
 
@@ -234,7 +234,7 @@ 
      * @param string $options
      * @return void
      */
-    private function getOptions (string $line, string &$options): void
+    private function getOptions(string $line, string &$options): void
     {
 
         $lineText = \trim(\str_replace([')', ';'], '', $line));
@@ -253,7 +253,7 @@ 
      * @param string $line
      * @return array
      */
-    private function getKey (string $line)
+    private function getKey(string $line)
     {
 
         $key = [];
@@ -276,11 +276,11 @@ 
                 if ('' == $name) {
                     $name = $columns;
                 }
-                if (\strpos($name,' ') > 0) {
+                if (\strpos($name, ' ') > 0) {
                     $name = "'" . $name . "'";
                 }
                 $key[$name] = [];
-                if (\strpos($columns,' ') > 0) {
+                if (\strpos($columns, ' ') > 0) {
                     $columns = "'" . $columns . "'";
                 }
                 $key[$name]['columns'] = $columns;

header.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
 
 require_once \dirname(__DIR__, 2) . '/mainfile.php';
 $dirname = $GLOBALS['xoopsModule']->getVar('dirname');
-$pathname = \XOOPS_ROOT_PATH. '/modules/'.$dirname;
+$pathname = \XOOPS_ROOT_PATH . '/modules/' . $dirname;
 include_once $pathname . '/include/common.php';
 // Get instance of module
 $helper = Helper::getInstance();
@@ -38,7 +38,7 @@ 
 //
 $myts = MyTextSanitizer::getInstance();
 $style = \WGSITENOTICE_URL . '/assets/css/style.css';
-if(\file_exists($style)) { return true; }
+if (\file_exists($style)) { return true; }
 //
 $sysPathIcon16 = $GLOBALS['xoopsModule']->getInfo('sysicons16');
 $sysPathIcon32 = $GLOBALS['xoopsModule']->getInfo('sysicons32');
@@ -47,7 +47,7 @@ 
 $modPathIcon16 = $xoopsModule->getInfo('modicons16');
 $modPathIcon32 = $xoopsModule->getInfo('modicons32');
 // Breadcrumbs
-$xoBreadcrumbs   = [];
+$xoBreadcrumbs = [];
 //
 \xoops_loadLanguage('modinfo', $dirname);
 \xoops_loadLanguage('main', $dirname);

index.php

Spacing +5 added lines, -5 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');
 
@@ -93,11 +93,11 @@ 
 
 
 // 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';

admin/contents.php

Spacing +6 added lines, -6 removed lines

@@ -88,7 +88,7 @@ 
                 $GLOBALS['xoopsTpl']->append('contents_list', $cont);
                 unset($cont);
             }
-            if ( $contents_rows > $limit ) {
+            if ($contents_rows > $limit) {
                 include_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
                 $pagenav = new \XoopsPageNav($contents_rows, $limit, $start, 'start', 'op=list&limit=' . $limit);
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav());
@@ -108,7 +108,7 @@ 
         $GLOBALS['xoopsTpl']->assign('form', $form->render());
     break;
     case 'save':
-        if ( !$GLOBALS['xoopsSecurity']->check() ) {
+        if (!$GLOBALS['xoopsSecurity']->check()) {
            \redirect_header('contents.php', 3, \implode(',', $GLOBALS['xoopsSecurity']->getErrors()));
         }
         if (isset($cont_id)) {
@@ -123,7 +123,7 @@ 
         $contentsObj->setVar('cont_header', Request::getString('cont_header'));
         // Set Var cont_text
         //fix for avoid hiding empty paragraphs in some browsers (instead of: $contentsObj->setVar('cont_weight', $_POST['cont_weight']);
-        $cont_text =  Request::getString('cont_text', '', 'default', 2);
+        $cont_text = Request::getString('cont_text', '', 'default', 2);
         $contentsObj->setVar('cont_text', \preg_replace('/<p><\/p>/', '<p>&nbsp;</p>', $cont_text));
         // Set Var cont_weight
         $contentsObj->setVar('cont_weight', Request::getInt('cont_weight'));
@@ -152,7 +152,7 @@ 
     case 'delete':
         $contentsObj = $contentsHandler->get($cont_id);
         if (1 === Request::getInt('ok')) {
-            if ( !$GLOBALS['xoopsSecurity']->check() ) {
+            if (!$GLOBALS['xoopsSecurity']->check()) {
                 \redirect_header('contents.php', 3, \implode(', ', $GLOBALS['xoopsSecurity']->getErrors()));
             }
             if ($contentsHandler->delete($contentsObj)) {
@@ -167,9 +167,9 @@ 
 
     case 'order':
         $corder = Request::getArray('corder');
-        for ($i = 0, $iMax = \count($corder); $i < $iMax; $i++){
+        for ($i = 0, $iMax = \count($corder); $i < $iMax; $i++) {
             $contentsObj = $contentsHandler->get($corder[$i]);
-            $contentsObj->setVar('cont_weight',$i+1);
+            $contentsObj->setVar('cont_weight', $i + 1);
             $contentsHandler->insert($contentsObj);
         }
         break;

xoops_version.php

Spacing +6 added lines, -6 removed lines

@@ -39,10 +39,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'        => '2023/04/08', // format: yyyy/mm/dd
     '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 RC1',
     'min_admin'           => '1.1',
@@ -184,10 +184,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();
@@ -204,7 +204,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');
         }
     }
 }