mambax7 / altsys

xoops_trust_path/libs/altsys/class/MyBlocksAdminForXCL21.class.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php declare(strict_types=1);
 // $Id: MyBlocksAdminForXCL21.class.php ,ver 0.0.7.1 2011/01/27 16:10:00 domifara Exp $
 
-require_once __DIR__ . '/MyBlocksAdmin.class.php';
+require_once __DIR__.'/MyBlocksAdmin.class.php';
 
 /**
  * Class MyBlocksAdminForXCL21
@@ -48,7 +48,7 @@ 
             $langman->read('admin.php', $this->target_dirname);
         }
 
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
         //HACK by domifara
 
@@ -70,6 +70,6 @@ 
      */
     public function checkFck()
     {
-        return (!altsysUtils::isInstalledXclHtmleditor() && file_exists(XOOPS_ROOT_PATH . '/common/fckeditor/fckeditor.js'));
+        return (!altsysUtils::isInstalledXclHtmleditor() && file_exists(XOOPS_ROOT_PATH.'/common/fckeditor/fckeditor.js'));
     }
 }

xoops_trust_path/libs/altsys/class/MyBlocksAdmin.class.php

Indentation +28 added lines, -28 removed lines

@@ -288,11 +288,11 @@ 
         // get selected targets
 
         if (is_array(@$block_data['bmodule'])) {
-			// bmodule originated from request (preview etc.)
+            // bmodule originated from request (preview etc.)
 
             $selected_mids = $block_data['bmodule'];
         } else {
-			// originated from the table of `block_module_link`
+            // originated from the table of `block_module_link`
 
             $result = $this->db->query('SELECT module_id FROM ' . $this->db->prefix('block_module_link') . " WHERE block_id='$bid'");
 
@@ -549,12 +549,12 @@ 
             ];
 
             $blocks4assign[] = [
-                                   'name_raw' => $block_data['name'],
-                                   'title_raw' => $block_data['title'],
-                                   'cell_position' => $this->renderCell4BlockPosition($block_data),
-                                   'cell_module_link' => $this->renderCell4BlockModuleLink($block_data),
-                                   'cell_group_perm' => $this->renderCell4BlockReadGroupPerm($block_data),
-                               ] + $block_data;
+                                    'name_raw' => $block_data['name'],
+                                    'title_raw' => $block_data['title'],
+                                    'cell_position' => $this->renderCell4BlockPosition($block_data),
+                                    'cell_module_link' => $this->renderCell4BlockModuleLink($block_data),
+                                    'cell_group_perm' => $this->renderCell4BlockReadGroupPerm($block_data),
+                                ] + $block_data;
         }
 
         // display
@@ -564,14 +564,14 @@ 
         $tpl = new D3Tpl();
 
         $tpl->assign([
-                         'target_mid' => $this->target_mid,
-                         'target_dirname' => $this->target_dirname,
-                         'target_mname' => $this->target_mname,
-                         'language' => $this->lang,
-                         'cachetime_options' => $this->cachetime_options,
-                         'blocks' => $blocks4assign,
-                         'gticket_hidden' => $xoopsGTicket->getTicketHtml(__LINE__, 1800, 'myblocksadmin'),
-                     ]);
+                            'target_mid' => $this->target_mid,
+                            'target_dirname' => $this->target_dirname,
+                            'target_mname' => $this->target_mname,
+                            'language' => $this->lang,
+                            'cachetime_options' => $this->cachetime_options,
+                            'blocks' => $blocks4assign,
+                            'gticket_hidden' => $xoopsGTicket->getTicketHtml(__LINE__, 1800, 'myblocksadmin'),
+                        ]);
 
         $tpl->display('db:altsys_main_myblocksadmin_list.tpl');
     }
@@ -1250,18 +1250,18 @@ 
         }
 
         $tpl->assign([
-                         'target_dirname' => $this->target_dirname,
-                         'target_mname' => $this->target_mname,
-                         'language' => $this->lang,
-                         'cachetime_options' => $this->cachetime_options,
-                         'ctype_options' => $this->ctype_options,
-                         'block' => $block4assign,
-                         'op' => $next_op,
-                         'form_title' => $form_title,
-                         'submit_button' => $button_value,
-                         'common_fck_installed' => $this->checkFck(),
-                         'gticket_hidden' => $GLOBALS['xoopsSecurity']->getTokenHTML('myblocksadmin'),
-                     ]);
+                            'target_dirname' => $this->target_dirname,
+                            'target_mname' => $this->target_mname,
+                            'language' => $this->lang,
+                            'cachetime_options' => $this->cachetime_options,
+                            'ctype_options' => $this->ctype_options,
+                            'block' => $block4assign,
+                            'op' => $next_op,
+                            'form_title' => $form_title,
+                            'submit_button' => $button_value,
+                            'common_fck_installed' => $this->checkFck(),
+                            'gticket_hidden' => $GLOBALS['xoopsSecurity']->getTokenHTML('myblocksadmin'),
+                        ]);
 
         if (defined('XOOPS_CUBE_LEGACY')) {
             $tpl->display('db:altsys_main_myblocksadmin_edit_4legacy.tpl');

Spacing +77 added lines, -77 removed lines

@@ -126,7 +126,7 @@ 
 
                 $this->target_mid = $target_module->getVar('mid');
 
-                $this->target_mname = $target_module->getVar('name') . ' ' . sprintf('(%2.2f)', (int)$target_module->getVar('version') / 100.0);
+                $this->target_mname = $target_module->getVar('name').' '.sprintf('(%2.2f)', (int)$target_module->getVar('version') / 100.0);
 
                 $this->target_dirname = $target_module->getVar('dirname');
 
@@ -136,9 +136,9 @@ 
 
                 $breadcrumbsObj = AltsysBreadcrumbs::getInstance();
 
-                $breadcrumbsObj->appendPath(XOOPS_URL . '/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', '_MI_ALTSYS_MENU_MYBLOCKSADMIN');
+                $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', '_MI_ALTSYS_MENU_MYBLOCKSADMIN');
 
-                $breadcrumbsObj->appendPath(XOOPS_URL . '/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname=' . $this->target_dirname, $this->target_mname);
+                $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname='.$this->target_dirname, $this->target_mname);
             } else {
                 // custom blocks
 
@@ -152,28 +152,28 @@ 
 
                 $breadcrumbsObj = AltsysBreadcrumbs::getInstance();
 
-                $breadcrumbsObj->appendPath(XOOPS_URL . '/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', '_MI_ALTSYS_MENU_MYBLOCKSADMIN');
+                $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', '_MI_ALTSYS_MENU_MYBLOCKSADMIN');
 
-                $breadcrumbsObj->appendPath(XOOPS_URL . '/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname=' . $this->target_dirname, '_MI_ALTSYS_MENU_CUSTOMBLOCKS');
+                $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname='.$this->target_dirname, '_MI_ALTSYS_MENU_CUSTOMBLOCKS');
             }
         } else {
             // myblocksadmin as a library
 
             $this->target_mid = $xoopsModule->getVar('mid');
 
-            $this->target_mname = $xoopsModule->getVar('name') . ' ' . sprintf('(%2.2f)', $xoopsModule->getVar('version') / 100.0);
+            $this->target_mname = $xoopsModule->getVar('name').' '.sprintf('(%2.2f)', $xoopsModule->getVar('version') / 100.0);
 
             $this->target_dirname = $xoopsModule->getVar('dirname');
 
-            $mod_url = XOOPS_URL . '/modules/' . $xoopsModule->getVar('dirname');
+            $mod_url = XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname');
 
             $modinfo = $xoopsModule->getInfo();
 
             $breadcrumbsObj = AltsysBreadcrumbs::getInstance();
 
-            $breadcrumbsObj->appendPath($mod_url . '/' . @$modinfo['adminindex'], $this->target_mname);
+            $breadcrumbsObj->appendPath($mod_url.'/'.@$modinfo['adminindex'], $this->target_mname);
 
-            $breadcrumbsObj->appendPath($mod_url . '/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', _MD_A_MYBLOCKSADMIN_BLOCKADMIN);
+            $breadcrumbsObj->appendPath($mod_url.'/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin', _MD_A_MYBLOCKSADMIN_BLOCKADMIN);
         }
 
         // read xoops_version.php of the target
