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
emreakay · Apr 13, 2016 · 4675b2f · 4675b2f
1 parent c221014
commit 4675b2f
Show file tree

Hide file tree

Showing 4 changed files with 112 additions and 7 deletions.
diff --git a/application/config/aauth.php b/application/config/aauth.php
@@ -81,6 +81,7 @@
 
 	'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
@@ -46,11 +46,13 @@
 $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
@@ -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 @@ public function delete_group($group_par) {
 
 		$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 @@ public function remove_member($user_id, $group_par) {
 		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 @@ public function get_group_id ( $group_par ) {
 		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 @@ public function is_group_allowed($perm_par, $group_par=FALSE){
 		// 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 @@ public function is_group_allowed($perm_par, $group_par=FALSE){
 			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
@@ -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
+-- ----------------------------
+