From 4675b2fc5f502f74adf73301ad8df62b8914df33 Mon Sep 17 00:00:00 2001 From: REJack Date: Wed, 13 Apr 2016 22:48:37 +0200 Subject: [PATCH] Sub-Groups added - added function add_subgroup($group_par, $subgroup_par) - added function remove_subgroup($group_par, $subgroup_par) - added function get_subgroups($group_par) - modified is_group_allowed() to check subgroups - modified delete_group() to remove subgroups --- application/config/aauth.php | 1 + application/language/english/aauth_lang.php | 2 + application/libraries/Aauth.php | 101 ++++++++++++++++++-- sql/Aauth_v2.sql | 15 +++ 4 files changed, 112 insertions(+), 7 deletions(-) diff --git a/application/config/aauth.php b/application/config/aauth.php index ef1cb27..523f026 100644 --- a/application/config/aauth.php +++ b/application/config/aauth.php @@ -81,6 +81,7 @@ $config_aauth["default"] = array( 'users' => 'aauth_users', 'groups' => 'aauth_groups', + 'group_to_group' => 'aauth_group_to_group', 'user_to_group' => 'aauth_user_to_group', 'perms' => 'aauth_perms', 'perm_to_group' => 'aauth_perm_to_group', diff --git a/application/language/english/aauth_lang.php b/application/language/english/aauth_lang.php index ced329e..6086fc9 100644 --- a/application/language/english/aauth_lang.php +++ b/application/language/english/aauth_lang.php @@ -46,11 +46,13 @@ $lang['aauth_error_recaptcha_not_correct'] = 'Sorry, the reCAPTCHA text entered $lang['aauth_error_no_user'] = 'User does not exist'; $lang['aauth_error_account_not_verified'] = 'Your account has not been verified. Please check your e-mail and verify your account.'; $lang['aauth_error_no_group'] = 'Group does not exist'; +$lang['aauth_error_no_subgroup'] = 'Subgroup does not exist'; $lang['aauth_error_self_pm'] = 'It is not possible to send a Message to yourself.'; $lang['aauth_error_no_pm'] = 'Private Message not found'; /* Info messages */ $lang['aauth_info_already_member'] = 'User is already member of group'; +$lang['aauth_info_already_subgroup'] = 'Subgroup is already member of group'; $lang['aauth_info_group_exists'] = 'Group name already exists'; $lang['aauth_info_perm_exists'] = 'Permission name already exists'; diff --git a/application/libraries/Aauth.php b/application/libraries/Aauth.php index 1c56944..8223964 100644 --- a/application/libraries/Aauth.php +++ b/application/libraries/Aauth.php @@ -13,7 +13,7 @@ * * @copyright 2014-2015 Emre Akay * - * @version 2.4.6 + * @version 2.4.7 * * @license LGPL * @license http://opensource.org/licenses/LGPL-3.0 Lesser GNU Public License @@ -1237,6 +1237,12 @@ class Aauth { $this->aauth_db->where('group_id', $group_id); $this->aauth_db->delete($this->config_vars['perm_to_group']); + + $this->aauth_db->where('group_id', $group_id); + $this->aauth_db->delete($this->config_vars['group_to_group']); + + $this->aauth_db->where('subgroup_id', $group_id); + $this->aauth_db->delete($this->config_vars['group_to_group']); $this->aauth_db->where('id', $group_id); return $this->aauth_db->delete($this->config_vars['groups']); @@ -1292,6 +1298,60 @@ class Aauth { return $this->aauth_db->delete($this->config_vars['user_to_group']); } + /** + * Add subgroup + * Add a subgroup to a group + * @param int $user_id User id to add to group + * @param int|string $group_par Group id or name to add user to + * @return bool Add success/failure + */ + public function add_subgroup($group_par, $subgroup_par) { + + $group_id = $this->get_group_id($group_par); + $subgroup_id = $this->get_group_id($subgroup_par); + + if( ! $group_id ) { + $this->error( $this->CI->lang->line('aauth_error_no_group') ); + return FALSE; + } + + if( ! $subgroup_id ) { + $this->error( $this->CI->lang->line('aauth_error_no_subgroup') ); + return FALSE; + } + + $query = $this->aauth_db->where('group_id',$group_id); + $query = $this->aauth_db->where('subgroup_id',$subgroup_id); + $query = $this->aauth_db->get($this->config_vars['group_to_group']); + + if ($query->num_rows() < 1) { + $data = array( + 'group_id' => $group_id, + 'subgroup_id' => $subgroup_id, + ); + + return $this->aauth_db->insert($this->config_vars['group_to_group'], $data); + } + $this->info($this->CI->lang->line('aauth_info_already_subgroup')); + return TRUE; + } + + /** + * Remove subgroup + * Remove a subgroup from a group + * @param int|string $group_par Group id or name to remove + * @param int|string $subgroup_par Sub-Group id or name to remove + * @return bool Remove success/failure + */ + public function remove_subgroup($group_par, $subgroup_par) { + + $group_par = $this->get_group_id($group_par); + $subgroup_par = $this->get_group_id($subgroup_par); + $this->aauth_db->where('group_id', $group_par); + $this->aauth_db->where('subgroup_id', $subgroup_par); + return $this->aauth_db->delete($this->config_vars['group_to_group']); + } + //tested /** * Remove member @@ -1399,6 +1459,26 @@ class Aauth { return $row->id; } + /** + * Get subgroups + * Get subgroups from group name or id ( ! Case sensitive) + * @param int|string $group_par Group id or name to get + * @return object Array of subgroup_id's + */ + public function get_subgroups ( $group_par ) { + + $group_id = $this->get_group_id($group_par); + + $query = $this->aauth_db->where('group_id', $group_id); + $query = $this->aauth_db->select('subgroup_id'); + $query = $this->aauth_db->get($this->config_vars['group_to_group']); + + if ($query->num_rows() == 0) + return FALSE; + + return $query->result(); + } + ######################## # Permission Functions ######################## @@ -1531,17 +1611,25 @@ class Aauth { // if group par is given if($group_par != FALSE){ + $subgroup_ids = $this->get_subgroups($group_par); $group_par = $this->get_group_id($group_par); - $query = $this->aauth_db->where('perm_id', $perm_id); $query = $this->aauth_db->where('group_id', $group_par); $query = $this->aauth_db->get( $this->config_vars['perm_to_group'] ); + + $g_allowed=FALSE; + if(is_array($subgroup_ids)){ + foreach ($subgroup_ids as $g ){ + if($this->is_group_allowed($perm_id, $g->subgroup_id)){ + $g_allowed=TRUE; + } + } + } if( $query->num_rows() > 0){ - return TRUE; - } else { - return FALSE; + $g_allowed=TRUE; } + return $g_allowed; } // if group par is not given // checks current user's all groups @@ -1555,9 +1643,8 @@ class Aauth { if (!$this->is_loggedin()){return FALSE;} $group_pars = $this->get_user_groups(); - foreach ($group_pars as $g ){ - if($this->is_group_allowed($perm_id, $g -> id)){ + if($this->is_group_allowed($perm_id, $g->id)){ return TRUE; } } diff --git a/sql/Aauth_v2.sql b/sql/Aauth_v2.sql index e5f20d3..4a260cd 100644 --- a/sql/Aauth_v2.sql +++ b/sql/Aauth_v2.sql @@ -160,3 +160,18 @@ CREATE TABLE `aauth_user_variables` ( -- ---------------------------- -- Records of aauth_user_variables -- ---------------------------- + +-- ---------------------------- +-- Table structure for `aauth_perm_to_group` +-- ---------------------------- +DROP TABLE IF EXISTS `aauth_group_to_group`; +CREATE TABLE `aauth_group_to_group` ( + `group_id` int(11) unsigned DEFAULT NULL, + `subgroup_id` int(11) unsigned DEFAULT NULL, + PRIMARY KEY (`group_id`,`subgroup_id`) +) ENGINE=InnoDB DEFAULT CHARSET=utf8; + +-- ---------------------------- +-- Records of aauth_perm_to_group +-- ---------------------------- +