mambax7 / altsys

xoops_trust_path/libs/altsys/include/mygrouppermform.php

Indentation +14 added lines, -14 removed lines

@@ -451,20 +451,20 @@
         }
 
         $tree .= '>'
-                 . $option['name']
-                 . '<input type="hidden" name="'
-                 . $this->getName()
-                 . '[parents]['
-                 . $option['id']
-                 . ']" value="'
-                 . implode(':', $parentIds)
-                 . '"><input type="hidden" name="'
-                 . $this->getName()
-                 . '[itemname]['
-                 . $option['id']
-                 . ']" value="'
-                 . htmlspecialchars($option['name'], ENT_QUOTES | ENT_HTML5)
-                 . "\"><br>\n";
+                    . $option['name']
+                    . '<input type="hidden" name="'
+                    . $this->getName()
+                    . '[parents]['
+                    . $option['id']
+                    . ']" value="'
+                    . implode(':', $parentIds)
+                    . '"><input type="hidden" name="'
+                    . $this->getName()
+                    . '[itemname]['
+                    . $option['id']
+                    . ']" value="'
+                    . htmlspecialchars($option['name'], ENT_QUOTES | ENT_HTML5)
+                    . "\"><br>\n";
 
         if (isset($option['children'])) {
             foreach ($option['children'] as $child) {

Spacing +22 added lines, -22 removed lines

@@ -18,11 +18,11 @@ 
     exit;
 }
 
-require_once XOOPS_ROOT_PATH . '/class/xoopsform/formelement.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsform/formhidden.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsform/formbutton.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsform/formelementtray.php';
-require_once XOOPS_ROOT_PATH . '/class/xoopsform/form.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsform/formelement.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsform/formhidden.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsform/formbutton.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsform/formelementtray.php';
+require_once XOOPS_ROOT_PATH.'/class/xoopsform/form.php';
 
 /**
  * Renders a form for setting module specific group permissions
@@ -79,7 +79,7 @@ 
 
         parent::__construct($title, 'groupperm_form', '', 'post');
 
-        $this->_modid = (int) $modid;
+        $this->_modid = (int)$modid;
 
         $this->_permName = $permname;
 
@@ -171,13 +171,13 @@ 
 
             $selected = $grouppermHandler->getItemIds($this->_permName, $i, $this->_modid);
 
-            $ele = new MyXoopsGroupFormCheckBox($glist[$i], 'perms[' . $this->_permName . ']', $i, $selected);
+            $ele = new MyXoopsGroupFormCheckBox($glist[$i], 'perms['.$this->_permName.']', $i, $selected);
 
             $ele->setOptionTree($this->_itemTree);
 
             // GIJ start
 
-            $ele->setDescription('<input type="checkbox" onclick="with(document.groupperm_form){for(i=0;i<length;i++){if(elements[i].name.match(/^perms\[(module_admin|module_read|block_read)\]\[groups\]\[' . $i . '\]/)){elements[i].checked=this.checked;}}};">');
+            $ele->setDescription('<input type="checkbox" onclick="with(document.groupperm_form){for(i=0;i<length;i++){if(elements[i].name.match(/^perms\[(module_admin|module_read|block_read)\]\[groups\]\['.$i.'\]/)){elements[i].checked=this.checked;}}};">');
 
             // GIJ_end
 
@@ -220,9 +220,9 @@ 
 
         $this->addElement($tray);
 
-        $ret = '<h4>' . $this->getTitle() . '</h4>' . $this->_permDesc . '<br>';
+        $ret = '<h4>'.$this->getTitle().'</h4>'.$this->_permDesc.'<br>';
 
-        $ret .= "<form name='" . $this->getName() . "' id='" . $this->getName() . "' action='" . $this->getAction() . "' method='" . $this->getMethod() . "'" . $this->getExtra() . ">\n<table width='100%' class='outer' cellspacing='1'>\n";
+        $ret .= "<form name='".$this->getName()."' id='".$this->getName()."' action='".$this->getAction()."' method='".$this->getMethod()."'".$this->getExtra().">\n<table width='100%' class='outer' cellspacing='1'>\n";
 
         $elements = &$this->getElements();
 
@@ -232,21 +232,21 @@ 
             } elseif (!$elements[$i]->isHidden()) {
                 // group name
 
-                $ret .= "<tr valign='top' align='" . _GLOBAL_LEFT . "'><td class='head'>" . $elements[$i]->getCaption();
+                $ret .= "<tr valign='top' align='"._GLOBAL_LEFT."'><td class='head'>".$elements[$i]->getCaption();
 
                 // group description
 
                 if ('' != $elements[$i]->getDescription()) {
-                    $ret .= '<br><br><span style="font-weight: normal;">' . $elements[$i]->getDescription() . '</span>';
+                    $ret .= '<br><br><span style="font-weight: normal;">'.$elements[$i]->getDescription().'</span>';
                 }
 
-                $ret .= "</td>\n<td class='even'>\n" . $elements[$i]->render() . "\n</td></tr>\n";
+                $ret .= "</td>\n<td class='even'>\n".$elements[$i]->render()."\n</td></tr>\n";
             } else {
                 $ret .= $elements[$i]->render();
             }
         }
 
-        $ret .= '</table>' . $xoopsGTicket->getTicketHtml(__LINE__, 1800, 'myblocksadmin') . '</form>';
+        $ret .= '</table>'.$xoopsGTicket->getTicketHtml(__LINE__, 1800, 'myblocksadmin').'</form>';
 
         return $ret;
     }
@@ -363,7 +363,7 @@ 
 
                 $checked = $append['selected'] ? 'checked' : '';
 
-                $name = 'perms[' . $append['permname'] . ']';
+                $name = 'perms['.$append['permname'].']';
 
                 $itemid = $append['itemid'];
 
@@ -395,7 +395,7 @@ 
 
                 $this->_renderOptionTree($tree, $this->_optionTree[$topitem], $prefix);
 
-                $ret .= $tree . '</td>';
+                $ret .= $tree.'</td>';
 
                 $cols++;
             }
@@ -416,7 +416,7 @@ 
      */
     private function _renderOptionTree(&$tree, $option, $prefix, $parentIds = []): void
     {
-        $tree .= $prefix . '<input type="checkbox" name="' . $this->getName() . '[groups][' . $this->_groupId . '][' . $option['id'] . ']" id="' . $this->getName() . '[groups][' . $this->_groupId . '][' . $option['id'] . ']" onclick="';
+        $tree .= $prefix.'<input type="checkbox" name="'.$this->getName().'[groups]['.$this->_groupId.']['.$option['id'].']" id="'.$this->getName().'[groups]['.$this->_groupId.']['.$option['id'].']" onclick="';
 
         // If there are parent elements, add javascript that will
 
@@ -425,9 +425,9 @@ 
         // sure permissions to parent items are added as well.
 
         foreach ($parentIds as $pid) {
-            $parent_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $pid . ']';
+            $parent_ele = $this->getName().'[groups]['.$this->_groupId.']['.$pid.']';
 
-            $tree .= "var ele = xoopsGetElementById('" . $parent_ele . "'); if(ele.checked !== true) {ele.checked = this.checked;}";
+            $tree .= "var ele = xoopsGetElementById('".$parent_ele."'); if(ele.checked !== true) {ele.checked = this.checked;}";
         }
 
         // If there are child elements, add javascript that will
@@ -439,9 +439,9 @@ 
         // is no permission to this item.
 
         foreach ($option['allchild'] as $cid) {
-            $child_ele = $this->getName() . '[groups][' . $this->_groupId . '][' . $cid . ']';
+            $child_ele = $this->getName().'[groups]['.$this->_groupId.']['.$cid.']';
 
-            $tree .= "var ele = xoopsGetElementById('" . $child_ele . "'); if(this.checked !== true) {ele.checked = false;}";
+            $tree .= "var ele = xoopsGetElementById('".$child_ele."'); if(this.checked !== true) {ele.checked = false;}";
         }
 
         $tree .= '" value="1"';
@@ -470,7 +470,7 @@ 
             foreach ($option['children'] as $child) {
                 $parentIds[] = $option['id'];
 
-                $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix . '&nbsp;-', $parentIds);
+                $this->_renderOptionTree($tree, $this->_optionTree[$child], $prefix.'&nbsp;-', $parentIds);
             }
         }
     }

