ggoffy / wgsitenotice

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 +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';

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

blocks/cookie-reg.php

Spacing +8 added lines, -8 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_cookie_reg_show($options)
@@ -71,7 +71,7 @@ 
         $block['position'] = '';
     }
 
-    if ( $dataprotect_id > 0 || $cookie_reg_id > 0 ) {
+    if ($dataprotect_id > 0 || $cookie_reg_id > 0) {
         $dataprotect_obj = $versionsHandler->get($dataprotect_id);
         if (\is_object($dataprotect_obj)) {
             $block['dataprotect_id'] = $dataprotect_id;
@@ -83,18 +83,18 @@ 
             $block['cookie_reg_text'] = $cookie_reg_obj->getVar('version_name');
         }
         $block['seperator'] = "";
-        if ( $dataprotect_id > 0 && $cookie_reg_id > 0 ) {
+        if ($dataprotect_id > 0 && $cookie_reg_id > 0) {
             $block['seperator'] = "|";
         }
     }
-    $tplSource = \XOOPS_ROOT_PATH.'/modules/wgsitenotice/templates/blocks/wgsitenotice_block_cookie_reg.tpl';
+    $tplSource = \XOOPS_ROOT_PATH . '/modules/wgsitenotice/templates/blocks/wgsitenotice_block_cookie_reg.tpl';
 
     if ("smarty" == $display_type) {
         //hide block and show in theme.html defined position
         $blockTpl = new \XoopsTpl();
         $blockTpl->assign('block', $block);
 
-        $GLOBALS['xoopsTpl']->assign( $type_tpl_id, $blockTpl->fetch($tplSource) );
+        $GLOBALS['xoopsTpl']->assign($type_tpl_id, $blockTpl->fetch($tplSource));
         $block = false;
     }
     return $block;
@@ -116,7 +116,7 @@ 
 
     $cookie = $options[0];
     //remove existing cookie in order to see results
-    \setcookie($cookie,"0");
+    \setcookie($cookie, "0");
 
     $unique_id = new \XoopsFormText('', 'options[0]', 100, 255, $cookie);
     $form = \_MB_WGSITENOTICE_COOKIE_REG_NAME . ': ' . $unique_id->render() . '<br><br>';
@@ -141,7 +141,7 @@ 
     $form .= \_MB_WGSITENOTICE_COOKIE_REG_DISPLAY . ': ' . $display_sel->render() . '<br>';
 
     $form .= '<br>' . \_MB_WGSITENOTICE_COOKIE_REG_OPT_STYLE . ':<br>';
-    $form .= \str_repeat( '-' , \strlen(\_MB_WGSITENOTICE_COOKIE_REG_OPT_STYLE) + 5 ) . '<br>';
+    $form .= \str_repeat('-', \strlen(\_MB_WGSITENOTICE_COOKIE_REG_OPT_STYLE) + 5) . '<br>';
 
     $opacity_sel = new \XoopsFormSelect("", 'options[4]', $options[4]);
     $opacity_sel->addOption('0.1', '0.1');
@@ -167,7 +167,7 @@ 
     $form .= \_MB_WGSITENOTICE_COOKIE_REG_HEIGHT . ': ' . $height_sel->render() . '<br><br>';
 
     $form .= '<br>' . \_MB_WGSITENOTICE_COOKIE_REG_OPT_SMARTY . ':<br>';
-    $form .= \str_repeat( '-' , \strlen(\_MB_WGSITENOTICE_COOKIE_REG_OPT_SMARTY) + 5 ) . '<br>';
+    $form .= \str_repeat('-', \strlen(\_MB_WGSITENOTICE_COOKIE_REG_OPT_SMARTY) + 5) . '<br>';
 
     $position_sel = new \XoopsFormSelect("", 'options[9]', $options[9]);
     $position_sel->addOption('top', \_MB_WGSITENOTICE_COOKIE_REG_POSITION_TOP);