mambax7 / altsys

xoops_trust_path/libs/altsys/onuninstall.php

Spacing +16 added lines, -16 removed lines

@@ -1,35 +1,35 @@ 
 <?php
 
-eval(' function xoops_module_uninstall_'.$mydirname.'( $module ) { return altsys_onuninstall_base( $module , "'.$mydirname.'" ) ; } ') ;
+eval(' function xoops_module_uninstall_'.$mydirname.'( $module ) { return altsys_onuninstall_base( $module , "'.$mydirname.'" ) ; } ');
 
 
-if (! function_exists('altsys_onuninstall_base')) {
+if (!function_exists('altsys_onuninstall_base')) {
     function altsys_onuninstall_base($module, $mydirname)
     {
         // transations on module uninstall
 
-    global $ret ; // TODO :-D
+    global $ret; // TODO :-D
 
     // for Cube 2.1
     if (defined('XOOPS_CUBE_LEGACY')) {
-        $root =& XCube_Root::getSingleton();
-        $root->mDelegateManager->add('Legacy.Admin.Event.ModuleUninstall.' . ucfirst($mydirname) . '.Success', 'altsys_message_append_onuninstall') ;
-        $ret = array() ;
+        $root = & XCube_Root::getSingleton();
+        $root->mDelegateManager->add('Legacy.Admin.Event.ModuleUninstall.'.ucfirst($mydirname).'.Success', 'altsys_message_append_onuninstall');
+        $ret = array();
     } else {
-        if (! is_array($ret)) {
-            $ret = array() ;
+        if (!is_array($ret)) {
+            $ret = array();
         }
     }
 
-        $db =& XoopsDatabaseFactory::getDatabaseConnection() ;
-        $mid = $module->getVar('mid') ;
+        $db = & XoopsDatabaseFactory::getDatabaseConnection();
+        $mid = $module->getVar('mid');
 
     // TABLES (loading mysql.sql)
-    $sql_file_path = __DIR__.'/sql/mysql.sql' ;
-        $prefix_mod = $db->prefix() . '_' . $mydirname ;
+    $sql_file_path = __DIR__.'/sql/mysql.sql';
+        $prefix_mod = $db->prefix().'_'.$mydirname;
         if (file_exists($sql_file_path)) {
-            $ret[] = 'SQL file found at <b>' . htmlspecialchars($sql_file_path) . '</b>.<br  /> Deleting tables...<br />';
-            $sql_lines = file($sql_file_path) ;
+            $ret[] = 'SQL file found at <b>'.htmlspecialchars($sql_file_path).'</b>.<br  /> Deleting tables...<br />';
+            $sql_lines = file($sql_file_path);
             foreach ($sql_lines as $sql_line) {
                 if (preg_match('/^CREATE TABLE \`?([a-zA-Z0-9_-]+)\`? /i', $sql_line, $regs)) {
                     $sql = 'DROP TABLE '.addslashes($prefix_mod.'_'.$regs[1]);
@@ -59,14 +59,14 @@ 
     unset($templates); */
 
 
-    return true ;
+    return true;
     }
 
     function altsys_message_append_onuninstall(&$module_obj, &$log)
     {
         if (is_array(@$GLOBALS['ret'])) {
             foreach ($GLOBALS['ret'] as $message) {
-                $log->add(strip_tags($message)) ;
+                $log->add(strip_tags($message));
             }
         }
 

xoops_trust_path/libs/altsys/mypreferences.php

Spacing +54 added lines, -54 removed lines

@@ -5,64 +5,64 @@ 
 //                     GIJOE <http://www.peak.ne.jp/>                        //
 // ------------------------------------------------------------------------- //
 
-require_once __DIR__.'/class/AltsysBreadcrumbs.class.php' ;
-include_once __DIR__.'/include/gtickets.php' ;
-include_once __DIR__.'/include/altsys_functions.php' ;
+require_once __DIR__.'/class/AltsysBreadcrumbs.class.php';
+include_once __DIR__.'/include/gtickets.php';
+include_once __DIR__.'/include/altsys_functions.php';
 
 // check access right (needs module_admin of this module)
-if (! is_object($xoopsUser) || ! is_object($xoopsModule) || ! $xoopsUser->isAdmin($xoopsModule->mid())) {
-    die('Access Denied') ;
+if (!is_object($xoopsUser) || !is_object($xoopsModule) || !$xoopsUser->isAdmin($xoopsModule->mid())) {
+    die('Access Denied');
 }
 
 
 // initials
-$db =& XoopsDatabaseFactory::getDatabaseConnection();
-(method_exists('MyTextSanitizer', 'sGetInstance') and $myts =& MyTextSanitizer::sGetInstance()) || $myts =& MyTextSanitizer::getInstance() ;
+$db = & XoopsDatabaseFactory::getDatabaseConnection();
+(method_exists('MyTextSanitizer', 'sGetInstance') and $myts = & MyTextSanitizer::sGetInstance()) || $myts = & MyTextSanitizer::getInstance();
 
 // language file
-altsys_include_language_file('mypreferences') ;
+altsys_include_language_file('mypreferences');
 
 
-$op = empty($_GET['op']) ? 'showmod' : preg_replace('/[^a-zA-Z0-9_-]/', '', $_GET['op']) ;
+$op = empty($_GET['op']) ? 'showmod' : preg_replace('/[^a-zA-Z0-9_-]/', '', $_GET['op']);
 
 if ($op == 'showmod') {
-    $config_handler =& xoops_gethandler('config');
-    $mod = $xoopsModule->mid() ;
-    $config =& $config_handler->getConfigs(new Criteria('conf_modid', $mod));
+    $config_handler = & xoops_gethandler('config');
+    $mod = $xoopsModule->mid();
+    $config = & $config_handler->getConfigs(new Criteria('conf_modid', $mod));
     $count = count($config);
     if ($count < 1) {
-        die('no configs') ;
+        die('no configs');
     }
     include_once XOOPS_ROOT_PATH.'/class/xoopsformloader.php';
     $form = new XoopsThemeForm(_MD_A_MYPREFERENCES_FORMTITLE, 'pref_form', 'index.php?mode=admin&lib=altsys&page=mypreferences&op=save');
-    $module_handler =& xoops_gethandler('module');
-    $module =& $module_handler->get($mod);
+    $module_handler = & xoops_gethandler('module');
+    $module = & $module_handler->get($mod);
 
     // language
-    $language = empty($xoopsConfig['language']) ? 'english' : $xoopsConfig['language'] ;
+    $language = empty($xoopsConfig['language']) ? 'english' : $xoopsConfig['language'];
 
     // load modinfo.php if necessary (judged by a specific constant is defined)
-    if (! defined('_MYMENU_CONSTANT_IN_MODINFO') || ! defined(_MYMENU_CONSTANT_IN_MODINFO)) {
+    if (!defined('_MYMENU_CONSTANT_IN_MODINFO') || !defined(_MYMENU_CONSTANT_IN_MODINFO)) {
         if (file_exists("$mydirpath/language/$language/modinfo.php")) {
             // user customized language file
-            include_once "$mydirpath/language/$language/modinfo.php" ;
+            include_once "$mydirpath/language/$language/modinfo.php";
         } elseif (file_exists("$mytrustdirpath/language/$language/modinfo.php")) {
             // default language file
-            include_once "$mytrustdirpath/language/$language/modinfo.php" ;
+            include_once "$mytrustdirpath/language/$language/modinfo.php";
         } else {
             // fallback english
-            include_once "$mytrustdirpath/language/english/modinfo.php" ;
+            include_once "$mytrustdirpath/language/english/modinfo.php";
         }
     }
 
     // if has comments feature, need comment lang file
-    if ($module->getVar('hascomments') == 1 && ! defined('_CM_TITLE')) {
+    if ($module->getVar('hascomments') == 1 && !defined('_CM_TITLE')) {
         include_once XOOPS_ROOT_PATH.'/language/'.$xoopsConfig['language'].'/comment.php';
     }
 
     // RMV-NOTIFY
     // if has notification feature, need notification lang file
-    if ($module->getVar('hasnotification') == 1 && ! defined('_NOT_NOTIFICATIONOPTIONS')) {
+    if ($module->getVar('hasnotification') == 1 && !defined('_NOT_NOTIFICATIONOPTIONS')) {
         include_once XOOPS_ROOT_PATH.'/language/'.$xoopsConfig['language'].'/notification.php';
     }
 
@@ -72,12 +72,12 @@ 
         //	$form->addElement(new XoopsFormHidden('redirect', XOOPS_URL.'/modules/'.$module->getVar('dirname').'/'.$module->getInfo('adminindex')));
     // }
     for ($i = 0; $i < $count; $i++) {
-        $title_icon = ($config[$i]->getVar('conf_valuetype') === 'encrypt')? '<img src="'.XOOPS_MODULE_URL.'/legacy/admin/theme/icons/textfield_key.png" alt="Encrypted">' : ''; // support XCL 2.2.3 'encrypt' of 'conf_valuetype'
+        $title_icon = ($config[$i]->getVar('conf_valuetype') === 'encrypt') ? '<img src="'.XOOPS_MODULE_URL.'/legacy/admin/theme/icons/textfield_key.png" alt="Encrypted">' : ''; // support XCL 2.2.3 'encrypt' of 'conf_valuetype'
         $title4tray = (!defined($config[$i]->getVar('conf_desc')) || constant($config[$i]->getVar('conf_desc')) == '') ? (constant($config[$i]->getVar('conf_title')).$title_icon) : (constant($config[$i]->getVar('conf_title')).$title_icon.'<br /><br /><span style="font-weight:normal;">'.constant($config[$i]->getVar('conf_desc')).'</span>'); // GIJ
-        $title = '' ; // GIJ
+        $title = ''; // GIJ
         switch ($config[$i]->getVar('conf_formtype')) {
         case 'textarea':
-            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts =& MyTextSanitizer::sGetInstance()) || $myts =& MyTextSanitizer::getInstance();
+            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts = & MyTextSanitizer::sGetInstance()) || $myts = & MyTextSanitizer::getInstance();
             if ($config[$i]->getVar('conf_valuetype') == 'array') {
                 // this is exceptional.. only when value type is arrayneed a smarter way for this
                 $ele = ($config[$i]->getVar('conf_value') != '') ? new XoopsFormTextArea($title, $config[$i]->getVar('conf_name'), $myts->htmlspecialchars(implode('|', $config[$i]->getConfValueForOutput())), 5, 50) : new XoopsFormTextArea($title, $config[$i]->getVar('conf_name'), '', 5, 50);
@@ -94,7 +94,7 @@ 
                 $ele = new XoopsFormRadio($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput());
                 $addBr = '<br />';
             }
-            $options =& $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));
+            $options = & $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));
             $opcount = count($options);
             for ($j = 0; $j < $opcount; $j++) {
                 $optval = defined($options[$j]->getVar('confop_value')) ? constant($options[$j]->getVar('confop_value')) : $options[$j]->getVar('confop_value');
@@ -111,7 +111,7 @@ 
                 $ele = new XoopsFormCheckBox($title, $config[$i]->getVar('conf_name'), $config[$i]->getConfValueForOutput());
                 $addBr = '<br />';
             }
-            $options =& $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));
+            $options = & $config_handler->getConfigOptions(new Criteria('conf_id', $config[$i]->getVar('conf_id')));
             $opcount = count($options);
             for ($j = 0; $j < $opcount; $j++) {
                 $optval = defined($options[$j]->getVar('confop_value')) ? constant($options[$j]->getVar('confop_value')) : $options[$j]->getVar('confop_value');
@@ -145,37 +145,37 @@ 
             $ele = new XoopsFormSelectUser($title, $config[$i]->getVar('conf_name'), false, $config[$i]->getConfValueForOutput(), 5, true);
             break;
         case 'password':
-            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts =& MyTextSanitizer::sGetInstance()) || $myts =& MyTextSanitizer::getInstance();
+            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts = & MyTextSanitizer::sGetInstance()) || $myts = & MyTextSanitizer::getInstance();
             $ele = new XoopsFormPassword($title, $config[$i]->getVar('conf_name'), 50, 255, $myts->htmlspecialchars($config[$i]->getConfValueForOutput()));
             break;
         case 'textbox':
         default:
-            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts =& MyTextSanitizer::sGetInstance()) || $myts =& MyTextSanitizer::getInstance();
+            (method_exists('MyTextSanitizer', 'sGetInstance') and $myts = & MyTextSanitizer::sGetInstance()) || $myts = & MyTextSanitizer::getInstance();
             $ele = new XoopsFormText($title, $config[$i]->getVar('conf_name'), 50, 255, $myts->htmlspecialchars($config[$i]->getConfValueForOutput()));
             break;
         }
         $hidden = new XoopsFormHidden('conf_ids[]', $config[$i]->getVar('conf_id'));
-        $ele_tray = new XoopsFormElementTray($title4tray, '') ;
+        $ele_tray = new XoopsFormElementTray($title4tray, '');
         $ele_tray->addElement($ele);
         $ele_tray->addElement($hidden);
-        $form->addElement($ele_tray) ;
+        $form->addElement($ele_tray);
         unset($ele_tray);
         unset($ele);
         unset($hidden);
     }
     // $button_tray->addElement(new XoopsFormHidden('op', 'save'));
-    $xoopsGTicket->addTicketXoopsFormElement($button_tray, __LINE__, 1800, 'mypreferences') ;
+    $xoopsGTicket->addTicketXoopsFormElement($button_tray, __LINE__, 1800, 'mypreferences');
     $button_tray->addElement(new XoopsFormButton('', 'button', _GO, 'submit'));
-    $form->addElement($button_tray) ;
+    $form->addElement($button_tray);
     xoops_cp_header();
 
     // GIJ patch start
-    altsys_include_mymenu() ;
-    $breadcrumbsObj =& AltsysBreadcrumbs::getInstance() ;
+    altsys_include_mymenu();
+    $breadcrumbsObj = & AltsysBreadcrumbs::getInstance();
     if ($breadcrumbsObj->hasPaths()) {
-        $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mypreferences', _PREFERENCES) ;
+        $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mypreferences', _PREFERENCES);
     }
-    echo "<h3 style='text-align:"._GLOBAL_LEFT.";'>".$module->getvar('name').' &nbsp; '._PREFERENCES."</h3>\n" ;
+    echo "<h3 style='text-align:"._GLOBAL_LEFT.";'>".$module->getvar('name').' &nbsp; '._PREFERENCES."</h3>\n";
     // GIJ patch end
 
     $form->display();
@@ -187,10 +187,10 @@ 
     //if ( !admin_refcheck("/modules/$admin_mydirname/admin/") ) {
     //	exit('Invalid referer');
     //}
-    if (! $xoopsGTicket->check(true, 'mypreferences')) {
+    if (!$xoopsGTicket->check(true, 'mypreferences')) {
         redirect_header(XOOPS_URL.'/', 3, $xoopsGTicket->getErrors());
     }
-    require_once XOOPS_ROOT_PATH.'/class/template.php' ;
+    require_once XOOPS_ROOT_PATH.'/class/template.php';
     $xoopsTpl = new XoopsTpl();
 //HACK by domifara for new XOOPS and XCL etc.
 //old xoops
@@ -210,8 +210,8 @@ 
     $lang_updated = false;
     if ($count > 0) {
         for ($i = 0; $i < $count; $i++) {
-            $config =& $config_handler->getConfig($conf_ids[$i]);
-            $new_value =& $_POST[$config->getVar('conf_name')];
+            $config = & $config_handler->getConfig($conf_ids[$i]);
+            $new_value = & $_POST[$config->getVar('conf_name')];
             if (is_array($new_value) || $new_value != $config->getVar('conf_value')) {
                 // if language has been changed
                 if (!$lang_updated && $config->getVar('conf_catid') == XOOPS_CONF && $config->getVar('conf_name') == 'language') {
@@ -223,7 +223,7 @@ 
 
                 // if default theme has been changed
                 if (!$theme_updated && $config->getVar('conf_catid') == XOOPS_CONF && $config->getVar('conf_name') == 'theme_set') {
-                    $member_handler =& xoops_gethandler('member');
+                    $member_handler = & xoops_gethandler('member');
                     $member_handler->updateUsersByField('theme', $_POST[$config->getVar('conf_name')]);
                     $theme_updated = true;
                 }
@@ -239,14 +239,14 @@ 
 
                         // generate compiled files for the new theme
                         // block files only for now..
-                        $tplfile_handler =& xoops_gethandler('tplfile');
-                        $dtemplates =& $tplfile_handler->find('default', 'block');
+                        $tplfile_handler = & xoops_gethandler('tplfile');
+                        $dtemplates = & $tplfile_handler->find('default', 'block');
                         $dcount = count($dtemplates);
 
                         // need to do this to pass to xoops_template_touch function
                         $GLOBALS['xoopsConfig']['template_set'] = $newtplset;
 
-                        altsys_clear_templates_c() ;
+                        altsys_clear_templates_c();
 
 /*						for ($i = 0; $i < $dcount; $i++) {
                             $found =& $tplfile_handler->find($newtplset, 'block', $dtemplates[$i]->getVar('tpl_refid'), null);
@@ -260,8 +260,8 @@ 
                         }*/
 
                         // generate image cache files from image binary data, save them under cache/
-                        $image_handler =& xoops_gethandler('imagesetimg');
-                        $imagefiles =& $image_handler->getObjects(new Criteria('tplset_name', $newtplset), true);
+                        $image_handler = & xoops_gethandler('imagesetimg');
+                        $imagefiles = & $image_handler->getObjects(new Criteria('tplset_name', $newtplset), true);
                         foreach (array_keys($imagefiles) as $i) {
                             if (!$fp = fopen(XOOPS_CACHE_PATH.'/'.$newtplset.'_'.$imagefiles[$i]->getVar('imgsetimg_file'), 'wb')) {
                             } else {
@@ -274,12 +274,12 @@ 
                 }
 
                 // add read permission for the start module to all groups
-                if (!$startmod_updated    && $new_value != '--' && $config->getVar('conf_catid') == XOOPS_CONF && $config->getVar('conf_name') == 'startpage') {
-                    $member_handler =& xoops_gethandler('member');
-                    $groups =& $member_handler->getGroupList();
-                    $moduleperm_handler =& xoops_gethandler('groupperm');
-                    $module_handler =& xoops_gethandler('module');
-                    $module =& $module_handler->getByDirname($new_value);
+                if (!$startmod_updated && $new_value != '--' && $config->getVar('conf_catid') == XOOPS_CONF && $config->getVar('conf_name') == 'startpage') {
+                    $member_handler = & xoops_gethandler('member');
+                    $groups = & $member_handler->getGroupList();
+                    $moduleperm_handler = & xoops_gethandler('groupperm');
+                    $module_handler = & xoops_gethandler('module');
+                    $module = & $module_handler->getByDirname($new_value);
                     foreach ($groups as $groupid => $groupname) {
                         if (!$moduleperm_handler->checkRight('module_read', $module->getVar('mid'), $groupid)) {
                             $moduleperm_handler->addRight('module_read', $module->getVar('mid'), $groupid);
@@ -295,5 +295,5 @@ 
         }
     }
 
-    redirect_header('index.php?mode=admin&lib=altsys&page=mypreferences', 2, _MD_A_MYPREFERENCES_UPDATED) ;
+    redirect_header('index.php?mode=admin&lib=altsys&page=mypreferences', 2, _MD_A_MYPREFERENCES_UPDATED);
 }

xoops_trust_path/libs/altsys/admin_menu.php

Spacing +18 added lines, -18 removed lines

@@ -1,35 +1,35 @@ 
 <?php
 
-if (! defined('XOOPS_ROOT_PATH')) {
-    exit ;
+if (!defined('XOOPS_ROOT_PATH')) {
+    exit;
 }
 
 $adminmenu = array(
     array(
-        'title' => _MI_ALTSYS_MENU_CUSTOMBLOCKS ,
+        'title' => _MI_ALTSYS_MENU_CUSTOMBLOCKS,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname=__CustomBlocks__'
-    ) ,
+    ),
     array(
-        'title' => _MI_ALTSYS_MENU_NEWCUSTOMBLOCK ,
-        'link' => 'admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname=__CustomBlocks__&op=edit' ,
+        'title' => _MI_ALTSYS_MENU_NEWCUSTOMBLOCK,
+        'link' => 'admin/index.php?mode=admin&lib=altsys&page=myblocksadmin&dirname=__CustomBlocks__&op=edit',
         'show' => false
-    ) ,
+    ),
     array(
-        'title' => _MI_ALTSYS_MENU_MYBLOCKSADMIN ,
+        'title' => _MI_ALTSYS_MENU_MYBLOCKSADMIN,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=myblocksadmin'
-    ) ,
+    ),
     array(
-        'title' => _MI_ALTSYS_MENU_MYTPLSADMIN ,
+        'title' => _MI_ALTSYS_MENU_MYTPLSADMIN,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=mytplsadmin'
-    ) ,
+    ),
     array(
-        'title' => _MI_ALTSYS_MENU_COMPILEHOOKADMIN ,
+        'title' => _MI_ALTSYS_MENU_COMPILEHOOKADMIN,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=compilehookadmin'
-    ) ,
+    ),
     array(
-        'title' => _MI_ALTSYS_MENU_MYLANGADMIN ,
+        'title' => _MI_ALTSYS_MENU_MYLANGADMIN,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=mylangadmin'
-    ) ,
+    ),
 /*  array(
         'title' => _MI_ALTSYS_MENU_MYAVATAR ,
         'link' => 'index.php?mode=admin&lib=altsys&page=myavatar' ,
@@ -38,11 +38,11 @@ 
         'title' => _MI_ALTSYS_MENU_MYSMILEY ,
         'link' => 'index.php?mode=admin&lib=altsys&page=mysmiley' ,
     ) ,*/
-) ;
+);
 
 $adminmenu4altsys = array(
     array(
-        'title' => 'ALTSYS '._PREFERENCES ,
+        'title' => 'ALTSYS '._PREFERENCES,
         'link' => 'admin/index.php?mode=admin&lib=altsys&page=mypreferences'
     )
-) ;
+);

xoops_trust_path/libs/altsys/xoops_version.php

Spacing +56 added lines, -56 removed lines

@@ -1,14 +1,14 @@ 
 <?php
 
-include_once __DIR__.'/include/altsys_functions.php' ;
+include_once __DIR__.'/include/altsys_functions.php';
 
 // language file (modinfo.php)
-altsys_include_language_file('modinfo') ;
+altsys_include_language_file('modinfo');
 
-$modversion['name'] = _MI_ALTSYS_MODULENAME ;
-$modversion['version'] = '0.82' ;
-$modversion['detailed_version'] = '0.82.5' ;
-$modversion['description'] = _MI_ALTSYS_MODULEDESC ;
+$modversion['name'] = _MI_ALTSYS_MODULENAME;
+$modversion['version'] = '0.82';
+$modversion['detailed_version'] = '0.82.5';
+$modversion['description'] = _MI_ALTSYS_MODULEDESC;
 $modversion['credits'] = 'PEAK Corp.';
 $modversion['author'] = 'GIJ=CHECKMATE<br />PEAK Corp.(http://www.peak.ne.jp/), XOOPS X Distribution(https://github.com/XoopsX)';
 $modversion['license'] = 'GPL see LICENSE';
@@ -22,18 +22,18 @@ 
 $modversion['adminmenu'] = 'admin/admin_menu.php';
 
 // All Templates can't be touched by modulesadmin.
-$modversion['templates'] = array() ;
+$modversion['templates'] = array();
 
 // Blocks
 $modversion['blocks'][1] = array(
-    'file'            => 'blocks.php' ,
-    'name'            => _MI_ALTSYS_BNAME_ADMIN_MENU ,
-    'description'    => '' ,
-    'show_func'        => 'b_altsys_admin_menu_show' ,
-    'edit_func'        => 'b_altsys_admin_menu_edit' ,
-    'options'        => "$mydirname" ,
-    'template'        => '' , // use "module" template instead
-) ;
+    'file'            => 'blocks.php',
+    'name'            => _MI_ALTSYS_BNAME_ADMIN_MENU,
+    'description'    => '',
+    'show_func'        => 'b_altsys_admin_menu_show',
+    'edit_func'        => 'b_altsys_admin_menu_edit',
+    'options'        => "$mydirname",
+    'template'        => '', // use "module" template instead
+);
 
 // Menu
 $modversion['hasMain'] = 1;
@@ -46,64 +46,64 @@ 
 
 // Configurations
 $modversion['config'][1] = array(
-    'name'            => 'adminmenu_hack_ft' ,
-    'title'            => '_MI_ALTSYS_ADMINMENU_HFT' ,
-    'description'    => '_MI_ALTSYS_ADMINMENU_HFTDSC' ,
-    'formtype'        => 'select' ,
-    'valuetype'        => 'int' ,
-    'default'        => 0 ,
-    'options'        => array( '_NONE' => 0 , '_MI_ALTSYS_AMHFT_OPT_2COL' => 1 , '_MI_ALTSYS_AMHFT_OPT_NOIMG' => 2 , '_MI_ALTSYS_AMHFT_OPT_XCSTY' => 3 )
-) ;
+    'name'            => 'adminmenu_hack_ft',
+    'title'            => '_MI_ALTSYS_ADMINMENU_HFT',
+    'description'    => '_MI_ALTSYS_ADMINMENU_HFTDSC',
+    'formtype'        => 'select',
+    'valuetype'        => 'int',
+    'default'        => 0,
+    'options'        => array('_NONE' => 0, '_MI_ALTSYS_AMHFT_OPT_2COL' => 1, '_MI_ALTSYS_AMHFT_OPT_NOIMG' => 2, '_MI_ALTSYS_AMHFT_OPT_XCSTY' => 3)
+);
 
 $modversion['config'][] = array(
-    'name'            => 'adminmenu_insert_mymenu' ,
-    'title'            => '_MI_ALTSYS_ADMINMENU_IM' ,
-    'description'    => '_MI_ALTSYS_ADMINMENU_IMDSC' ,
-    'formtype'        => 'yesno' ,
-    'valuetype'        => 'int' ,
-    'default'        => 0 ,
+    'name'            => 'adminmenu_insert_mymenu',
+    'title'            => '_MI_ALTSYS_ADMINMENU_IM',
+    'description'    => '_MI_ALTSYS_ADMINMENU_IMDSC',
+    'formtype'        => 'yesno',
+    'valuetype'        => 'int',
+    'default'        => 0,
     'options'        => array()
-) ;
+);
 
 $modversion['config'][] = array(
-    'name'            => 'admin_in_theme' ,
-    'title'            => '_MI_ALTSYS_ADMIN_IN_THEME' ,
-    'description'    => '_MI_ALTSYS_ADMIN_IN_THEMEDSC' ,
-    'formtype'        => 'textbox' ,
-    'valuetype'        => 'text' ,
-    'default'        => 'default' ,
+    'name'            => 'admin_in_theme',
+    'title'            => '_MI_ALTSYS_ADMIN_IN_THEME',
+    'description'    => '_MI_ALTSYS_ADMIN_IN_THEMEDSC',
+    'formtype'        => 'textbox',
+    'valuetype'        => 'text',
+    'default'        => 'default',
     'options'        => array()
-) ;
+);
 
 $modversion['config'][] = array(
-    'name'            => 'enable_force_clone' ,
-    'title'            => '_MI_ALTSYS_ENABLEFORCECLONE' ,
-    'description'    => '_MI_ALTSYS_ENABLEFORCECLONEDSC' ,
-    'formtype'        => 'yesno' ,
-    'valuetype'        => 'int' ,
-    'default'        => 1 ,
+    'name'            => 'enable_force_clone',
+    'title'            => '_MI_ALTSYS_ENABLEFORCECLONE',
+    'description'    => '_MI_ALTSYS_ENABLEFORCECLONEDSC',
+    'formtype'        => 'yesno',
+    'valuetype'        => 'int',
+    'default'        => 1,
     'options'        => array()
-) ;
+);
 
 $modversion['config'][] = array(
-    'name'            => 'images_dir' ,
-    'title'            => '_MI_ALTSYS_IMAGES_DIR' ,
-    'description'    => '_MI_ALTSYS_IMAGES_DIRDSC' ,
-    'formtype'        => 'textbox' ,
-    'valuetype'        => 'text' ,
-    'default'        => 'images' ,
+    'name'            => 'images_dir',
+    'title'            => '_MI_ALTSYS_IMAGES_DIR',
+    'description'    => '_MI_ALTSYS_IMAGES_DIRDSC',
+    'formtype'        => 'textbox',
+    'valuetype'        => 'text',
+    'default'        => 'images',
     'options'        => array()
-) ;
+);
 
 // Notification
 
 $modversion['hasNotification'] = 0;
 