xoops_trust_path/libs/altsys/include/Legacy_AltsysAdminRenderSystem.class.php

Spacing +3 added lines, -3 removed lines

@@ -1,8 +1,8 @@
 <?php declare(strict_types=1);
 
-require_once XOOPS_ROOT_PATH . '/modules/legacyRender/kernel/Legacy_AdminRenderSystem.class.php';
-require_once XOOPS_TRUST_PATH . '/libs/altsys/include/altsys_functions.php';
-require_once XOOPS_TRUST_PATH . '/libs/altsys/include/admin_in_theme_functions.php';
+require_once XOOPS_ROOT_PATH.'/modules/legacyRender/kernel/Legacy_AdminRenderSystem.class.php';
+require_once XOOPS_TRUST_PATH.'/libs/altsys/include/altsys_functions.php';
+require_once XOOPS_TRUST_PATH.'/libs/altsys/include/admin_in_theme_functions.php';
 
 /**
  * Class Legacy_AltsysAdminRenderSystem

xoops_trust_path/libs/altsys/include/admin_in_theme_functions.php

Indentation +14 added lines, -14 removed lines

@@ -162,20 +162,20 @@
     // assignment
 
     $xoopsTpl->assign([
-                          'xoops_theme' => $xoopsConfig['theme_set'],
-                          'xoops_imageurl' => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
-                          'xoops_themecss' => xoops_getcss($xoopsConfig['theme_set']),
-                          'xoops_requesturi' => htmlspecialchars($GLOBALS['xoopsRequestUri'], ENT_QUOTES | ENT_HTML5),
-                          'xoops_sitename' => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES | ENT_HTML5),
-                          'xoops_showlblock' => 1,
-                          //        'xoops_js' => '//--></script><script type="text/javascript" src="'.XOOPS_URL.'/include/xoops.js"></script><script type="text/javascript" src="'.XOOPS_URL.'/include/layersmenu.js"></script><script type="text/javascript"><!--'."\n".$xoops_admin_menu_js ,
-                          'xoops_js' => '//--></script><script type="text/javascript" src="' . XOOPS_URL . '/include/xoops.js"></script><script type="text/javascript"><!--' . "\n",
-                          'xoops_runs_admin_side' => 1,
-                          'xoops_breadcrumbs' => $xoops_breadcrumbs,
-                          'xoops_slogan' => htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES | ENT_HTML5),
-                          'xoops_contents' => $xoops_admin_contents, //. '<div id="adminmenu_layers">' . $xoops_admin_menu_dv . '</div>' ,
-                          'xoops_module_header' => $xoops_module_header,
-                      ]);
+                            'xoops_theme' => $xoopsConfig['theme_set'],
+                            'xoops_imageurl' => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
+                            'xoops_themecss' => xoops_getcss($xoopsConfig['theme_set']),
+                            'xoops_requesturi' => htmlspecialchars($GLOBALS['xoopsRequestUri'], ENT_QUOTES | ENT_HTML5),
+                            'xoops_sitename' => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES | ENT_HTML5),
+                            'xoops_showlblock' => 1,
+                            //        'xoops_js' => '//--></script><script type="text/javascript" src="'.XOOPS_URL.'/include/xoops.js"></script><script type="text/javascript" src="'.XOOPS_URL.'/include/layersmenu.js"></script><script type="text/javascript"><!--'."\n".$xoops_admin_menu_js ,
+                            'xoops_js' => '//--></script><script type="text/javascript" src="' . XOOPS_URL . '/include/xoops.js"></script><script type="text/javascript"><!--' . "\n",
+                            'xoops_runs_admin_side' => 1,
+                            'xoops_breadcrumbs' => $xoops_breadcrumbs,
+                            'xoops_slogan' => htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES | ENT_HTML5),
+                            'xoops_contents' => $xoops_admin_contents, //. '<div id="adminmenu_layers">' . $xoops_admin_menu_dv . '</div>' ,
+                            'xoops_module_header' => $xoops_module_header,
+                        ]);
 
     // rendering
 

Spacing +15 added lines, -15 removed lines

@@ -46,7 +46,7 @@ 
         return $s;
     }
 
-    [$tmp_s, $tmp_after] = explode("<td width='1%' background='" . XOOPS_URL . "/modules/system/images/bg_content.gif'>", $tmp_s);
+    [$tmp_s, $tmp_after] = explode("<td width='1%' background='".XOOPS_URL."/modules/system/images/bg_content.gif'>", $tmp_s);
 
     if (empty($tmp_after)) {
         define('ALTSYS_DONT_USE_ADMIN_IN_THEME', 1);
@@ -54,7 +54,7 @@ 
         return $s;
     }
 
-    $xoops_admin_contents = $former_outputs . mb_substr(strrev(mb_strstr(strrev($tmp_s), strrev('</div>'))), 0, -6);
+    $xoops_admin_contents = $former_outputs.mb_substr(strrev(mb_strstr(strrev($tmp_s), strrev('</div>'))), 0, -6);
 
     return '';
 }
@@ -88,10 +88,10 @@ 
 
     // language files
 
-    if (is_file(dirname(__DIR__) . '/language/' . $xoopsConfig['language'] . '/admin_in_theme.php')) {
-        require_once dirname(__DIR__) . '/language/' . $xoopsConfig['language'] . '/admin_in_theme.php';
+    if (is_file(dirname(__DIR__).'/language/'.$xoopsConfig['language'].'/admin_in_theme.php')) {
+        require_once dirname(__DIR__).'/language/'.$xoopsConfig['language'].'/admin_in_theme.php';
     } else {
-        require_once dirname(__DIR__) . '/language/english/admin_in_theme.php';
+        require_once dirname(__DIR__).'/language/english/admin_in_theme.php';
     }
 
     // set the theme
@@ -104,15 +104,15 @@ 
 
     error_reporting($original_error_level & ~E_NOTICE);
 
-    if (is_file(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/' . $xoopsConfig['language'] . '.php')) {
-        require_once XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/' . $xoopsConfig['language'] . '.php';
-    } elseif (is_file(XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/english.php')) {
-        require_once XOOPS_THEME_PATH . '/' . $xoopsConfig['theme_set'] . '/language/english.php';
+    if (is_file(XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/'.$xoopsConfig['language'].'.php')) {
+        require_once XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/'.$xoopsConfig['language'].'.php';
+    } elseif (is_file(XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/english.php')) {
+        require_once XOOPS_THEME_PATH.'/'.$xoopsConfig['theme_set'].'/language/english.php';
     }
 
     error_reporting($original_error_level);
 
-    require __DIR__ . '/admin_in_theme_header.inc.php';
+    require __DIR__.'/admin_in_theme_header.inc.php';
 
     /*  // include adminmenu
         include XOOPS_CACHE_PATH.'/adminmenu.php' ;
@@ -152,10 +152,10 @@ 
     $xoops_module_header = '';
 
     if (ALTSYS_CORE_TYPE_XCL21 == altsys_get_core_type()) {
-        $xoops_module_header .= '<link rel="stylesheet" type="text/css" media="all" href="' . XOOPS_URL . '/modules/legacyRender/admin/css.php?file=style.css">' . "\n";
+        $xoops_module_header .= '<link rel="stylesheet" type="text/css" media="all" href="'.XOOPS_URL.'/modules/legacyRender/admin/css.php?file=style.css">'."\n";
 
         if (is_object(@$xoopsModule)) {
-            $xoops_module_header .= '<link rel="stylesheet" type="text/css" media="all" href="' . XOOPS_URL . '/modules/legacyRender/admin/css.php?file=module.css&amp;dirname=' . $xoopsModule->getVar('dirname') . '">' . "\n";
+            $xoops_module_header .= '<link rel="stylesheet" type="text/css" media="all" href="'.XOOPS_URL.'/modules/legacyRender/admin/css.php?file=module.css&amp;dirname='.$xoopsModule->getVar('dirname').'">'."\n";
         }
     }
 
@@ -163,13 +163,13 @@ 
 
     $xoopsTpl->assign([
                           'xoops_theme' => $xoopsConfig['theme_set'],
-                          'xoops_imageurl' => XOOPS_THEME_URL . '/' . $xoopsConfig['theme_set'] . '/',
+                          'xoops_imageurl' => XOOPS_THEME_URL.'/'.$xoopsConfig['theme_set'].'/',
                           'xoops_themecss' => xoops_getcss($xoopsConfig['theme_set']),
                           'xoops_requesturi' => htmlspecialchars($GLOBALS['xoopsRequestUri'], ENT_QUOTES | ENT_HTML5),
                           'xoops_sitename' => htmlspecialchars($xoopsConfig['sitename'], ENT_QUOTES | ENT_HTML5),
                           'xoops_showlblock' => 1,
                           //        'xoops_js' => '//--></script><script type="text/javascript" src="'.XOOPS_URL.'/include/xoops.js"></script><script type="text/javascript" src="'.XOOPS_URL.'/include/layersmenu.js"></script><script type="text/javascript"><!--'."\n".$xoops_admin_menu_js ,
-                          'xoops_js' => '//--></script><script type="text/javascript" src="' . XOOPS_URL . '/include/xoops.js"></script><script type="text/javascript"><!--' . "\n",
+                          'xoops_js' => '//--></script><script type="text/javascript" src="'.XOOPS_URL.'/include/xoops.js"></script><script type="text/javascript"><!--'."\n",
                           'xoops_runs_admin_side' => 1,
                           'xoops_breadcrumbs' => $xoops_breadcrumbs,
                           'xoops_slogan' => htmlspecialchars($xoopsConfig['slogan'], ENT_QUOTES | ENT_HTML5),
@@ -179,7 +179,7 @@ 
 
     // rendering
 
-    $xoopsTpl->display($xoopsConfig['theme_set'] . '/theme.html');
+    $xoopsTpl->display($xoopsConfig['theme_set'].'/theme.html');
 
     // for XOOPS 2.0.14/15/16 from xoops.org
 

xoops_trust_path/libs/altsys/include/Text_Diff_Renderer_unified.php

Spacing +3 added lines, -3 removed lines

@@ -29,11 +29,11 @@ 
     public function _blockHeader($xbeg, $xlen, $ybeg, $ylen)
     {
         if (1 != $xlen) {
-            $xbeg .= ',' . $xlen;
+            $xbeg .= ','.$xlen;
         }
 
         if (1 != $ylen) {
-            $ybeg .= ',' . $ylen;
+            $ybeg .= ','.$ylen;
         }
 
         return "@@ -$xbeg +$ybeg @@";
@@ -64,6 +64,6 @@ 
      */
     public function _changed($orig, $final)
     {
-        return $this->_deleted($orig) . $this->_added($final);
+        return $this->_deleted($orig).$this->_added($final);
     }
 }

