XoopsModules25x / groupmanager

admin/admin_footer.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
     ."  <div style='text-align: center;'>\n"
     ."    <a href='http://www.xoops.org' target='_blank'><img src='{$pathIcon32}/xoopsmicrobutton.gif' alt='XOOPS' title='XOOPS'></a>\n"
     ."  </div>\n"
-    ."  " . _AM_MODULEADMIN_ADMIN_FOOTER . "\n"
+    ."  "._AM_MODULEADMIN_ADMIN_FOOTER."\n"
     ."</div>";
 
 xoops_cp_footer();
\ No newline at end of file

admin/about.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 //  ------------------------------------------------------------------------ //
 //
 
-include_once dirname(__FILE__) . '/admin_header.php';
+include_once dirname(__FILE__).'/admin_header.php';
 
 xoops_cp_header();
 

admin/menu.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
         $groups = XOOPS_GROUP_ANONYMOUS;
 }
 $module_id = $xoopsModule->getVar('mid');
-$gperm_handler =& xoops_gethandler('groupperm');
+$gperm_handler = & xoops_gethandler('groupperm');
 
 if ($gperm_handler->checkRight($perm_name, $perm_itemid, $groups, $module_id)) {
         // allowed, so display contents within the category
@@ -49,5 +49,5 @@ 
 
 $i++;
 $adminmenu[$i]['title'] = _MI_GROUPS_ABOUT;
-$adminmenu[$i]['link'] =  "admin/about.php";
+$adminmenu[$i]['link'] = "admin/about.php";
 $adminmenu[$i]["icon"] = $pathIcon32.'/about.png';
\ No newline at end of file

admin/index.php

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 
 xoops_cp_header();
 
-	$indexAdmin = new ModuleAdmin();
+    $indexAdmin = new ModuleAdmin();
 
     echo $indexAdmin->addNavigation('index.php');
     echo $indexAdmin->renderIndex();

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-require_once dirname(dirname(dirname(dirname(__FILE__)))) . '/include/cp_header.php';
-include_once dirname(__FILE__) . '/admin_header.php';
+require_once dirname(dirname(dirname(dirname(__FILE__)))).'/include/cp_header.php';
+include_once dirname(__FILE__).'/admin_header.php';
 
 xoops_cp_header();
 

admin/main.php

Spacing +26 added lines, -26 removed lines

@@ -30,24 +30,24 @@ 
 include_once('admin_header.php');
 
 
-require_once XOOPS_ROOT_PATH . '/class/template.php';
-if (!isset($xoopsTpl)) {$xoopsTpl = new XoopsTpl();}
+require_once XOOPS_ROOT_PATH.'/class/template.php';
+if (!isset($xoopsTpl)) {$xoopsTpl = new XoopsTpl(); }
 //$xoopsTpl->xoops_setCaching(0);
-$xoopsTpl->caching=0;
+$xoopsTpl->caching = 0;
 $xoopsTpl->assign('xoops_dirname', $xoopsModule->getVar('dirname'));
 
 
 // CHECK IF SUBMIT WAS PRESSED
-if (isset($_POST['add_x']) or isset($_POST['del_x'])){
+if (isset($_POST['add_x']) or isset($_POST['del_x'])) {
 
-if (isset($_POST['add_x'])){
-$hMember =& xoops_gethandler('member');
-$membership =& $hMember->addUserToGroup($_POST['groupid'],$_POST['all']);
+if (isset($_POST['add_x'])) {
+$hMember = & xoops_gethandler('member');
+$membership = & $hMember->addUserToGroup($_POST['groupid'], $_POST['all']);
 }
 
-if (isset($_POST['del_x'])){
-$hMember =& xoops_gethandler('member');
-$membership =& $hMember->removeUsersFromGroup($_POST['groupid'],array($_POST['curr']));
+if (isset($_POST['del_x'])) {
+$hMember = & xoops_gethandler('member');
+$membership = & $hMember->removeUsersFromGroup($_POST['groupid'], array($_POST['curr']));
 }
 }
 
@@ -80,16 +80,16 @@ 
 //---------------------------------------
 */
 
-$grpInfo=array();
+$grpInfo = array();
 
-$member_handler =& xoops_gethandler('member');
-$groups =& $member_handler->getGroups();
+$member_handler = & xoops_gethandler('member');
+$groups = & $member_handler->getGroups();
 
 $count = count($groups);
 for ($i = 0; $i < $count; $i++) {
 
 $id = $groups[$i]->getVar('groupid');
-$name=$groups[$i]->getVar('name');
+$name = $groups[$i]->getVar('name');
 //Skip anonymous group
 if ($id==3) continue;
 
@@ -103,7 +103,7 @@ 
         $groups2 = XOOPS_GROUP_ANONYMOUS;
 }
 $module_id = $xoopsModule->getVar('mid');
-$gperm_handler =& xoops_gethandler('groupperm');
+$gperm_handler = & xoops_gethandler('groupperm');
 if ($gperm_handler->checkRight($perm_name, $perm_itemid, $groups2, $module_id)) {
 } else {
         continue;
@@ -111,27 +111,27 @@ 
 //----------------*/
 
 
-$uids =& $member_handler->getUsersByGroup($id);
+$uids = & $member_handler->getUsersByGroup($id);
 $criteria = new Criteria('uid', "(".implode(',', $uids).")", "IN");
 $criteria->setSort('uname');
-$users=$member_handler->getUserList($criteria);
+$users = $member_handler->getUserList($criteria);
 
-$grpInfo[$i]['users']=$users;
-$grpInfo[$i]['name']=$name;
-$grpInfo[$i]['id']=$id;
+$grpInfo[$i]['users'] = $users;
+$grpInfo[$i]['name'] = $name;
+$grpInfo[$i]['id'] = $id;
 }
 
 /*---------------------------//
 Get all users
 //----------------------------------*/
-$allUsr=array();
-$member_handler =& xoops_gethandler('member');
-$foundusers =& $member_handler->getUsers();
+$allUsr = array();
+$member_handler = & xoops_gethandler('member');
+$foundusers = & $member_handler->getUsers();
 foreach (array_keys($foundusers) as $j) {
 //echo $foundusers[$j]->getVar("uname").'<br>';
-$uid=$foundusers[$j]->getVar("uid");
-$username=$foundusers[$j]->getVar("uname");
-$allUsr[$uid]=$username;
+$uid = $foundusers[$j]->getVar("uid");
+$username = $foundusers[$j]->getVar("uname");
+$allUsr[$uid] = $username;
 }
 //--------------------------//
 

Braces +3 added lines, -1 removed lines

@@ -91,7 +91,9 @@
 $id = $groups[$i]->getVar('groupid');
 $name=$groups[$i]->getVar('name');
 //Skip anonymous group
-if ($id==3) continue;
+if ($id==3) {
+    continue;
+}
 
 
 //check if user has permission to change this group

admin/admin_header.php

Spacing +5 added lines, -5 removed lines

@@ -1,9 +1,9 @@ 
 <?php
 
 $path = dirname(dirname(dirname(dirname(__FILE__))));
-include_once $path . '/mainfile.php';
-include_once $path . '/include/cp_functions.php';
-require_once $path . '/include/cp_header.php';
+include_once $path.'/mainfile.php';
+include_once $path.'/include/cp_functions.php';
+require_once $path.'/include/cp_header.php';
 
 global $xoopsModule;
 
@@ -18,8 +18,8 @@ 
 $pathIcon32 = '../'.$xoopsModule->getInfo('icons32');
 $pathModuleAdmin = $xoopsModule->getInfo('dirmoduleadmin');
 
-if ( file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))){
+if (file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))) {
         include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
-    }else{
+    } else {
         redirect_header("../../../admin.php", 5, _AM_GROUPS_MODULEADMIN_MISSING, false);
     }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -20,6 +20,6 @@
 
 if ( file_exists($GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php'))){
         include_once $GLOBALS['xoops']->path($pathModuleAdmin.'/moduleadmin.php');
-    }else{
+    } else{
         redirect_header("../../../admin.php", 5, _AM_GROUPS_MODULEADMIN_MISSING, false);
     }
\ No newline at end of file

admin/perm.php

Indentation +16 added lines, -16 removed lines

@@ -9,25 +9,25 @@ 
 
 xoops_cp_header();
 
-   include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
+    include_once XOOPS_ROOT_PATH.'/class/xoopsform/grouppermform.php';
     $module_id = $xoopsModule->getVar('mid');
 
 
-	//$item_list = array('1' => 'Category 1', '2' => 'Category 2', '3' => 'Category 3');
-	$item_list=array();
-	$member_handler =& xoops_gethandler('member');
-	$groups =& $member_handler->getGroups();
-	foreach ($groups as $grp){
-	$item_list[$grp->getVar('groupid')]=$grp->getVar('name');
-	}
+    //$item_list = array('1' => 'Category 1', '2' => 'Category 2', '3' => 'Category 3');
+    $item_list=array();
+    $member_handler =& xoops_gethandler('member');
+    $groups =& $member_handler->getGroups();
+    foreach ($groups as $grp){
+    $item_list[$grp->getVar('groupid')]=$grp->getVar('name');
+    }
 	
     //--------------------------------------------------------------//
     //Added constants for use in translation - by GibaPhp           //
     //--------------------------------------------------------------//
 
-	$title_of_form = _AM_GROUPS_PERMISS_GRP_TITLE;
-	$perm_name = 'groupper';
-	$perm_desc = _AM_GROUPS_PERMISS_GRP_DESC;
+    $title_of_form = _AM_GROUPS_PERMISS_GRP_TITLE;
+    $perm_name = 'groupper';
+    $perm_desc = _AM_GROUPS_PERMISS_GRP_DESC;
 
     $form = new XoopsGroupPermForm($title_of_form, $module_id, $perm_name, $perm_desc);
     foreach ($item_list as $item_id => $item_name) {
@@ -35,12 +35,12 @@ 
     }
     echo $form->render();
 	
-	echo "<br /><br />";
+    echo "<br /><br />";
 	
-	$item_list = array('1' => _AM_GROUPS_ITEM_LIST);
-	$title_of_form = _AM_GROUPS_TITLE_OF_FORM;
-	$perm_desc = _AM_GROUPS_PERM_DESC;
-	$perm_name = 'allowedgrp';
+    $item_list = array('1' => _AM_GROUPS_ITEM_LIST);
+    $title_of_form = _AM_GROUPS_TITLE_OF_FORM;
+    $perm_desc = _AM_GROUPS_PERM_DESC;
+    $perm_name = 'allowedgrp';
     $form = new XoopsGroupPermForm($title_of_form, $module_id, $perm_name, $perm_desc);
     foreach ($item_list as $item_id => $item_name) {
     $form->addItem($item_id, $item_name);

Spacing +8 added lines, -8 removed lines

@@ -1,10 +1,10 @@ 
 <?php
 require_once '../../../include/cp_header.php';
 
-require_once XOOPS_ROOT_PATH . '/class/template.php';
-if (!isset($xoopsTpl)) {$xoopsTpl = new XoopsTpl();}
+require_once XOOPS_ROOT_PATH.'/class/template.php';
+if (!isset($xoopsTpl)) {$xoopsTpl = new XoopsTpl(); }
 //$xoopsTpl->xoops_setCaching(0);
-$xoopsTpl->caching=0;
+$xoopsTpl->caching = 0;
 $xoopsTpl->assign('xoops_dirname', $xoopsModule->getVar('dirname'));
 
 xoops_cp_header();
@@ -14,11 +14,11 @@ 
 
 
 	//$item_list = array('1' => 'Category 1', '2' => 'Category 2', '3' => 'Category 3');
-	$item_list=array();
-	$member_handler =& xoops_gethandler('member');
-	$groups =& $member_handler->getGroups();
-	foreach ($groups as $grp){
-	$item_list[$grp->getVar('groupid')]=$grp->getVar('name');
+	$item_list = array();
+	$member_handler = & xoops_gethandler('member');
+	$groups = & $member_handler->getGroups();
+	foreach ($groups as $grp) {
+	$item_list[$grp->getVar('groupid')] = $grp->getVar('name');
 	}
 	
     //--------------------------------------------------------------//

xoops_version.php

Spacing +4 added lines, -4 removed lines

@@ -28,10 +28,10 @@
 $modversion["module_status"] = "RC";
 $modversion["author_website_url"] = "http://www.xoops.org/";
 $modversion["author_website_name"] = "XOOPS";
-$modversion['min_php']='5.2';
-$modversion['min_xoops']='2.5';
-$modversion['min_admin']='1.1';
-$modversion['min_db']= array('mysql'=>'5.0.7', 'mysqli'=>'5.0.7');
+$modversion['min_php'] = '5.2';
+$modversion['min_xoops'] = '2.5';
+$modversion['min_admin'] = '1.1';
+$modversion['min_db'] = array('mysql'=>'5.0.7', 'mysqli'=>'5.0.7');
 
 // Sql file (must contain sql generated by phpMyAdmin or phpPgAdmin)
 // All tables should not have any prefix!

language/english/admin.php

Spacing +12 added lines, -12 removed lines

@@ -26,19 +26,19 @@
 //  ------------------------------------------------------------------------ //
 //
 
-define("_AM_GROUPS_PERMISS_GRP_TITLE","Permission for each Group");
-define("_AM_GROUPS_PERMISS_GRP_DESC","Set which groups each group can change");
-define("_AM_GROUPS_ITEM_LIST","Can Change Permissions");
-define("_AM_GROUPS_TITLE_OF_FORM","Groups allowed to change permissions");
-define("_AM_GROUPS_PERM_DESC","Select which groups can change these permissions");
+define("_AM_GROUPS_PERMISS_GRP_TITLE", "Permission for each Group");
+define("_AM_GROUPS_PERMISS_GRP_DESC", "Set which groups each group can change");
+define("_AM_GROUPS_ITEM_LIST", "Can Change Permissions");
+define("_AM_GROUPS_TITLE_OF_FORM", "Groups allowed to change permissions");
+define("_AM_GROUPS_PERM_DESC", "Select which groups can change these permissions");
 
-define("_AM_GROUPS_NAME_GROUP","Group Name");
-define("_AM_GROUPS_MEMBERS_ACTUALS","Current Members");
-define("_AM_GROUPS_ACTIONS","Action");
-define("_AM_GROUPS_ALL_USERS","All Users");
-define("_AM_GROUPS_MAIN","Main");
-define("_AM_GROUPS_PERMISSIONS","Permissions");
+define("_AM_GROUPS_NAME_GROUP", "Group Name");
+define("_AM_GROUPS_MEMBERS_ACTUALS", "Current Members");
+define("_AM_GROUPS_ACTIONS", "Action");
+define("_AM_GROUPS_ALL_USERS", "All Users");
+define("_AM_GROUPS_MAIN", "Main");
+define("_AM_GROUPS_PERMISSIONS", "Permissions");
 
 
 //ModuleAdmin
-define('_AM_GROUPS_MODULEADMIN_MISSING','Error: The ModuleAdmin class is missing. Please install the ModuleAdmin Class into /Frameworks (see /docs/readme.txt)');
\ No newline at end of file
+define('_AM_GROUPS_MODULEADMIN_MISSING', 'Error: The ModuleAdmin class is missing. Please install the ModuleAdmin Class into /Frameworks (see /docs/readme.txt)');
\ No newline at end of file