@@ -256,7 +256,7 @@ 
      */
     public function renderCell4BlockOptions($block_data)
     {
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
         //HACK by domifara
 
@@ -283,7 +283,7 @@ 
      */
     public function renderCell4BlockModuleLink($block_data)
     {
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
         // get selected targets
 
@@ -294,12 +294,12 @@ 
         } else {
 			// originated from the table of `block_module_link`
 
-            $result = $this->db->query('SELECT module_id FROM ' . $this->db->prefix('block_module_link') . " WHERE block_id='$bid'");
+            $result = $this->db->query('SELECT module_id FROM '.$this->db->prefix('block_module_link')." WHERE block_id='$bid'");
 
             $selected_mids = [];
 
             while ([$selected_mid] = $this->db->fetchRow($result)) {
-                $selected_mids[] = (int) $selected_mid;
+                $selected_mids[] = (int)$selected_mid;
             }
 
             if (empty($selected_mids)) {
@@ -351,7 +351,7 @@ 
      */
     public function renderCell4BlockReadGroupPerm($block_data)
     {
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
         // get selected targets
 
@@ -362,12 +362,12 @@ 
         } else {
             // origined from the table of `group_perm`
 
-            $result = $this->db->query('SELECT gperm_groupid FROM ' . $this->db->prefix('group_permission') . " WHERE gperm_itemid='$bid' AND gperm_name='block_read'");
+            $result = $this->db->query('SELECT gperm_groupid FROM '.$this->db->prefix('group_permission')." WHERE gperm_itemid='$bid' AND gperm_name='block_read'");
 
             $selected_gids = [];
 
             while ([$selected_gid] = $this->db->fetchRow($result)) {
-                $selected_gids[] = (int) $selected_gid;
+                $selected_gids[] = (int)$selected_gid;
             }
 
             if (0 == $bid && empty($selected_gids)) {
@@ -415,11 +415,11 @@ 
      */
     public function renderCell4BlockPosition($block_data)
     {
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
-        $side = (int) $block_data['side'];
+        $side = (int)$block_data['side'];
 
-        $visible = (int) $block_data['visible'];
+        $visible = (int)$block_data['visible'];
 
         $sseln = $ssel0 = $ssel1 = $ssel2 = $ssel3 = $ssel4 = '';
 
@@ -464,31 +464,31 @@ 
 
         return "
                 <div class='blockposition $scol0'>
-                    <input type='radio' name='sides[$bid]' value='" . XOOPS_SIDEBLOCK_LEFT . "' class='blockposition' $ssel0 onclick='document.getElementById(\"extra_side_$bid\").value=" . XOOPS_SIDEBLOCK_LEFT . ";'>
+                    <input type='radio' name='sides[$bid]' value='".XOOPS_SIDEBLOCK_LEFT."' class='blockposition' $ssel0 onclick='document.getElementById(\"extra_side_$bid\").value=".XOOPS_SIDEBLOCK_LEFT.";'>
                 </div>
-                <div style='float:" . _GLOBAL_LEFT . ";'>-</div>
+                <div style='float:" . _GLOBAL_LEFT.";'>-</div>
                 <div class='blockposition $scol2'>
-                    <input type='radio' name='sides[$bid]' value='" . XOOPS_CENTERBLOCK_LEFT . "' class='blockposition' $ssel2 onclick='document.getElementById(\"extra_side_$bid\").value=" . XOOPS_CENTERBLOCK_LEFT . ";'>
+                    <input type='radio' name='sides[$bid]' value='".XOOPS_CENTERBLOCK_LEFT."' class='blockposition' $ssel2 onclick='document.getElementById(\"extra_side_$bid\").value=".XOOPS_CENTERBLOCK_LEFT.";'>
                 </div>
                 <div class='blockposition $scol3'>
-                    <input type='radio' name='sides[$bid]' value='" . XOOPS_CENTERBLOCK_CENTER . "' class='blockposition' $ssel3 onclick='document.getElementById(\"extra_side_$bid\").value=" . XOOPS_CENTERBLOCK_CENTER . ";'>
+                    <input type='radio' name='sides[$bid]' value='".XOOPS_CENTERBLOCK_CENTER."' class='blockposition' $ssel3 onclick='document.getElementById(\"extra_side_$bid\").value=".XOOPS_CENTERBLOCK_CENTER.";'>
                 </div>
                 <div class='blockposition $scol4'>
-                    <input type='radio' name='sides[$bid]' value='" . XOOPS_CENTERBLOCK_RIGHT . "' class='blockposition' $ssel4 onclick='document.getElementById(\"extra_side_$bid\").value=" . XOOPS_CENTERBLOCK_RIGHT . ";'>
+                    <input type='radio' name='sides[$bid]' value='".XOOPS_CENTERBLOCK_RIGHT."' class='blockposition' $ssel4 onclick='document.getElementById(\"extra_side_$bid\").value=".XOOPS_CENTERBLOCK_RIGHT.";'>
                 </div>
-                <div style='float:" . _GLOBAL_LEFT . ";'>-</div>
+                <div style='float:" . _GLOBAL_LEFT.";'>-</div>
                 <div class='blockposition $scol1'>
-                    <input type='radio' name='sides[$bid]' value='" . XOOPS_SIDEBLOCK_RIGHT . "' class='blockposition' $ssel1 onclick='document.getElementById(\"extra_side_$bid\").value=" . XOOPS_SIDEBLOCK_RIGHT . ";'>
+                    <input type='radio' name='sides[$bid]' value='".XOOPS_SIDEBLOCK_RIGHT."' class='blockposition' $ssel1 onclick='document.getElementById(\"extra_side_$bid\").value=".XOOPS_SIDEBLOCK_RIGHT.";'>
                 </div>
                 <br>
                 <br>
-                <div style='float:" . _GLOBAL_LEFT . ";width:50px;' class='$stextbox'>
-                    <input type='text' name='extra_sides[$bid]' value='" . $value4extra_side . "' style='width:20px;' id='extra_side_$bid'>
+                <div style='float:" . _GLOBAL_LEFT.";width:50px;' class='$stextbox'>
+                    <input type='text' name='extra_sides[$bid]' value='".$value4extra_side."' style='width:20px;' id='extra_side_$bid'>
                 </div>
                 <div class='blockposition $scoln'>
                     <input type='radio' name='sides[$bid]' value='-1' class='blockposition' $sseln onclick='document.getElementById(\"extra_side_$bid\").value=-1;'>
                 </div>
-                <div style='float:" . _GLOBAL_LEFT . ";'>" . _NONE . '</div>
+                <div style='float:"._GLOBAL_LEFT.";'>"._NONE.'</div>
     ';
     }
 
@@ -500,7 +500,7 @@ 
 
         // main query
 
-        $sql = 'SELECT * FROM ' . $this->db->prefix('newblocks') . " WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";
+        $sql = 'SELECT * FROM '.$this->db->prefix('newblocks')." WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";
 
         $result = $this->db->query($sql);
 
@@ -536,13 +536,13 @@ 
 
         foreach ($block_arr as $i => $block) {
             $block_data = [
-                'bid' => (int) $block->getVar('bid'),
+                'bid' => (int)$block->getVar('bid'),
                 'name' => $block->getVar('name', 'n'),
                 'title' => $block->getVar('title', 'n'),
-                'weight' => (int) $block->getVar('weight'),
-                'bcachetime' => (int) $block->getVar('bcachetime'),
-                'side' => (int) $block->getVar('side'),
-                'visible' => (int) $block->getVar('visible'),
+                'weight' => (int)$block->getVar('weight'),
+                'bcachetime' => (int)$block->getVar('bcachetime'),
+                'side' => (int)$block->getVar('side'),
+                'visible' => (int)$block->getVar('visible'),
                 'can_edit' => $this->canEdit($block),
                 'can_delete' => $this->canDelete($block),
                 'can_clone' => $this->canClone($block),
@@ -559,7 +559,7 @@ 
 
         // 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();
 
@@ -585,7 +585,7 @@ 
             return [];
         }
 
-        require XOOPS_ROOT_PATH . '/modules/' . $this->target_dirname . '/xoops_version.php';
+        require XOOPS_ROOT_PATH.'/modules/'.$this->target_dirname.'/xoops_version.php';
 
         if (empty($modversion['blocks'])) {
             return [];
@@ -598,7 +598,7 @@ 
     {
         // query for getting blocks
 
-        $sql = 'SELECT * FROM ' . $this->db->prefix('newblocks') . " WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";
+        $sql = 'SELECT * FROM '.$this->db->prefix('newblocks')." WHERE mid='$this->target_mid' ORDER BY visible DESC,side,weight";
 
         $result = $this->db->query($sql);
 
@@ -635,9 +635,9 @@ 
         // skip system (TODO)
 
         if ($this->target_mid > 1) {
-            $form->addAppendix('module_admin', $this->target_mid, $this->target_mname . ' ' . _MD_A_MYBLOCKSADMIN_PERM_MADMIN);
+            $form->addAppendix('module_admin', $this->target_mid, $this->target_mname.' '._MD_A_MYBLOCKSADMIN_PERM_MADMIN);
 
-            $form->addAppendix('module_read', $this->target_mid, $this->target_mname . ' ' . _MD_A_MYBLOCKSADMIN_PERM_MREAD);
+            $form->addAppendix('module_read', $this->target_mid, $this->target_mname.' '._MD_A_MYBLOCKSADMIN_PERM_MREAD);
         }
 
         foreach ($item_list as $item_id => $item_name) {
@@ -708,25 +708,25 @@ 
         $msg = _MD_A_MYBLOCKSADMIN_DBUPDATED;
 
         if (false !== $block->store()) {
-            require_once XOOPS_ROOT_PATH . '/class/template.php';
+            require_once XOOPS_ROOT_PATH.'/class/template.php';
 
             $xoopsTpl = new \XoopsTpl();
 
             $xoopsTpl->xoops_setCaching(2);
 
             if ('' != $block->getVar('template')) {
-                if ($xoopsTpl->isCached('db:' . $block->getVar('template'))) {
-                    if (!$xoopsTpl->clearCache('db:' . $block->getVar('template'))) {
-                        $msg = 'Unable to clear cache for block ID' . $bid;
+                if ($xoopsTpl->isCached('db:'.$block->getVar('template'))) {
+                    if (!$xoopsTpl->clearCache('db:'.$block->getVar('template'))) {
+                        $msg = 'Unable to clear cache for block ID'.$bid;
                     }
                 }
-            } elseif ($xoopsTpl->isCached('db:system_dummy.tpl', 'blk_' . $bid)) {
-                if (!$xoopsTpl->clearCache('db:system_dummy.tpl', 'blk_' . $bid)) {
-                    $msg = 'Unable to clear cache for block ID' . $bid;
+            } elseif ($xoopsTpl->isCached('db:system_dummy.tpl', 'blk_'.$bid)) {
+                if (!$xoopsTpl->clearCache('db:system_dummy.tpl', 'blk_'.$bid)) {
+                    $msg = 'Unable to clear cache for block ID'.$bid;
                 }
             }
         } else {
-            $msg = 'Failed update of block. ID:' . $bid;
+            $msg = 'Failed update of block. ID:'.$bid;
         }
 
         return $msg;
@@ -740,7 +740,7 @@ 
      */
     public function updateBlockModuleLink($bid, $bmodules): void
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         $table = $this->db->prefix('block_module_link');
 
@@ -749,7 +749,7 @@ 
         $this->db->query($sql);
 
         foreach ($bmodules as $mid) {
-            $mid = (int) $mid;
+            $mid = (int)$mid;
 
             $sql = "INSERT INTO `$table` (`block_id`,`module_id`) VALUES ($bid,$mid)";
 
@@ -765,7 +765,7 @@ 
      */
     public function updateBlockReadGroupPerm($bid, $req_gids): void
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         $table = $this->db->prefix('group_permission');
 
@@ -800,7 +800,7 @@ 
         $this->db->query($sql);
 
         foreach ($req_gids as $gid) {
-            $gid = (int) $gid;
+            $gid = (int)$gid;
 
             $sql = "INSERT INTO `$table` (`gperm_groupid`,`gperm_itemid`,`gperm_modid`,`gperm_name`) VALUES ($gid,$bid,1,'block_read')";
 
@@ -840,7 +840,7 @@ 
      */
     public function fetchRequest4Block($bid)
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         (method_exists('MyTextSanitizer', 'sGetInstance') and $myts = MyTextSanitizer::sGetInstance()) || $myts = MyTextSanitizer::getInstance();
 
@@ -858,13 +858,13 @@ 
 
         return [
             'bid' => $bid,
-            'side' => (int) (@$_POST['sides'][$bid]),
-            'weight' => (int) (@$_POST['weights'][$bid]),
+            'side' => (int)(@$_POST['sides'][$bid]),
+            'weight' => (int)(@$_POST['weights'][$bid]),
             'visible' => $visible,
             'title' => (@$_POST['titles'][$bid]),
             'content' => (@$_POST['contents'][$bid]),
             'ctype' => preg_replace('/[^A-Z]/', '', @$_POST['ctypes'][$bid]),
-            'bcachetime' => (int) (@$_POST['bcachetimes'][$bid]),
+            'bcachetime' => (int)(@$_POST['bcachetimes'][$bid]),
             'bmodule' => \is_array(@$_POST['bmodules'][$bid]) ? $_POST['bmodules'][$bid] : [0],
             'bgroup' => \is_array(@$_POST['bgroups'][$bid]) ? $_POST['bgroups'][$bid] : [],
             'options' => \is_array(@$_POST['options'][$bid]) ? $_POST['options'][$bid] : [],
@@ -877,7 +877,7 @@ 
      */
     public function do_delete($bid)
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         //HACK by domifara
 
@@ -913,7 +913,7 @@ 
      */
     public function do_deleteBlockReadGroupPerm($bid): void
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         $table = $this->db->prefix('group_permission');
 
@@ -927,7 +927,7 @@ 
      */
     public function form_delete($bid): void
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         //HACK by domifara
 
@@ -966,7 +966,7 @@ 
      */
     public function do_clone($bid)
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         $request = $this->fetchRequest4Block($bid);
 
@@ -1056,7 +1056,7 @@ 
         do {
             $func_num--;
 
-            [$count] = $this->db->fetchRow($this->db->query('SELECT COUNT(*) FROM ' . $this->db->prefix('newblocks') . ' WHERE mid=' . (int) $mid . ' AND func_num=' . $func_num));
+            [$count] = $this->db->fetchRow($this->db->query('SELECT COUNT(*) FROM '.$this->db->prefix('newblocks').' WHERE mid='.(int)$mid.' AND func_num='.$func_num));
         } while ($count > 0);
 
         return $func_num > 128 ? $func_num : 255;
@@ -1068,7 +1068,7 @@ 
      */
     public function do_edit($bid)
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         if ($bid <= 0) {
             // new custom block
@@ -1098,7 +1098,7 @@ 
             // permission copy
 
             foreach ($GLOBALS['xoopsUser']->getGroups() as $gid) {
-                $sql = 'INSERT INTO ' . $this->db->prefix('group_permission') . " (gperm_groupid, gperm_itemid, gperm_modid, gperm_name) VALUES ($gid, $bid, 1, 'block_read')";
+                $sql = 'INSERT INTO '.$this->db->prefix('group_permission')." (gperm_groupid, gperm_itemid, gperm_modid, gperm_name) VALUES ($gid, $bid, 1, 'block_read')";
 
                 $this->db->query($sql);
             }
@@ -1127,7 +1127,7 @@ 
      */
     public function form_edit($bid, $mode = 'edit'): void
     {
-        $bid = (int) $bid;
+        $bid = (int)$bid;
 
         //HACK by domifara
 
@@ -1209,10 +1209,10 @@ 
                 'bid' => $bid,
                 'name' => $block->getVar('name', 'n'),
                 'title' => $block->getVar('title', 'n'),
-                'weight' => (int) $block->getVar('weight'),
-                'bcachetime' => (int) $block->getVar('bcachetime'),
-                'side' => (int) $block->getVar('side'),
-                'visible' => (int) $block->getVar('visible'),
+                'weight' => (int)$block->getVar('weight'),
+                'bcachetime' => (int)$block->getVar('bcachetime'),
+                'side' => (int)$block->getVar('side'),
+                'visible' => (int)$block->getVar('visible'),
                 'template' => $block_template,
                 'template_tplset' => $block_template_tplset,
                 'options' => $block->getVar('options'),
@@ -1235,7 +1235,7 @@ 
 
         // 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();
 
@@ -1244,7 +1244,7 @@ 
         if (defined('XOOPS_CUBE_LEGACY')) {
             $tpl->assign('xoops_cube_legacy', true);
 
-            require_once XOOPS_ROOT_PATH . '/class/xoopsformloader.php';
+            require_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
         } else {
             $tpl->assign('xoops_cube_legacy', false);
         }
@@ -1275,7 +1275,7 @@ 
      */
     public function checkFck()
     {
-        return file_exists(XOOPS_ROOT_PATH . '/common/fckeditor/fckeditor.js');
+        return file_exists(XOOPS_ROOT_PATH.'/common/fckeditor/fckeditor.js');
     }
 
     /**
@@ -1284,7 +1284,7 @@ 
      */
     public function previewContent($block_data)
     {
-        $bid = (int) $block_data['bid'];
+        $bid = (int)$block_data['bid'];
 
         if (!$block_data['is_custom']) {
             return '';
@@ -1346,12 +1346,12 @@ 
         // Ticket Check
 
         if (!$GLOBALS['xoopsSecurity']->check(true, 'myblocksadmin')) {
-            redirect_header(XOOPS_URL . '/', 3, $GLOBALS['xoopsSecurity']->getErrors());
+            redirect_header(XOOPS_URL.'/', 3, $GLOBALS['xoopsSecurity']->getErrors());
         }
 
         $msg = '';
 
-        $bid = (int) (@$_GET['bid']);
+        $bid = (int)(@$_GET['bid']);
 
         if (!empty($_POST['preview'])) {
             // preview
@@ -1380,19 +1380,19 @@ 
         } elseif (!empty($_POST['submit'])) {
             // update module_admin,module_read,block_read
 
-            include dirname(__DIR__) . '/include/mygroupperm.php';
+            include dirname(__DIR__).'/include/mygroupperm.php';
 
             $msg = _MD_A_MYBLOCKSADMIN_PERMUPDATED;
         }
 
-        redirect_header('?mode=admin&lib=altsys&page=myblocksadmin&dirname=' . $this->target_dirname, 1, $msg);
+        redirect_header('?mode=admin&lib=altsys&page=myblocksadmin&dirname='.$this->target_dirname, 1, $msg);
 
         exit;
     }
 
     public function processGet(): void
     {
-        $bid = (int) (@$_GET['bid']);
+        $bid = (int)(@$_GET['bid']);
 
         switch (@$_GET['op']) {
             case 'clone':

xoops_trust_path/libs/altsys/class/SetupAltsysLangMgr.class.php

Spacing +10 added lines, -10 removed lines

@@ -5,12 +5,12 @@ 
 }
 
 if (!defined('XOOPS_TRUST_PATH') || XOOPS_TRUST_PATH == '') {
-    header('Location: ' . XOOPS_URL . '/modules/altsys/setup_xoops_trust_path.php');
+    header('Location: '.XOOPS_URL.'/modules/altsys/setup_xoops_trust_path.php');
 
     exit;
 }
 
-define('ALTSYS_MYLANGUAGE_ROOT_PATH', XOOPS_ROOT_PATH . '/my_language');
+define('ALTSYS_MYLANGUAGE_ROOT_PATH', XOOPS_ROOT_PATH.'/my_language');
 
 /**
  * Class SetupAltsysLangMgr
@@ -32,8 +32,8 @@ 
     }
 }
 
-require_once XOOPS_ROOT_PATH . '/core/XCube_LanguageManager.class.php';
-require_once XOOPS_ROOT_PATH . '/modules/legacy/kernel/Legacy_LanguageManager.class.php';
+require_once XOOPS_ROOT_PATH.'/core/XCube_LanguageManager.class.php';
+require_once XOOPS_ROOT_PATH.'/modules/legacy/kernel/Legacy_LanguageManager.class.php';
 
 /**
  * Class AltsysLangMgr_LanguageManager
@@ -46,7 +46,7 @@ 
 
     public function prepare(): void
     {
-        $langmanpath = XOOPS_TRUST_PATH . '/libs/altsys/class/D3LanguageManager.class.php';
+        $langmanpath = XOOPS_TRUST_PATH.'/libs/altsys/class/D3LanguageManager.class.php';
 
         if (!is_file($langmanpath)) {
             die('install the latest altsys');
@@ -73,12 +73,12 @@ 
             $root = XCube_Root::getSingleton();
 
             if (!empty($root->mContext->mXoopsConfig['theme_set'])) {
-                $langdir = XOOPS_THEME_PATH . '/' . $root->mContext->mXoopsConfig['theme_set'] . '/language';
+                $langdir = XOOPS_THEME_PATH.'/'.$root->mContext->mXoopsConfig['theme_set'].'/language';
 
                 if (file_exists($langdir)) {
-                    $langfile = $langdir . '/' . $this->mLanguageName . '.php';
+                    $langfile = $langdir.'/'.$this->mLanguageName.'.php';
 
-                    $engfile = $langdir . '/english.php';
+                    $engfile = $langdir.'/english.php';
 
                     if (is_file($langfile)) {
                         require_once $langfile;
@@ -93,7 +93,7 @@ 
 
         // read normal
 
-        $this->langman->read($fileBodyName . '.php', $dirname);
+        $this->langman->read($fileBodyName.'.php', $dirname);
     }
 
     /**
@@ -104,7 +104,7 @@ 
         // I dare not to use langman...
 
         if (false === mb_strpos($type, '.') && $this->langman->my_language) {
-            $mylang_file = $this->langman->my_language . '/' . $this->mLanguageName . '/' . $type . '.php';
+            $mylang_file = $this->langman->my_language.'/'.$this->mLanguageName.'/'.$type.'.php';
 
             if (is_file($mylang_file)) {
                 require_once $mylang_file;

xoops_trust_path/libs/altsys/class/D3Tpl.class.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php declare(strict_types=1);
 
-require_once XOOPS_ROOT_PATH . '/class/template.php';
-require_once XOOPS_TRUST_PATH . '/libs/altsys/include/altsys_functions.php';
+require_once XOOPS_ROOT_PATH.'/class/template.php';
+require_once XOOPS_TRUST_PATH.'/libs/altsys/include/altsys_functions.php';
 
 /**
  * Class D3Tpl
@@ -20,7 +20,7 @@ 
         parent::__construct();
 
         if (in_array(altsys_get_core_type(), [ALTSYS_CORE_TYPE_X20S, ALTSYS_CORE_TYPE_X23P], true)) {
-            array_unshift($this->plugins_dir, XOOPS_TRUST_PATH . '/libs/altsys/smarty_plugins');
+            array_unshift($this->plugins_dir, XOOPS_TRUST_PATH.'/libs/altsys/smarty_plugins');
         }
 
         // for RTL users

xoops_trust_path/libs/altsys/class/altsysUtils.class.php

Indentation +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
         }
 
         if ($phpVersion > 7.4) {
-             return htmlspecialchars((string) $str, $flags, $encoding, $double_encode);
+                return htmlspecialchars((string) $str, $flags, $encoding, $double_encode);
         }
 
         $ret = htmlspecialchars((string) $str, $flags, $encoding);

Spacing +2 added lines, -2 removed lines

@@ -89,10 +89,10 @@
         }
 
         if ($phpVersion > 7.4) {
-             return htmlspecialchars((string) $str, $flags, $encoding, $double_encode);
+             return htmlspecialchars((string)$str, $flags, $encoding, $double_encode);
         }
 
-        $ret = htmlspecialchars((string) $str, $flags, $encoding);
+        $ret = htmlspecialchars((string)$str, $flags, $encoding);
 
         if (!$double_encode) {
             $ret = str_replace('&', '&', $ret);

xoops_trust_path/libs/altsys/class/D3LanguageManager.class.php

Spacing +11 added lines, -11 removed lines

@@ -29,15 +29,15 @@ 
     {
         $this->language = preg_replace('/[^0-9a-zA-Z_-]/', '', @$GLOBALS['xoopsConfig']['language']);
 
-        $this->salt = mb_substr(md5(XOOPS_ROOT_PATH . XOOPS_DB_USER . XOOPS_DB_PREFIX), 0, 6);
+        $this->salt = mb_substr(md5(XOOPS_ROOT_PATH.XOOPS_DB_USER.XOOPS_DB_PREFIX), 0, 6);
 
 //        $this->cache_path = XOOPS_TRUST_PATH . '/cache';
 
-        if (\defined('XOOPS_TRUST_PATH') and \file_exists(XOOPS_TRUST_PATH . '/cache')) {
-            $this->cache_path = XOOPS_TRUST_PATH . '/cache';
+        if (\defined('XOOPS_TRUST_PATH') and \file_exists(XOOPS_TRUST_PATH.'/cache')) {
+            $this->cache_path = XOOPS_TRUST_PATH.'/cache';
         } else {
 //            $this->cache_path = XOOPS_ROOT_PATH . '/cache';
-            $this->cache_path = XOOPS_VAR_PATH  . '/caches/xoops_cache';
+            $this->cache_path = XOOPS_VAR_PATH.'/caches/xoops_cache';
         }
 
 
@@ -73,7 +73,7 @@ 
      */
     public function read($resource, $mydirname, $mytrustdirname = null, bool $read_once = true): void
     {
-        $d3file = XOOPS_ROOT_PATH . '/modules/' . $mydirname . '/mytrustdirname.php';
+        $d3file = XOOPS_ROOT_PATH.'/modules/'.$mydirname.'/mytrustdirname.php';
 
         if (empty($mytrustdirname) && is_file($d3file)) {
             require $d3file;
@@ -85,12 +85,12 @@ 
 
         $cache_file = $this->getCacheFileName($resource, $mydirname);
 
-        $root_file = XOOPS_ROOT_PATH . '/modules/' . $mydirname . '/language/' . $this->language . '/' . $resource;
+        $root_file = XOOPS_ROOT_PATH.'/modules/'.$mydirname.'/language/'.$this->language.'/'.$resource;
 
         // language overriding by XOOPS_ROOT_PATH/my_language
 
         if ($this->my_language) {
-            $mylang_file = $this->my_language . '/modules/' . $mydirname . '/' . $this->language . '/' . $resource;
+            $mylang_file = $this->my_language.'/modules/'.$mydirname.'/'.$this->language.'/'.$resource;
 
             if (is_file($mylang_file)) {
                 require_once $mylang_file;
@@ -104,7 +104,7 @@ 
         if (empty($mytrustdirname)) {
             // conventional module
 
-            $default_file = XOOPS_ROOT_PATH . '/modules/' . $mydirname . '/language/' . $this->default_language . '/' . $resource;
+            $default_file = XOOPS_ROOT_PATH.'/modules/'.$mydirname.'/language/'.$this->default_language.'/'.$resource;
 
             if (is_file($cache_file)) {
                 require_once $cache_file;
@@ -118,9 +118,9 @@ 
         } else {
             // D3 modules
 
-            $trust_file = XOOPS_TRUST_PATH . '/modules/' . $mytrustdirname . '/language/' . $this->language . '/' . $resource;
+            $trust_file = XOOPS_TRUST_PATH.'/modules/'.$mytrustdirname.'/language/'.$this->language.'/'.$resource;
 
-            $default_file = XOOPS_TRUST_PATH . '/modules/' . $mytrustdirname . '/language/' . $this->default_language . '/' . $resource;
+            $default_file = XOOPS_TRUST_PATH.'/modules/'.$mytrustdirname.'/language/'.$this->default_language.'/'.$resource;
 
             if (is_file($cache_file)) {
                 require_once $cache_file;
@@ -160,6 +160,6 @@ 
             $language = $this->language;
         }
 
-        return $this->cache_path . '/' . $this->cache_prefix . '_' . $this->salt . '_' . $mydirname . '_' . $language . '_' . $resource;
+        return $this->cache_path.'/'.$this->cache_prefix.'_'.$this->salt.'_'.$mydirname.'_'.$language.'_'.$resource;
     }
 }

xoops_trust_path/libs/altsys/class/D3NotificationHandler.class.php

Spacing +8 added lines, -8 removed lines

@@ -2,7 +2,7 @@ 
 
 // notification handler for D3 modules
 
-require_once XOOPS_ROOT_PATH . '/include/notification_functions.php';
+require_once XOOPS_ROOT_PATH.'/include/notification_functions.php';
 
 /**
  * Class D3NotificationHandler
@@ -37,9 +37,9 @@ 
     {
         global $xoopsConfig;
 
-        $mydirpath = XOOPS_ROOT_PATH . '/modules/' . $mydirname;
+        $mydirpath = XOOPS_ROOT_PATH.'/modules/'.$mydirname;
 
-        $mytrustdirpath = XOOPS_TRUST_PATH . '/modules/' . $mytrustdirname;
+        $mytrustdirpath = XOOPS_TRUST_PATH.'/modules/'.$mytrustdirname;
 
         $language = empty($xoopsConfig['language']) ? 'english' : $xoopsConfig['language'];
 
@@ -142,11 +142,11 @@ 
 
         $criteria = new \CriteriaCompo();
 
-        $criteria->add(new \Criteria('not_modid', (int) $mid));
+        $criteria->add(new \Criteria('not_modid', (int)$mid));
 
         $criteria->add(new \Criteria('not_category', $category));
 
-        $criteria->add(new \Criteria('not_itemid', (int) $item_id));
+        $criteria->add(new \Criteria('not_itemid', (int)$item_id));
 
         $criteria->add(new \Criteria('not_event', $event));
 
@@ -174,13 +174,13 @@ 
 
         $tags['X_MODULE'] = $module->getVar('name', 'n');
 
-        $tags['X_MODULE_URL'] = XOOPS_URL . '/modules/' . $module->getVar('dirname') . '/';
+        $tags['X_MODULE_URL'] = XOOPS_URL.'/modules/'.$module->getVar('dirname').'/';
 
         $tags['X_NOTIFY_CATEGORY'] = $category;
 
         $tags['X_NOTIFY_EVENT'] = $event;
 
-        $template = $event_info['mail_template'] . '.tpl';
+        $template = $event_info['mail_template'].'.tpl';
 
         $subject = $event_info['mail_subject'];
 
@@ -199,7 +199,7 @@ 
 
                 // TODO: don't show unsubscribe link if it is 'one-time' ??
 
-                $tags['X_UNSUBSCRIBE_URL'] = XOOPS_URL . '/notifications.php';
+                $tags['X_UNSUBSCRIBE_URL'] = XOOPS_URL.'/notifications.php';
 
                 $tags = array_merge($tags, $extra_tags);
 

xoops_trust_path/libs/altsys/class/d3utilities.php

Spacing +25 added lines, -25 removed lines

@@ -36,7 +36,7 @@ 
 
         $this->mydirname = $mydirname;
 
-        $this->table = $db->prefix($mydirname ? $mydirname . '_' . $table_body : $table_body);
+        $this->table = $db->prefix($mydirname ? $mydirname.'_'.$table_body : $table_body);
 
         $this->primary_key = $primary_key;
 
@@ -47,7 +47,7 @@ 
         $module = $moduleHandler->getByDirname($this->mydirname);
 
         if (!empty($module)) {
-            $this->mid = (int) $module->getVar('mid');
+            $this->mid = (int)$module->getVar('mid');
         }
 
         $this->page_name = $page_name;
@@ -61,7 +61,7 @@ 
      */
     public function get_language_constant($name)
     {
-        return constant(mb_strtoupper('_MD_A_' . $this->dirname . '_' . $this->page_name . '_' . $name));
+        return constant(mb_strtoupper('_MD_A_'.$this->dirname.'_'.$this->page_name.'_'.$name));
     }
 
     /**
@@ -74,23 +74,23 @@ 
         switch ($col['type']) {
             case 'text':
             case 'blob':
-                $length = empty($col['length']) ? 65535 : (int) $col['length'];
+                $length = empty($col['length']) ? 65535 : (int)$col['length'];
 
-                return "`{$col['name']}`='" . addslashes(xoops_substr($value, 0, $length)) . "'";
+                return "`{$col['name']}`='".addslashes(xoops_substr($value, 0, $length))."'";
             case 'char':
             case 'varchar':
             case 'string':
-                $length = empty($col['length']) ? 255 : (int) $col['length'];
+                $length = empty($col['length']) ? 255 : (int)$col['length'];
 
-                return "`{$col['name']}`='" . addslashes(xoops_substr($value, 0, $length)) . "'";
+                return "`{$col['name']}`='".addslashes(xoops_substr($value, 0, $length))."'";
             case 'int':
             case 'integer':
-                $value = (int) $value;
+                $value = (int)$value;
                 if (!empty($col['max'])) {
-                    $value = min($value, (int) $col['max']);
+                    $value = min($value, (int)$col['max']);
                 }
                 if (!empty($col['min'])) {
-                    $value = max($value, (int) $col['min']);
+                    $value = max($value, (int)$col['min']);
                 }
 
                 return "`{$col['name']}`=$value";
@@ -119,21 +119,21 @@ 
                 continue;
             }
 
-            $set4sql .= $this->get_set4sql(@$_POST[$col['name']], $col) . ',';
+            $set4sql .= $this->get_set4sql(@$_POST[$col['name']], $col).',';
         }
 
         if (!empty($set4sql)) {
             if ($id > 0) {
                 // UPDATE
 
-                $db->queryF("UPDATE $this->table SET " . mb_substr($set4sql, 0, -1) . " WHERE $this->primary_key='" . addslashes($id) . "'");
+                $db->queryF("UPDATE $this->table SET ".mb_substr($set4sql, 0, -1)." WHERE $this->primary_key='".addslashes($id)."'");
 
                 return [$id, 'update'];
             }
 
             // INSERT
 
-            $db->queryF("INSERT INTO $this->table SET " . mb_substr($set4sql, 0, -1));
+            $db->queryF("INSERT INTO $this->table SET ".mb_substr($set4sql, 0, -1));
 
             return [$db->getInsertId(), 'insert'];
         }
@@ -165,7 +165,7 @@ 
         $ret = [];
 
         foreach (array_keys($_POST[$column4key]) as $id) {
-            $id = (int) $id;    // primary_key should be 'integer'
+            $id = (int)$id; // primary_key should be 'integer'
 
             $set4sql = '';
 
@@ -178,14 +178,14 @@ 
                     continue;
                 }
 
-                $set4sql .= $this->get_set4sql(@$_POST[$col['name']][$id], $col) . ',';
+                $set4sql .= $this->get_set4sql(@$_POST[$col['name']][$id], $col).',';
             }
 
             if (!empty($set4sql)) {
                 $result = $db->query("SELECT * FROM $this->table WHERE $this->primary_key=$id");
 
                 if (1 == $db->getRowsNum($result)) {
-                    $db->queryF("UPDATE $this->table SET " . mb_substr($set4sql, 0, -1) . " WHERE $this->primary_key=$id");
+                    $db->queryF("UPDATE $this->table SET ".mb_substr($set4sql, 0, -1)." WHERE $this->primary_key=$id");
 
                     if (1 == $db->getAffectedRows()) {
                         $ret[$id] = $db->fetchArray($result);
@@ -209,7 +209,7 @@ 
         $ret = [];
 
         foreach (array_keys($_POST['admin_main_checkboxes']) as $id) {
-            $id = (int) $id;    // primary_key should be 'integer'
+            $id = (int)$id; // primary_key should be 'integer'
 
             $result = $db->query("SELECT * FROM $this->table WHERE $this->primary_key=$id");
 
@@ -221,13 +221,13 @@ 
                 if ($delete_comments) {
                     // remove comments
 
-                    $db->queryF('DELETE FROM ' . $db->prefix('xoopscomments') . " WHERE com_modid=$this->mid AND com_itemid=$id");
+                    $db->queryF('DELETE FROM '.$db->prefix('xoopscomments')." WHERE com_modid=$this->mid AND com_itemid=$id");
                 }
 
                 if ($delete_notifications) {
                     // remove notifications
 
-                    $db->queryF('DELETE FROM ' . $db->prefix('xoopsnotifications') . " WHERE not_modid=$this->mid AND not_itemid=$id");
+                    $db->queryF('DELETE FROM '.$db->prefix('xoopsnotifications')." WHERE not_modid=$this->mid AND not_itemid=$id");
                 }
             }
         }
@@ -304,9 +304,9 @@ 
                     break;
                 case 'text':
                 default:
-                    $size = empty($col['edit_size']) ? 32 : (int) $col['edit_size'];
-                    $length = empty($col['length']) ? 255 : (int) $col['length'];
-                    $lines[$col['name']] = "<input type='text' name='{$col['name']}' size='$size' maxlength='$length' value='" . htmlspecialchars($col['default_value'], ENT_QUOTES | ENT_HTML5) . "'>";
+                    $size = empty($col['edit_size']) ? 32 : (int)$col['edit_size'];
+                    $length = empty($col['length']) ? 255 : (int)$col['length'];
+                    $lines[$col['name']] = "<input type='text' name='{$col['name']}' size='$size' maxlength='$length' value='".htmlspecialchars($col['default_value'], ENT_QUOTES | ENT_HTML5)."'>";
                     break;
                 case false:
                     $lines[$col['name']] = htmlspecialchars($col['default_value'], ENT_QUOTES | ENT_HTML5);
@@ -345,7 +345,7 @@ 
             <form action='' method='get' name='admin_control' id='admin_control'>
                 $hiddens
                 $controllers_html
-                <input type='submit' value='" . _SUBMIT . "'>
+                <input type='submit' value='"._SUBMIT."'>
             </form>\n";
     }
 
@@ -357,12 +357,12 @@ 
      */
     public function get_select($name, $options, $current_value)
     {
-        $ret = "<select name='" . htmlspecialchars($name, ENT_QUOTES | ENT_HTML5) . "'>\n";
+        $ret = "<select name='".htmlspecialchars($name, ENT_QUOTES | ENT_HTML5)."'>\n";
 
         foreach ($options as $key => $val) {
             $selected = $val == $current_value ? "selected='selected'" : '';
 
-            $ret .= "<option value='" . htmlspecialchars($key, ENT_QUOTES | ENT_HTML5) . "' $selected>" . htmlspecialchars($val, ENT_QUOTES | ENT_HTML5) . "</option>\n";
+            $ret .= "<option value='".htmlspecialchars($key, ENT_QUOTES | ENT_HTML5)."' $selected>".htmlspecialchars($val, ENT_QUOTES | ENT_HTML5)."</option>\n";
         }
 
         $ret .= "</select>\n";

xoops_trust_path/libs/altsys/get_tplsvarsinfo.php

Indentation +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 
 // this page can be called only from altsys
 if ('altsys' != $xoopsModule->getVar('dirname')) {
-   exit('this page can be called only from altsys');
+    exit('this page can be called only from altsys');
 }
 
 // language file
@@ -162,11 +162,11 @@ 
         $tplsvarsinfo_total = array_merge($tplsvarsinfo_total, $tplsvarsinfo);
     }
 } else {
-   exit('XOOPS_COMPILE_PATH cannot be opened');
+    exit('XOOPS_COMPILE_PATH cannot be opened');
 }
 
 if (empty($tplsvarsinfo_total)) {
-   exit(_TPLSADMIN_ERR_NOTPLSVARSINFO);
+    exit(_TPLSADMIN_ERR_NOTPLSVARSINFO);
 }
 
 //

Spacing +16 added lines, -16 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')) {
@@ -44,7 +44,7 @@ 
                     continue;
                 }
 
-                convert_array2info_recursive($var_name . '.' . $key, $val, $sum_array_name);
+                convert_array2info_recursive($var_name.'.'.$key, $val, $sum_array_name);
             }
 
             return;
@@ -56,7 +56,7 @@ 
         case 'integer':
         case 'float':
         case 'double':
-            $GLOBALS[$sum_array_name][$var_name] = (string) $var_value;
+            $GLOBALS[$sum_array_name][$var_name] = (string)$var_value;
 
             return;
         case 'null':
@@ -81,7 +81,7 @@ 
 {
     global $site_name;
 
-    return '<macromedia-extension name="XOOPS-' . $site_name . ' ' . $mxi_name . '" version="1.0" type="Suite" requires-restart="true">
+    return '<macromedia-extension name="XOOPS-'.$site_name.' '.$mxi_name.'" version="1.0" type="Suite" requires-restart="true">
     <products>
         <product name="Dreamweaver" version="6" primary="true">
     </products>
@@ -96,7 +96,7 @@ 
     ]]></license-agreement>
     <description><![CDATA[
 
-Usable template variables in ' . $mxi_name . '
+Usable template variables in ' . $mxi_name.'
 
     ]]></description>
     <ui-access><![CDATA[
@@ -107,7 +107,7 @@ 
 
     ]]></ui-access>
     <files>
-' . $file_entries . '
+' . $file_entries.'
     </files>
     <configuration-changes>
     </configuration-changes>
@@ -121,7 +121,7 @@ 
 $tplsvarsinfo_mod_tpl = [];
 $tplsvarsinfo_total = [];
 
-if ($handler = opendir(XOOPS_COMPILE_PATH . '/')) {
+if ($handler = opendir(XOOPS_COMPILE_PATH.'/')) {
     while (false !== ($file = readdir($handler))) {
         // skip files other than tplsvars_* files
 
@@ -137,7 +137,7 @@ 
             continue;
         }
 
-        $file_path = XOOPS_COMPILE_PATH . '/' . $file;
+        $file_path = XOOPS_COMPILE_PATH.'/'.$file;
 
         $file_body = implode('', file($file_path));
 
@@ -183,13 +183,13 @@ 
 </snippet>';
 
 if (!empty($_POST['as_dw_extension_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['as_dw_extension_tgz'])) {
-    require_once XOOPS_ROOT_PATH . '/class/tardownloader.php';
+    require_once XOOPS_ROOT_PATH.'/class/tardownloader.php';
 
     $downloader = new \XoopsTarDownloader();
 
@@ -222,9 +222,9 @@ 
 
         $snippet_body = sprintf($snippet_format, $name, $description);
 
-        $file_name = strtr($key, '.', '_') . '.csn';
+        $file_name = strtr($key, '.', '_').'.csn';
 
-        $downloader->addFileData($snippet_body, $dw_snippets_dirname . '/' . $file_name);
+        $downloader->addFileData($snippet_body, $dw_snippets_dirname.'/'.$file_name);
     }
 
     // make a mxi file per module
@@ -236,15 +236,15 @@ 
             foreach ($tplsvarsinfo as $key => $val) {
                 $name = mb_substr($key, 1);
 
-                $file_name = strtr($key, '.', '_') . '.csn';
+                $file_name = strtr($key, '.', '_').'.csn';
 
-                $file_entries .= "\t\t" . '<file name="' . $dw_snippets_dirname . '/' . $file_name . '" destination="$Dreamweaver/Configuration/Snippets/XOOPS-' . $site_name . '/' . $tpl_name . '">' . "\n";
+                $file_entries .= "\t\t".'<file name="'.$dw_snippets_dirname.'/'.$file_name.'" destination="$Dreamweaver/Configuration/Snippets/XOOPS-'.$site_name.'/'.$tpl_name.'">'."\n";
             }
         }
 
         $mxi_body = get_mxi_body($mod_name, $file_entries);
 
-        $downloader->addFileData($mxi_body, $mod_name . '.mxi');
+        $downloader->addFileData($mxi_body, $mod_name.'.mxi');
     }
 
     //bugfix by nao-pon ,echo is not necessary for downloader