xoops_trust_path/libs/altsys/include/adminmenu_functions.php

Indentation +29 added lines, -29 removed lines

@@ -106,14 +106,14 @@ 
     // write back
 
     altsys_adminmenu_save_x20([
-                                  'xoops_admin_menu_js' => $xoops_admin_menu_js,
-                                  'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
-                                  'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
-                                  'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
-                                  'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
-                                  'altsys_adminmenu_ft_hacked' => (int) (@$altsys_adminmenu_ft_hacked),
-                                  'altsys_adminmenu_dv_updated' => true,
-                              ]);
+                                    'xoops_admin_menu_js' => $xoops_admin_menu_js,
+                                    'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
+                                    'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
+                                    'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
+                                    'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
+                                    'altsys_adminmenu_ft_hacked' => (int) (@$altsys_adminmenu_ft_hacked),
+                                    'altsys_adminmenu_dv_updated' => true,
+                                ]);
 }
 
 //
@@ -223,13 +223,13 @@ 
     // write back
 
     altsys_adminmenu_save_x20([
-                                  'xoops_admin_menu_js' => $xoops_admin_menu_js,
-                                  'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
-                                  'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
-                                  'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
-                                  'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
-                                  'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_2COL,
-                              ]);
+                                    'xoops_admin_menu_js' => $xoops_admin_menu_js,
+                                    'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
+                                    'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
+                                    'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
+                                    'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
+                                    'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_2COL,
+                                ]);
 }
 
 function altsys_adminmenu_hack_ft_noimg_x20(): void