-$modversion['onInstall'] = 'include/oninstall.php' ;
-$modversion['onUpdate'] = 'include/onupdate.php' ;
-$modversion['onUninstall'] = 'include/onuninstall.php' ;
+$modversion['onInstall'] = 'include/oninstall.php';
+$modversion['onUpdate'] = 'include/onupdate.php';
+$modversion['onUninstall'] = 'include/onuninstall.php';
 
 // keep block's options
-if (! defined('XOOPS_CUBE_LEGACY') && substr(XOOPS_VERSION, 6, 3) < 2.1 && ! empty($_POST['fct']) && ! empty($_POST['op']) && $_POST['fct'] == 'modulesadmin' && $_POST['op'] == 'update_ok' && $_POST['dirname'] == $modversion['dirname']) {
-    include __DIR__.'/include/x20_keepblockoptions.inc.php' ;
+if (!defined('XOOPS_CUBE_LEGACY') && substr(XOOPS_VERSION, 6, 3) < 2.1 && !empty($_POST['fct']) && !empty($_POST['op']) && $_POST['fct'] == 'modulesadmin' && $_POST['op'] == 'update_ok' && $_POST['dirname'] == $modversion['dirname']) {
+    include __DIR__.'/include/x20_keepblockoptions.inc.php';
 }

xoops_trust_path/libs/altsys/blocks/block_functions.php

Spacing +11 added lines, -12 removed lines

@@ -1,29 +1,29 @@ 
 <?php
 
-require_once dirname(__DIR__).'/include/altsys_functions.php' ;
+require_once dirname(__DIR__).'/include/altsys_functions.php';
 
 function b_altsys_admin_menu_show($options)
 {
-    global $xoopsUser ;
+    global $xoopsUser;
 
-    $mydirname = empty($options[0]) ? 'altsys' : $options[0] ;
-    $this_template = empty($options[1]) ? 'db:'.$mydirname.'_block_admin_menu.html' : trim($options[1]) ;
+    $mydirname = empty($options[0]) ? 'altsys' : $options[0];
+    $this_template = empty($options[1]) ? 'db:'.$mydirname.'_block_admin_menu.html' : trim($options[1]);
 
     if (preg_match('/[^0-9a-zA-Z_-]/', $mydirname)) {
-        die('Invalid mydirname') ;
+        die('Invalid mydirname');
     }
-    if (! is_object(@$xoopsUser)) {
-        return array() ;
+    if (!is_object(@$xoopsUser)) {
+        return array();
     }
 
     // coretype
-    $coretype = altsys_get_core_type() ;
+    $coretype = altsys_get_core_type();
 
     // mid_selected
     if (is_object(@$GLOBALS['xoopsModule'])) {
-        $mid_selected = $GLOBALS['xoopsModule']->getVar('mid') ;
+        $mid_selected = $GLOBALS['xoopsModule']->getVar('mid');
         // for system->preferences
-<<<<<<< HEAD
+<< << <<< HEAD
         if ($mid_selected == 1 && @$_GET['fct'] == 'preferences' && @$_GET['op'] == 'showmod' && ! empty($_GET['mod'])) {
             $mid_selected = intval($_GET['mod']) ;
 =======
@@ -156,5 +156,4 @@ 
 		<br />
 	\n" ;
 
-    return $form;
-}
+    return $form
\ No newline at end of file

xoops_trust_path/libs/altsys/mymenu.php

Spacing +24 added lines, -24 removed lines

@@ -2,30 +2,30 @@ 
 
 /********* mymenu only for ALTSYS ********/
 
-if (! defined('XOOPS_ROOT_PATH')) {
-    exit ;
+if (!defined('XOOPS_ROOT_PATH')) {
+    exit;
 }
 
 // Skip for ORETEKI XOOPS
 if (defined('XOOPS_ORETEKI')) {
-    return ;
+    return;
 }
 
-if (! isset($module) || ! is_object($module)) {
-    $module = $xoopsModule ;
-} elseif (! is_object($xoopsModule)) {
-    die('$xoopsModule is not set')  ;
+if (!isset($module) || !is_object($module)) {
+    $module = $xoopsModule;
+} elseif (!is_object($xoopsModule)) {
+    die('$xoopsModule is not set');
 }
 
 // language files (modinfo.php)
-altsys_include_language_file('modinfo') ;
+altsys_include_language_file('modinfo');
 
-include __DIR__.'/admin_menu.php' ;
+include __DIR__.'/admin_menu.php';
 
-$adminmenu = array_merge($adminmenu, $adminmenu4altsys) ;
+$adminmenu = array_merge($adminmenu, $adminmenu4altsys);
 
-$mymenu_uri = empty($mymenu_fake_uri) ? $_SERVER['REQUEST_URI'] : $mymenu_fake_uri ;
-$mymenu_link = substr(strstr($mymenu_uri, '/admin/'), 1) ;
+$mymenu_uri = empty($mymenu_fake_uri) ? $_SERVER['REQUEST_URI'] : $mymenu_fake_uri;
+$mymenu_link = substr(strstr($mymenu_uri, '/admin/'), 1);
 
 // xoops_breadcrumbs
 // $GLOBALS['altsysXoopsBreadcrumbs'] = array( array( 'url' => XOOPS_URL.'/modules/altsys/admin/index.php' , 'name' => $xoopsModule->getVar('name') ) ) ;
@@ -33,19 +33,19 @@ 
 // highlight
 foreach (array_keys($adminmenu) as $i) {
     if ($mymenu_link == $adminmenu[$i]['link']) {
-        $adminmenu[$i]['selected'] = true ;
-        $adminmenu_hilighted = true ;
+        $adminmenu[$i]['selected'] = true;
+        $adminmenu_hilighted = true;
 //		$GLOBALS['altsysXoopsBreadcrumbs'][] = array( 'url' => XOOPS_URL."/modules/altsys/".htmlspecialchars($adminmenu[$i]['link'],ENT_QUOTES) , 'name' => htmlspecialchars( $adminmenu[$i]['title'] , ENT_QUOTES ) ) ;
     } else {
-        $adminmenu[$i]['selected'] = false ;
+        $adminmenu[$i]['selected'] = false;
     }
 }
 if (empty($adminmenu_hilighted)) {
     foreach (array_keys($adminmenu) as $i) {
         if (stristr($mymenu_uri, $adminmenu[$i]['link'])) {
-            $adminmenu[$i]['selected'] = true ;
+            $adminmenu[$i]['selected'] = true;
 //			$GLOBALS['altsysXoopsBreadcrumbs'][] = array( 'url' => XOOPS_URL."/modules/altsys/".htmlspecialchars($adminmenu[$i]['link'],ENT_QUOTES) , 'name' => htmlspecialchars( $adminmenu[$i]['title'] , ENT_QUOTES ) ) ;
-            break ;
+            break;
         }
     }
 }
@@ -53,20 +53,20 @@ 
 // link conversion from relative to absolute
 foreach (array_keys($adminmenu) as $i) {
     if (stristr($adminmenu[$i]['link'], XOOPS_URL) === false) {
-        $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' ;
-$tpl = new D3Tpl() ;
+require_once XOOPS_TRUST_PATH.'/libs/altsys/class/D3Tpl.class.php';
+$tpl = new D3Tpl();
 $tpl->assign(array(
     'adminmenu' => $adminmenu
-)) ;
-$tpl->display('db:altsys_inc_mymenu.html') ;
+));
+$tpl->display('db:altsys_inc_mymenu.html');
 
 // submenu
-$page = preg_replace('/[^0-9a-zA-Z_-]/', '', @$_GET['page']) ;
+$page = preg_replace('/[^0-9a-zA-Z_-]/', '', @$_GET['page']);
 if (file_exists(__DIR__.'/mymenusub/'.$page.'.php')) {
-    include __DIR__.'/mymenusub/'.$page.'.php' ;
+    include __DIR__.'/mymenusub/'.$page.'.php';
 }

xoops_trust_path/libs/altsys/mylangadmin.php

Spacing +116 added lines, -116 removed lines

@@ -5,170 +5,170 @@ 
 //                       GIJOE <http://www.peak.ne.jp/>                      //
 // ------------------------------------------------------------------------- //
 
-require_once __DIR__.'/class/AltsysBreadcrumbs.class.php' ;
-include_once __DIR__."/include/gtickets.php" ;
-include_once __DIR__.'/include/altsys_functions.php' ;
-include_once __DIR__.'/include/lang_functions.php' ;
-include_once __DIR__.'/class/D3LanguageManager.class.php' ;
+require_once __DIR__.'/class/AltsysBreadcrumbs.class.php';
+include_once __DIR__."/include/gtickets.php";
+include_once __DIR__.'/include/altsys_functions.php';
+include_once __DIR__.'/include/lang_functions.php';
+include_once __DIR__.'/class/D3LanguageManager.class.php';
 
 
 // only groups have 'module_admin' of 'altsys' can do that.
-$module_handler =& xoops_gethandler('module') ;
-$module =& $module_handler->getByDirname('altsys') ;
-if (! is_object($module)) {
-    die('install altsys') ;
+$module_handler = & xoops_gethandler('module');
+$module = & $module_handler->getByDirname('altsys');
+if (!is_object($module)) {
+    die('install altsys');
 }
-$moduleperm_handler =& xoops_gethandler('groupperm') ;
-if (! is_object(@$xoopsUser) || ! $moduleperm_handler->checkRight('module_admin', $module->getVar('mid'), $xoopsUser->getGroups())) {
-    die('only admin of altsys can access this area') ;
+$moduleperm_handler = & xoops_gethandler('groupperm');
+if (!is_object(@$xoopsUser) || !$moduleperm_handler->checkRight('module_admin', $module->getVar('mid'), $xoopsUser->getGroups())) {
+    die('only admin of altsys can access this area');
 }
 
 
 // initials
-$db =& XoopsDatabaseFactory::getDatabaseConnection();
-(method_exists('MyTextSanitizer', 'sGetInstance') and $myts =& MyTextSanitizer::sGetInstance()) || $myts =& MyTextSanitizer::getInstance() ;
-$langman =& D3LanguageManager::getInstance() ;
+$db = & XoopsDatabaseFactory::getDatabaseConnection();
+(method_exists('MyTextSanitizer', 'sGetInstance') and $myts = & MyTextSanitizer::sGetInstance()) || $myts = & MyTextSanitizer::getInstance();
+$langman = & D3LanguageManager::getInstance();
 
 // language file of this controller
-altsys_include_language_file('mylangadmin') ;
+altsys_include_language_file('mylangadmin');
 
 // check $xoopsModule
-if (! is_object($xoopsModule)) {
-    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM) ;
+if (!is_object($xoopsModule)) {
+    redirect_header(XOOPS_URL.'/user.php', 1, _NOPERM);
 }
 
 // set target_module if specified by $_GET['dirname']
-$module_handler =& xoops_gethandler('module');
-if (! empty($_GET['dirname'])) {
-    $dirname = preg_replace('/[^0-9a-zA-Z_-]/', '', $_GET['dirname']) ;
-    $target_module =& $module_handler->getByDirname($dirname) ;
+$module_handler = & xoops_gethandler('module');
+if (!empty($_GET['dirname'])) {
+    $dirname = preg_replace('/[^0-9a-zA-Z_-]/', '', $_GET['dirname']);
+    $target_module = & $module_handler->getByDirname($dirname);
 }
 
-if (! empty($target_module) && is_object($target_module)) {
+if (!empty($target_module) && is_object($target_module)) {
     // specified by dirname (for langadmin as an independent module)
-    $target_mid = $target_module->getVar('mid') ;
-    $target_dirname = $target_module->getVar('dirname') ;
-    $target_dirname4sql = addslashes($target_dirname) ;
-    $target_mname = $target_module->getVar('name') . '&nbsp;' . sprintf('(%2.2f)', $target_module->getVar('version') / 100.0) ;
+    $target_mid = $target_module->getVar('mid');
+    $target_dirname = $target_module->getVar('dirname');
+    $target_dirname4sql = addslashes($target_dirname);
+    $target_mname = $target_module->getVar('name').'&nbsp;'.sprintf('(%2.2f)', $target_module->getVar('version') / 100.0);
     //$query4redirect = '?dirname='.urlencode(strip_tags($_GET['dirname'])) ;
 } else {
     // not specified by dirname (for 3rd party modules as mylangadmin)
-    $target_mid = $xoopsModule->getVar('mid') ;
-    $target_dirname = $xoopsModule->getVar('dirname') ;
-    $target_dirname4sql = addslashes($target_dirname) ;
-    $target_mname = $xoopsModule->getVar('name') ;
+    $target_mid = $xoopsModule->getVar('mid');
+    $target_dirname = $xoopsModule->getVar('dirname');
+    $target_dirname4sql = addslashes($target_dirname);
+    $target_mname = $xoopsModule->getVar('name');
     //$query4redirect = '' ;
 }
 
 // basic GET variables
-$target_lang = preg_replace('/[^0-9a-zA-Z_-]/', '', @$_GET['target_lang']) ;
+$target_lang = preg_replace('/[^0-9a-zA-Z_-]/', '', @$_GET['target_lang']);
 if (empty($target_lang)) {
-    $target_lang = $GLOBALS['xoopsConfig']['language'] ;
+    $target_lang = $GLOBALS['xoopsConfig']['language'];
 }
-$target_lang4sql = addslashes($target_lang) ;
-$target_file = preg_replace('/[^0-9a-zA-Z_.-]/', '', @$_GET['target_file']) ;
+$target_lang4sql = addslashes($target_lang);
+$target_file = preg_replace('/[^0-9a-zA-Z_.-]/', '', @$_GET['target_file']);
 if (empty($target_file)) {
-    $target_file = 'main.php' ;
+    $target_file = 'main.php';
 }
 
 // get $target_trustdirname
-$mytrustdirname = '' ;
+$mytrustdirname = '';
 if (file_exists(XOOPS_ROOT_PATH.'/modules/'.$target_dirname.'/mytrustdirname.php')) {
-    require XOOPS_ROOT_PATH.'/modules/'.$target_dirname.'/mytrustdirname.php' ;
+    require XOOPS_ROOT_PATH.'/modules/'.$target_dirname.'/mytrustdirname.php';
 }
-$target_trustdirname = $mytrustdirname ;
+$target_trustdirname = $mytrustdirname;
 
 // get base directory
 if (empty($target_trustdirname)) {
     // conventinal module
-    $base_dir = XOOPS_ROOT_PATH.'/modules/'.$target_dirname.'/language' ;
+    $base_dir = XOOPS_ROOT_PATH.'/modules/'.$target_dirname.'/language';
 } else {
     // D3 module
-    $base_dir = XOOPS_TRUST_PATH.'/modules/'.$target_trustdirname.'/language' ;
+    $base_dir = XOOPS_TRUST_PATH.'/modules/'.$target_trustdirname.'/language';
 }
 
 // make list of language and check $target_lang
-$languages = array() ;
-$languages4disp = array() ;
-if (! is_dir($base_dir)) {
-    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODNOLANGUAGE) ;
+$languages = array();
+$languages4disp = array();
+if (!is_dir($base_dir)) {
+    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODNOLANGUAGE);
 }
-$dh = opendir($base_dir) ;
+$dh = opendir($base_dir);
 if ($dh) {
     while ($file = readdir($dh)) {
         if (substr($file, 0, 1) == '.') {
-            continue ;
+            continue;
         }
         if (is_dir("$base_dir/$file")) {
-            list($count) = $db->fetchRow($db->query('SELECT COUNT(*) FROM ' . $db->prefix('altsys_language_constants') . " WHERE mid=$target_mid AND language='" . addslashes($file) . "'")) ;
-            $languages[] = $file ;
-            $languages4disp[] = $file . " ($count)" ;
+            list($count) = $db->fetchRow($db->query('SELECT COUNT(*) FROM '.$db->prefix('altsys_language_constants')." WHERE mid=$target_mid AND language='".addslashes($file)."'"));
+            $languages[] = $file;
+            $languages4disp[] = $file." ($count)";
         }
     }
 }
-closedir($dh) ;
-if (! in_array($target_lang, $languages)) {
-    $target_lang = $languages[0] ;
+closedir($dh);
+if (!in_array($target_lang, $languages)) {
+    $target_lang = $languages[0];
 }
 
 // get base directory seleced language
-$lang_base_dir = $base_dir.'/'.$target_lang ;
-if (! is_dir($lang_base_dir)) {
-    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODLANGINCOMPATIBLE) ;
+$lang_base_dir = $base_dir.'/'.$target_lang;
+if (!is_dir($lang_base_dir)) {
+    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODLANGINCOMPATIBLE);
 }
 
 // make list of files and check $target_file
-$lang_files = array() ;
-$dh = opendir($lang_base_dir) ;
+$lang_files = array();
+$dh = opendir($lang_base_dir);
 if ($dh) {
     while ($file = readdir($dh)) {
         if (substr($file, 0, 1) == '.') {
-            continue ;
+            continue;
         }
         if ($file == 'index.html') {
-            continue ;
+            continue;
         }
     //if( $file == 'modinfo.php' ) continue ; // TODO(?)
     //if( $file == 'global.php' ) continue ; // TODO(?)
     if (is_file("$lang_base_dir/$file")) {
-        $lang_files[] = $file ;
+        $lang_files[] = $file;
     }
     }
 }
-closedir($dh) ;
+closedir($dh);
 if (empty($lang_files)) {
-    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODEMPTYLANGDIR) ;
+    altsys_mylangadmin_errordie($target_mname, _MYLANGADMIN_ERR_MODEMPTYLANGDIR);
 }
-if (! in_array($target_file, $lang_files)) {
-    $target_file = $lang_files[0] ;
+if (!in_array($target_file, $lang_files)) {
+    $target_file = $lang_files[0];
 }
 
 // get unique path of language_file
-$langfile_unique_path = "$lang_base_dir/$target_file" ;
+$langfile_unique_path = "$lang_base_dir/$target_file";
 
 // get constants defined by the target_file
-list($langfile_names, $constpref, $already_read) = altsys_mylangadmin_get_constant_names($langfile_unique_path, $target_dirname) ;
+list($langfile_names, $constpref, $already_read) = altsys_mylangadmin_get_constant_names($langfile_unique_path, $target_dirname);
 
 // get user_values should be overridden
-$langfile_constants = array() ;
+$langfile_constants = array();
 foreach ($langfile_names as $name) {
-    list($value) = $db->fetchRow($db->query('SELECT value FROM ' . $db->prefix('altsys_language_constants') . " WHERE mid=$target_mid AND language='$target_lang4sql' AND name='" . addslashes($name) . "'")) ;
-    $langfile_constants[ $name ] = $value ;
+    list($value) = $db->fetchRow($db->query('SELECT value FROM '.$db->prefix('altsys_language_constants')." WHERE mid=$target_mid AND language='$target_lang4sql' AND name='".addslashes($name)."'"));
+    $langfile_constants[$name] = $value;
 }
 
 // constants defined in XOOPS_ROOT_PATH/my_language/(dirname)/...
 if ($langman->my_language) {
-    $mylang_unique_path = $langman->my_language.'/modules/'.$target_dirname.'/'.$target_lang.'/'.$target_file ;
-    $mylang_constants = array_map('htmlspecialchars', altsys_mylangadmin_get_constants_by_pcre($mylang_unique_path)) ;
+    $mylang_unique_path = $langman->my_language.'/modules/'.$target_dirname.'/'.$target_lang.'/'.$target_file;
+    $mylang_constants = array_map('htmlspecialchars', altsys_mylangadmin_get_constants_by_pcre($mylang_unique_path));
     foreach ($mylang_constants as $key => $val) {
-        if (! in_array($key, array_keys($langfile_constants))) {
-            $langfile_constants[ $key ] = null ;
-            define($key, _MYLANGADMIN_NOTE_ADDEDBYMYLANG) ;
+        if (!in_array($key, array_keys($langfile_constants))) {
+            $langfile_constants[$key] = null;
+            define($key, _MYLANGADMIN_NOTE_ADDEDBYMYLANG);
         }
     }
 } else {
-    $mylang_unique_path = '' ;
-    $mylang_constants = array() ;
+    $mylang_unique_path = '';
+    $mylang_constants = array();
 }
 
 
@@ -177,57 +177,57 @@ 
 //
 
 // Update language table and cache file
-if (! empty($_POST['do_update'])) {
+if (!empty($_POST['do_update'])) {
     // Ticket Check
-    if (! $xoopsGTicket->check(true, 'altsys')) {
+    if (!$xoopsGTicket->check(true, 'altsys')) {
         redirect_header(XOOPS_URL.'/', 3, $xoopsGTicket->getErrors());
     }
 
     // read original file
-    $file_contents = file_get_contents($langfile_unique_path) ;
+    $file_contents = file_get_contents($langfile_unique_path);
 
     // insert fingerprint of langfile_unique_path
-    $langfile_fingerprint = '_MYLANGADMIN_'.md5($langfile_unique_path) ;
-    $file_contents = str_replace('<?php', "<?php\nif(!defined('$langfile_fingerprint'))define('$langfile_fingerprint',1);", $file_contents) ;
+    $langfile_fingerprint = '_MYLANGADMIN_'.md5($langfile_unique_path);
+    $file_contents = str_replace('<?php', "<?php\nif(!defined('$langfile_fingerprint'))define('$langfile_fingerprint',1);", $file_contents);
 
     // constants loop
-    $overrides_counter = 0 ;
+    $overrides_counter = 0;
     foreach (array_reverse($langfile_names) as $name) {
-        $user_value = $myts->stripSlashesGPC(@$_POST[$name]) ;
-        $db->query('DELETE FROM ' . $db->prefix('altsys_language_constants') . " WHERE mid=$target_mid AND language='$target_lang4sql' AND name='" . addslashes($name) . "'") ;
+        $user_value = $myts->stripSlashesGPC(@$_POST[$name]);
+        $db->query('DELETE FROM '.$db->prefix('altsys_language_constants')." WHERE mid=$target_mid AND language='$target_lang4sql' AND name='".addslashes($name)."'");
         if ($user_value !== '') {
-            $overrides_counter ++ ;
+            $overrides_counter++;
             // Update table
-            $db->query('INSERT INTO ' . $db->prefix('altsys_language_constants') . " (mid,language,name,value) VALUES ($target_mid,'$target_lang4sql','" . addslashes($name) . "','" . addslashes($user_value) . "')") ;
+            $db->query('INSERT INTO '.$db->prefix('altsys_language_constants')." (mid,language,name,value) VALUES ($target_mid,'$target_lang4sql','".addslashes($name)."','".addslashes($user_value)."')");
             // rewrite script for cache
             // comment-out the line of define()
             if (empty($constpref)) {
-                $from = '/.*define\s?\(\s*(["\'])'.preg_quote($name).'(\\1).*\;.*/' ;
+                $from = '/.*define\s?\(\s*(["\'])'.preg_quote($name).'(\\1).*\;.*/';
             } else {
-                $from = '/.*define\s?\(\s*\$constpref\s*\.\s*(["\'])'.preg_quote(substr($name, strlen($constpref))).'(\\1).*\;.*/' ;
+                $from = '/.*define\s?\(\s*\$constpref\s*\.\s*(["\'])'.preg_quote(substr($name, strlen($constpref))).'(\\1).*\;.*/';
             }
-            $to = '//$0'."\ndefine('".addslashes($name)."','".addslashes($user_value)."');" ;
-            $file_contents = preg_replace($from, $to, $file_contents) ;
+            $to = '//$0'."\ndefine('".addslashes($name)."','".addslashes($user_value)."');";
+            $file_contents = preg_replace($from, $to, $file_contents);
         }
     }
 
     // get the file name for caching
-    $cache_file_name = $langman->getCacheFileName($target_file, $target_dirname, $target_lang) ;
+    $cache_file_name = $langman->getCacheFileName($target_file, $target_dirname, $target_lang);
 
     // Create language cache file
     if ($overrides_counter > 0) {
-        $fp = fopen($cache_file_name, 'wb') ;
-        if (! $fp) {
-            die('Invalid Cache Directory. (Set XOOPS_TRUST_PATH/cache writable)') ;
+        $fp = fopen($cache_file_name, 'wb');
+        if (!$fp) {
+            die('Invalid Cache Directory. (Set XOOPS_TRUST_PATH/cache writable)');
         }
-        fwrite($fp, $file_contents) ;
-        fclose($fp) ;
+        fwrite($fp, $file_contents);
+        fclose($fp);
     } else {
-        unlink($cache_file_name) ;
+        unlink($cache_file_name);
     }
 
-    redirect_header('?mode=admin&lib=altsys&page=mylangadmin&dirname='.$target_dirname.'&target_lang='.rawurlencode($target_lang).'&target_file='.rawurlencode($target_file), 1, _MYLANGADMIN_CACHEUPDATED) ;
-    exit ;
+    redirect_header('?mode=admin&lib=altsys&page=mylangadmin&dirname='.$target_dirname.'&target_lang='.rawurlencode($target_lang).'&target_file='.rawurlencode($target_file), 1, _MYLANGADMIN_CACHEUPDATED);
+    exit;
 }
 
 
@@ -236,22 +236,22 @@ 
 //
 
 // check cache file
-$cache_file_name = $langman->getCacheFileName($target_file, $target_dirname, $target_lang) ;
-$cache_file_mtime = file_exists($cache_file_name) ? filemtime($cache_file_name) : 0 ;
+$cache_file_name = $langman->getCacheFileName($target_file, $target_dirname, $target_lang);
+$cache_file_mtime = file_exists($cache_file_name) ? filemtime($cache_file_name) : 0;
 
 // check core version and generate message to enable D3LanguageManager
 if (altsys_get_core_type() == ALTSYS_CORE_TYPE_XCL21) {
     // XoopsCube Legacy without preload
     if (class_exists('AltsysLangMgr_LanguageManager')) {
         // the preload enabled
-        $notice4disp = _MYLANGADMIN_MSG_D3LANGMANENABLED ;
+        $notice4disp = _MYLANGADMIN_MSG_D3LANGMANENABLED;
     } else {
         // the preload disabled
-        $notice4disp = sprintf(_MYLANGADMIN_FMT_HOWTOENABLED3LANGMAN4XCL, 'SetupAltsysLangMgr.class.php', 'XOOPS_ROOT_PATH/preload') ;
+        $notice4disp = sprintf(_MYLANGADMIN_FMT_HOWTOENABLED3LANGMAN4XCL, 'SetupAltsysLangMgr.class.php', 'XOOPS_ROOT_PATH/preload');
     }
 } else {
     // X2 core etc.
-    $notice4disp = _MYLANGADMIN_MSG_HOWTOENABLED3LANGMAN4X2.'<br />' ;
+    $notice4disp = _MYLANGADMIN_MSG_HOWTOENABLED3LANGMAN4X2.'<br />';
     $notice4disp .= '
 		<h4>include/common.php</h4>
 		<pre>
@@ -275,20 +275,20 @@ 
 //
 
 
-xoops_cp_header() ;
+xoops_cp_header();
 
 // mymenu
-altsys_include_mymenu() ;
+altsys_include_mymenu();
 
 // breadcrumbs
-$breadcrumbsObj =& AltsysBreadcrumbs::getInstance() ;
+$breadcrumbsObj = & AltsysBreadcrumbs::getInstance();
 if ($breadcrumbsObj->hasPaths()) {
-    $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mylangadmin', _MI_ALTSYS_MENU_MYLANGADMIN) ;
-    $breadcrumbsObj->appendPath('', $target_mname) ;
+    $breadcrumbsObj->appendPath(XOOPS_URL.'/modules/altsys/admin/index.php?mode=admin&amp;lib=altsys&amp;page=mylangadmin', _MI_ALTSYS_MENU_MYLANGADMIN);
+    $breadcrumbsObj->appendPath('', $target_mname);
 }
 
-require_once XOOPS_TRUST_PATH.'/libs/altsys/class/D3Tpl.class.php' ;
-$tpl = new D3Tpl() ;
+require_once XOOPS_TRUST_PATH.'/libs/altsys/class/D3Tpl.class.php';
+$tpl = new D3Tpl();
 $tpl->assign(array(
     'target_dirname' => $target_dirname,
     'target_mname' => $target_mname,
@@ -307,8 +307,8 @@ 
     'notice' => $notice4disp,
     'already_read' => $already_read,
     'gticket_hidden' => $xoopsGTicket->getTicketHtml(__LINE__, 1800, 'altsys')
-)) ;
-$tpl->display('db:altsys_main_mylangadmin.html') ;
+));
+$tpl->display('db:altsys_main_mylangadmin.html');
 
-xoops_cp_footer() ;
-exit ;
+xoops_cp_footer();
+exit;

xoops_trust_path/libs/altsys/index.php

Spacing +23 added lines, -23 removed lines

@@ -1,51 +1,51 @@
 <?php
 
-if (! defined('XOOPS_MODULE_PATH')) {
-    define('XOOPS_MODULE_PATH', XOOPS_ROOT_PATH . '/modules');
+if (!defined('XOOPS_MODULE_PATH')) {
+    define('XOOPS_MODULE_PATH', XOOPS_ROOT_PATH.'/modules');
 }
-if (! defined('XOOPS_MODULE_URL')) {
-    define('XOOPS_MODULE_URL', XOOPS_URL . '/modules');
+if (!defined('XOOPS_MODULE_URL')) {
+    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)) {
-    $moduleperm_handler =& xoops_gethandler('module') ;
-    $xoopsModule =& $moduleperm_handler->getByDirname('altsys') ;
+    $moduleperm_handler = & xoops_gethandler('module');
+    $xoopsModule = & $moduleperm_handler->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 = & AltsysBreadcrumbs::getInstance();
+$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' ;
-if (! in_array($page, $controllers)) {
-    $_GET['page'] = $page = 'myblocksadmin' ;
-    $_SERVER['REQUEST_URI'] = '/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin' ;
+$page = preg_replace('/[^a-zA-Z0-9_-]/', '', @$_GET['page']);
+require __DIR__.'/controllers.php';
+if (!in_array($page, $controllers)) {
+    $_GET['page'] = $page = 'myblocksadmin';
+    $_SERVER['REQUEST_URI'] = '/admin/index.php?mode=admin&lib=altsys&page=myblocksadmin';
 }
 
 
 // half measure ... (TODO)
 if (empty($_GET['dirname'])) {
-    $module_handler =& xoops_gethandler('module') ;
-    list($top_module) = $module_handler->getObjects(new Criteria('isactive', 1)) ;
-    $_GET['dirname'] = $top_module->getVar('dirname') ;
+    $module_handler = & xoops_gethandler('module');
+    list($top_module) = $module_handler->getObjects(new Criteria('isactive', 1));
+    $_GET['dirname'] = $top_module->getVar('dirname');
 }
 
 // language file
-altsys_include_language_file($page) ;
+altsys_include_language_file($page);
 
 // branch to each pages
-$mytrustdirpath = __DIR__ ;
+$mytrustdirpath = __DIR__;
 if (file_exists(XOOPS_TRUST_PATH.'/libs/altsys/'.$page.'.php')) {
-    include XOOPS_TRUST_PATH.'/libs/altsys/'.$page.'.php' ;
+    include XOOPS_TRUST_PATH.'/libs/altsys/'.$page.'.php';
 } else {
-    die('wrong request') ;
+    die('wrong request');
 }

xoops_trust_path/libs/altsys/include/formcheckboxgroup.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
 /**
  * Parent
  */
-include_once XOOPS_ROOT_PATH . '/class/xoopsform/formcheckbox.php';
+include_once XOOPS_ROOT_PATH.'/class/xoopsform/formcheckbox.php';
 
 /**
  * A checkbox field with a choice of available groups
@@ -65,17 +65,17 @@ 
      * @param   bool    $include_anon   Include group "anonymous"?
      * @param   mixed   $value          Pre-selected value (or array of them).
      */
-    public function __construct($caption, $name, $include_anon=false, $value=null)
+    public function __construct($caption, $name, $include_anon = false, $value = null)
     {
         parent::__construct($caption, $name, $value);
-        $member_handler =& xoops_gethandler('member');
+        $member_handler = & xoops_gethandler('member');
         if (!$include_anon) {
             $options = $member_handler->getGroupList(new Criteria('groupid', XOOPS_GROUP_ANONYMOUS, '!='));
         } else {
             $options = $member_handler->getGroupList();
         }
         foreach ($options as $k => $v) {
-            $options[$k] = $v . '<br />';
+            $options[$k] = $v.'<br />';
         }
         $this->addOptionArray($options);
     }