@@ -291,13 +291,13 @@ 
     // write back
 
     altsys_adminmenu_save_x20([
-                                  'xoops_admin_menu_js' => $xoops_admin_menu_js,
-                                  'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
-                                  'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
-                                  'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
-                                  'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
-                                  'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_NOIMG,
-                              ]);
+                                    'xoops_admin_menu_js' => $xoops_admin_menu_js,
+                                    'xoops_admin_menu_ml' => $xoops_admin_menu_ml,
+                                    'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
+                                    'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
+                                    'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
+                                    'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_NOIMG,
+                                ]);
 }
 
 function altsys_adminmenu_hack_ft_xcsty_x20(): void
@@ -426,13 +426,13 @@ 
     // write back
 
     altsys_adminmenu_save_x20([
-                                  'xoops_admin_menu_js' => $xoops_admin_menu_js,
-                                  'xoops_admin_menu_ml' => [],
-                                  'xoops_admin_menu_sd' => [],
-                                  'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
-                                  'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
-                                  'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_XCSTY,
-                              ]);
+                                    'xoops_admin_menu_js' => $xoops_admin_menu_js,
+                                    'xoops_admin_menu_ml' => [],
+                                    'xoops_admin_menu_sd' => [],
+                                    'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
+                                    'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
+                                    'altsys_adminmenu_ft_hacked' => ALTSYS_ADMINMENU_HACK_XCSTY,
+                                ]);
 }
 
 //

Spacing +31 added lines, -31 removed lines

@@ -1,7 +1,7 @@ 
 <?php declare(strict_types=1);
-require_once dirname(__DIR__) . '/class/altsysUtils.class.php';
+require_once dirname(__DIR__).'/class/altsysUtils.class.php';
 
-define('ALTSYS_ADMINMENU_FILE', XOOPS_CACHE_PATH . '/adminmenu.php');
+define('ALTSYS_ADMINMENU_FILE', XOOPS_CACHE_PATH.'/adminmenu.php');
 define('ALTSYS_ADMINMENU_HACK_NONE', 0);
 define('ALTSYS_ADMINMENU_HACK_2COL', 1);
 define('ALTSYS_ADMINMENU_HACK_NOIMG', 2);
@@ -34,7 +34,7 @@ 
     // read
 
     if (!is_file(ALTSYS_ADMINMENU_FILE)) {
-        redirect_header(XOOPS_URL . '/admin.php', 1, 'Rebuild adminmenu');
+        redirect_header(XOOPS_URL.'/admin.php', 1, 'Rebuild adminmenu');
 
         exit;
     }
@@ -47,7 +47,7 @@ 
 
     $mid = $module->getVar('mid');
 
-    $anchor = '<!-- ALTSYS ANCHOR ' . $dirname . ' -->';
+    $anchor = '<!-- ALTSYS ANCHOR '.$dirname.' -->';
 
     // fetch popup_no
 
@@ -59,17 +59,17 @@ 
         return;
     }
 
-    $popup_no = (int) $regs[1];
+    $popup_no = (int)$regs[1];
 
     // replace
 
-    $search = '<img src=\'' . XOOPS_URL . '/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\'' . XOOPS_URL . '/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod=' . $mid . '\'';
+    $search = '<img src=\''.XOOPS_URL.'/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\''.XOOPS_URL.'/modules/system/admin.php?fct=preferences&amp;op=showmod&amp;mod='.$mid.'\'';
 
-    $replace = $anchor . '<img src=\'' . XOOPS_URL . '/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\'' . XOOPS_URL . '/modules/' . $dirname . '/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mypreferences\'';
+    $replace = $anchor.'<img src=\''.XOOPS_URL.'/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\''.XOOPS_URL.'/modules/'.$dirname.'/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mypreferences\'';
 
     // do replacement
 
-    $new_xoops_admin_menu_dv = preg_replace('#' . preg_quote($search) . '#', $replace, $xoops_admin_menu_dv);
+    $new_xoops_admin_menu_dv = preg_replace('#'.preg_quote($search).'#', $replace, $xoops_admin_menu_dv);
 
     if ($xoops_admin_menu_dv == $new_xoops_admin_menu_dv) {
         return;
@@ -84,24 +84,24 @@ 
     if ('altsys' != $dirname) {
         $blocksadmin_title = defined('_MD_A_MYMENU_MYBLOCKSADMIN') ? _MD_A_MYMENU_MYBLOCKSADMIN : 'blocksadmin';
 
-        $insert .= '<img src=\'' . XOOPS_URL . '/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\'' . XOOPS_URL . '/modules/' . $dirname . '/admin/index.php?mode=admin&amp;lib=altsys&amp;page=myblocksadmin\'>' . $blocksadmin_title . '</a><br>' . "\n";
+        $insert .= '<img src=\''.XOOPS_URL.'/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\''.XOOPS_URL.'/modules/'.$dirname.'/admin/index.php?mode=admin&amp;lib=altsys&amp;page=myblocksadmin\'>'.$blocksadmin_title.'</a><br>'."\n";
     }
 
     // insert tplsadmin
 
     $db = XoopsDatabaseFactory::getDatabaseConnection();
 
-    [$count] = $db->fetchRow($db->query('SELECT COUNT(*) FROM ' . $db->prefix('tplfile') . " WHERE tpl_module='$dirname'"));
+    [$count] = $db->fetchRow($db->query('SELECT COUNT(*) FROM '.$db->prefix('tplfile')." WHERE tpl_module='$dirname'"));
 
     if ($count > 0) {
         $tplsadmin_title = defined('_MD_A_MYMENU_MYTPLSADMIN') ? _MD_A_MYMENU_MYTPLSADMIN : 'tplsadmin';
 
-        $insert = '<img src=\'' . XOOPS_URL . '/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\'' . XOOPS_URL . '/modules/' . $dirname . '/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mytplsadmin\'>' . $tplsadmin_title . '</a><br>' . "\n" . $insert;
+        $insert = '<img src=\''.XOOPS_URL.'/images/pointer.gif\' width=\'8\' height=\'8\' alt=\'\'>&nbsp;<a href=\''.XOOPS_URL.'/modules/'.$dirname.'/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mytplsadmin\'>'.$tplsadmin_title.'</a><br>'."\n".$insert;
     }
 
     // do insertion
 
-    $xoops_admin_menu_dv = preg_replace('#' . preg_quote($anchor) . '#', $anchor . $insert, $xoops_admin_menu_dv);
+    $xoops_admin_menu_dv = preg_replace('#'.preg_quote($anchor).'#', $anchor.$insert, $xoops_admin_menu_dv);
 
     // write back
 
@@ -111,7 +111,7 @@ 
                                   'xoops_admin_menu_sd' => $xoops_admin_menu_sd,
                                   'xoops_admin_menu_ft' => $xoops_admin_menu_ft,
                                   'xoops_admin_menu_dv' => $xoops_admin_menu_dv,
-                                  'altsys_adminmenu_ft_hacked' => (int) (@$altsys_adminmenu_ft_hacked),
+                                  'altsys_adminmenu_ft_hacked' => (int)(@$altsys_adminmenu_ft_hacked),
                                   'altsys_adminmenu_dv_updated' => true,
                               ]);
 }
@@ -145,7 +145,7 @@ 
     // read
 
     if (!is_file(ALTSYS_ADMINMENU_FILE)) {
-        redirect_header(XOOPS_URL . '/admin.php', 1, 'Rebuild adminmenu');
+        redirect_header(XOOPS_URL.'/admin.php', 1, 'Rebuild adminmenu');
 
         exit;
     }
@@ -165,7 +165,7 @@ 
 
         // rebuild adminmenu
 
-        require_once XOOPS_ROOT_PATH . '/include/cp_functions.php';
+        require_once XOOPS_ROOT_PATH.'/include/cp_functions.php';
 
         xoops_module_write_admin_menu(xoops_module_get_admin_menu());
 
@@ -197,11 +197,11 @@ 
     foreach ($mids as $mid) {
         $module = $moduleHandler->get($mid);
 
-        $new_menu_ft = preg_replace('#' . preg_quote($search) . '#', sprintf($replace_fmt, $module->getVar('name')), $xoops_admin_menu_ft[$mid]);
+        $new_menu_ft = preg_replace('#'.preg_quote($search).'#', sprintf($replace_fmt, $module->getVar('name')), $xoops_admin_menu_ft[$mid]);
 
         if ($is_left) {
             if ($mid == $last_mid) {
-                $xoops_admin_menu_ft[$mid] = $new_menu_ft . '</td><td>';
+                $xoops_admin_menu_ft[$mid] = $new_menu_ft.'</td><td>';
             } else {
                 $left_body = $new_menu_ft;
 
@@ -210,7 +210,7 @@ 
                 $xoops_admin_menu_ml[$mid] = str_replace(',105);', ',85);', $xoops_admin_menu_ml[$mid]);
             }
         } else {
-            $xoops_admin_menu_ft[$mid] = $left_body . '</td><td>' . $new_menu_ft;
+            $xoops_admin_menu_ft[$mid] = $left_body.'</td><td>'.$new_menu_ft;
 
             unset($xoops_admin_menu_ft[$left_key]);
 
@@ -237,7 +237,7 @@ 
     // read
 
     if (!is_file(ALTSYS_ADMINMENU_FILE)) {
-        redirect_header(XOOPS_URL . '/admin.php', 1, 'Rebuild adminmenu');
+        redirect_header(XOOPS_URL.'/admin.php', 1, 'Rebuild adminmenu');
 
         exit;
     }
@@ -257,7 +257,7 @@ 
 
         // rebuild adminmenu
 
-        require_once XOOPS_ROOT_PATH . '/include/cp_functions.php';
+        require_once XOOPS_ROOT_PATH.'/include/cp_functions.php';
 
         xoops_module_write_admin_menu(xoops_module_get_admin_menu());
 
@@ -281,9 +281,9 @@ 
     foreach ($mids as $mid) {
         $module = $moduleHandler->get($mid);
 
-        $xoops_admin_menu_ft[$mid] = preg_replace('/\<img src\=.*$/', $module->getVar('name') . '</a>', $xoops_admin_menu_ft[$mid]);
+        $xoops_admin_menu_ft[$mid] = preg_replace('/\<img src\=.*$/', $module->getVar('name').'</a>', $xoops_admin_menu_ft[$mid]);
 
-        $xoops_admin_menu_ft[$mid] = '<div style="text-align:' . _GLOBAL_LEFT . ';background-color:#CCC;" title="' . $module->getVar('dirname') . '">' . $xoops_admin_menu_ft[$mid] . '</div>';
+        $xoops_admin_menu_ft[$mid] = '<div style="text-align:'._GLOBAL_LEFT.';background-color:#CCC;" title="'.$module->getVar('dirname').'">'.$xoops_admin_menu_ft[$mid].'</div>';
 
         $xoops_admin_menu_ml[$mid] = str_replace(',105);', ',45);', $xoops_admin_menu_ml[$mid]);
     }
@@ -305,7 +305,7 @@ 
     // read
 
     if (!is_file(ALTSYS_ADMINMENU_FILE)) {
-        redirect_header(XOOPS_URL . '/admin.php', 1, 'Rebuild adminmenu');
+        redirect_header(XOOPS_URL.'/admin.php', 1, 'Rebuild adminmenu');
 
         exit;
     }
@@ -325,7 +325,7 @@ 
 
         // rebuild adminmenu
 
-        require_once XOOPS_ROOT_PATH . '/include/cp_functions.php';
+        require_once XOOPS_ROOT_PATH.'/include/cp_functions.php';
 
         $fp = fopen(ALTSYS_ADMINMENU_FILE, 'wb');
 
@@ -358,10 +358,10 @@ 
         if (preg_match('/popUpL\d+/', $xoops_admin_menu_ft[$mid], $regs)) {
             $popup = $regs[0];
 
-            preg_match_all('#\<a href.*' . $popup . '\(\).*\</a>#U', $xoops_admin_menu_dv, $regs);
+            preg_match_all('#\<a href.*'.$popup.'\(\).*\</a>#U', $xoops_admin_menu_dv, $regs);
 
             foreach ($regs[0] as $submenuitem) {
-                $submenuitems[] = str_replace($popup . '();', '', $submenuitem);
+                $submenuitems[] = str_replace($popup.'();', '', $submenuitem);
             }
         } else {
             return;
@@ -370,14 +370,14 @@ 
         // module icon
 
         if (preg_match('#\<img .*/\>#U', $xoops_admin_menu_ft[$mid], $regs)) {
-            $icon_img = str_replace("alt=''", 'alt="' . $module->getVar('name') . '"', $regs[0]);
+            $icon_img = str_replace("alt=''", 'alt="'.$module->getVar('name').'"', $regs[0]);
         } else {
             $icon_img = '';
         }
 
         // version number
 
-        $icon_img .= '<span class="version" style="">' . sprintf('%.2f', $module->getVar('version') / 100.0) . '</span>';
+        $icon_img .= '<span class="version" style="">'.sprintf('%.2f', $module->getVar('version') / 100.0).'</span>';
 
         $newline = preg_replace('/ onmouseover.*$/', '', $xoops_admin_menu_ft[$mid]);
 
@@ -403,10 +403,10 @@ 
                     . '" style="display:none;"><ul>';
 
         foreach ($submenuitems as $submenuitem) {
-            $newline .= '<li>' . $submenuitem . '</li>';
+            $newline .= '<li>'.$submenuitem.'</li>';
         }
 
-        $newline .= '</ul>' . $icon_img . '</div>';
+        $newline .= '</ul>'.$icon_img.'</div>';
 
         $xoops_admin_menu_ft[$mid] = $newline;
     }
@@ -451,7 +451,7 @@ 
     echo "<?php\n// modified by altsys\nif( ! defined('XOOPS_ROOT_PATH') ) exit ;\n";
 
     foreach ($xoops_admin_vars as $key => $val) {
-        echo '$' . $key . " = \n";
+        echo '$'.$key." = \n";
 
         @var_export($val);
 

xoops_trust_path/libs/altsys/index.php

Spacing +10 added lines, -10 removed lines

@@ -1,14 +1,14 @@ 
 <?php declare(strict_types=1);
 
 if (!defined('XOOPS_MODULE_PATH')) {
-    define('XOOPS_MODULE_PATH', XOOPS_ROOT_PATH . '/modules');
+    define('XOOPS_MODULE_PATH', XOOPS_ROOT_PATH.'/modules');
 }
 if (!defined('XOOPS_MODULE_URL')) {
-    define('XOOPS_MODULE_URL', XOOPS_URL . '/modules');
+    define('XOOPS_MODULE_URL', XOOPS_URL.'/modules');
 }
 
-require_once __DIR__ . '/class/AltsysBreadcrumbs.class.php';
-require_once __DIR__ . '/include/altsys_functions.php';
+require_once __DIR__.'/class/AltsysBreadcrumbs.class.php';
+require_once __DIR__.'/include/altsys_functions.php';
 
 if (empty($xoopsModule)) {
     $grouppermHandler = xoops_getHandler('module');
@@ -16,15 +16,15 @@ 
     $xoopsModule = $grouppermHandler->getByDirname('altsys');
 }
 
-require XOOPS_ROOT_PATH . '/include/cp_functions.php';
+require XOOPS_ROOT_PATH.'/include/cp_functions.php';
 
 // breadcrumbs
 $breadcrumbsObj = AltsysBreadcrumbs::getInstance();
-$breadcrumbsObj->appendPath(XOOPS_URL . '/modules/altsys/admin/index.php', $GLOBALS['xoopsModule']->getVar('name'));
+$breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php', $GLOBALS['xoopsModule']->getVar('name'));
 
 // get page
-$page = preg_replace('/[^a-zA-Z0-9_-]/', '', @$_GET['page']??'');
-require __DIR__ . '/controllers.php';
+$page = preg_replace('/[^a-zA-Z0-9_-]/', '', @$_GET['page'] ?? '');
+require __DIR__.'/controllers.php';
 if (!in_array($page, $controllers, true)) {
     $_GET['page'] = $page = 'myblocksadmin';
 
@@ -46,8 +46,8 @@ 
 
 // branch to each pages
 $mytrustdirpath = __DIR__;
-if (file_exists(XOOPS_TRUST_PATH . '/libs/altsys/' . $page . '.php')) {
-    include XOOPS_TRUST_PATH . '/libs/altsys/' . $page . '.php';
+if (file_exists(XOOPS_TRUST_PATH.'/libs/altsys/'.$page.'.php')) {
+    include XOOPS_TRUST_PATH.'/libs/altsys/'.$page.'.php';
 } else {
     die('wrong request');
 }

xoops_trust_path/libs/altsys/xoops_version.php

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php declare(strict_types=1);
 
-require_once __DIR__ . '/include/altsys_functions.php';
+require_once __DIR__.'/include/altsys_functions.php';
 
 // language file (modinfo.php)
 altsys_include_language_file('modinfo');
@@ -38,7 +38,7 @@ 
     'description' => '',
     'show_func'   => 'b_altsys_admin_menu_show',
     'edit_func'   => 'b_altsys_admin_menu_edit',
-    'options'     => (string) $mydirname,
+    'options'     => (string)$mydirname,
     'template'    => '', // use "module" template instead
 ];
 
@@ -112,5 +112,5 @@ 
 
 // keep block's options
 if (!defined('XOOPS_CUBE_LEGACY') && mb_substr(XOOPS_VERSION, 6, 3) < 2.1 && !empty($_POST['fct']) && !empty($_POST['op']) && 'modulesadmin' == $_POST['fct'] && 'update_ok' == $_POST['op'] && $_POST['dirname'] == $modversion['dirname']) {
-    require __DIR__ . '/include/x20_keepblockoptions.inc.php';
+    require __DIR__.'/include/x20_keepblockoptions.inc.php';
 }

xoops_trust_path/libs/altsys/get_templates.php

Spacing +7 added lines, -7 removed lines

@@ -7,8 +7,8 @@ 
 
 error_reporting(0);
 
-require_once __DIR__ . '/include/gtickets.php';
-require_once __DIR__ . '/include/altsys_functions.php';
+require_once __DIR__.'/include/gtickets.php';
+require_once __DIR__.'/include/altsys_functions.php';
 
 // this page can be called only from altsys
 if ('altsys' != $xoopsModule->getVar('dirname')) {
@@ -19,13 +19,13 @@ 
 altsys_include_language_file('compilehookadmin');
 
 if (!empty($_POST['download_zip'])) {
-    require_once XOOPS_ROOT_PATH . '/class/zipdownloader.php';
+    require_once XOOPS_ROOT_PATH.'/class/zipdownloader.php';
 
     $downloader = new \XoopsZipDownloader();
 
     $do_download = true;
 } elseif (!empty($_POST['download_tgz'])) {
-    require_once XOOPS_ROOT_PATH . '/class/tardownloader.php';
+    require_once XOOPS_ROOT_PATH.'/class/tardownloader.php';
 
     $downloader = new \XoopsTarDownloader();
 
@@ -51,13 +51,13 @@ 
         break;
     }
 }
-$trs = $xoopsDB->query('SELECT DISTINCT tpl_file,tpl_source,tpl_lastmodified FROM ' . $xoopsDB->prefix('tplfile') . ' NATURAL LEFT JOIN ' . $xoopsDB->prefix('tplsource') . " WHERE tpl_tplset='" . addslashes($tplset) . "' ORDER BY tpl_file");
+$trs = $xoopsDB->query('SELECT DISTINCT tpl_file,tpl_source,tpl_lastmodified FROM '.$xoopsDB->prefix('tplfile').' NATURAL LEFT JOIN '.$xoopsDB->prefix('tplsource')." WHERE tpl_tplset='".addslashes($tplset)."' ORDER BY tpl_file");
 if ($xoopsDB->getRowsNum($trs) <= 0) {
     die(_TPLSADMIN_ERR_INVALIDTPLSET);
 }
 
 while ([$tpl_file, $tpl_source, $tpl_lastmodified] = $xoopsDB->fetchRow($trs)) {
-    $downloader->addFileData($tpl_source, $tplset . '/' . $tpl_file, $tpl_lastmodified);
+    $downloader->addFileData($tpl_source, $tplset.'/'.$tpl_file, $tpl_lastmodified);
 }
 //bugfix by nao-pon ,echo is not necessary for downloader
-$downloader->download('template_' . $tplset, true);
+$downloader->download('template_'.$tplset, true);

xoops_trust_path/libs/altsys/mymenu.php

Indentation +2 added lines, -2 removed lines

@@ -65,8 +65,8 @@
 require_once XOOPS_TRUST_PATH . '/libs/altsys/class/D3Tpl.class.php';
 $tpl = new D3Tpl();
 $tpl->assign([
-                 'adminmenu' => $adminmenu,
-             ]);
+                    'adminmenu' => $adminmenu,
+                ]);
 $tpl->display('db:altsys_inc_mymenu.tpl');
 //$tpl->display('db:inc_mymenu.tpl');
 

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 // language files (modinfo.php)
 altsys_include_language_file('modinfo');
 
-require __DIR__ . '/admin_menu.php';
+require __DIR__.'/admin_menu.php';
 
 $adminmenu = array_merge($adminmenu, $adminmenu4altsys);
 
@@ -57,12 +57,12 @@ 
 // link conversion from relative to absolute
 foreach (array_keys($adminmenu) as $i) {
     if (false === mb_stristr($adminmenu[$i]['link'], XOOPS_URL)) {
-        $adminmenu[$i]['link'] = XOOPS_URL . "/modules/$mydirname/" . $adminmenu[$i]['link'];
+        $adminmenu[$i]['link'] = XOOPS_URL."/modules/$mydirname/".$adminmenu[$i]['link'];
     }
 }
 
 // display
-require_once XOOPS_TRUST_PATH . '/libs/altsys/class/D3Tpl.class.php';
+require_once XOOPS_TRUST_PATH.'/libs/altsys/class/D3Tpl.class.php';
 $tpl = new D3Tpl();
 $tpl->assign([
                  'adminmenu' => $adminmenu,
@@ -72,6 +72,6 @@ 
 
 // submenu
 $page = preg_replace('/[^0-9a-zA-Z_-]/', '', @$_GET['page']);
-if (file_exists(__DIR__ . '/mymenusub/' . $page . '.php')) {
-    require __DIR__ . '/mymenusub/' . $page . '.php';
+if (file_exists(__DIR__.'/mymenusub/'.$page.'.php')) {
+    require __DIR__.'/mymenusub/'.$page.'.php';
 }