@@ -258,8 +258,7 @@ |
||
258 | 258 | if ($group_rule['autogroups_group_id'] == $current_rule['autogroups_group_id'] && |
259 | 259 | $group_rule['autogroups_type_id'] != $current_rule['autogroups_type_id'] && |
260 | 260 | count($user_id_ary) |
261 | - ) |
|
262 | - { |
|
261 | + ) { |
|
263 | 262 | // Load other auto group rule's condition type and get new data for our user(s) |
264 | 263 | $condition = $this->container->get($group_rule['autogroups_type_name']); |
265 | 264 | $condition_user_data = $condition->get_users_for_condition(array( |
@@ -296,11 +296,14 @@ |
||
296 | 296 | // Format autogroups_excluded_groups for storage in the db |
297 | 297 | $data['autogroups_excluded_groups'] = !empty($data['autogroups_excluded_groups']) ? json_encode($data['autogroups_excluded_groups']) : ''; |
298 | 298 | |
299 | - if ($autogroups_id != 0) // Update existing auto group data |
|
299 | + if ($autogroups_id != 0) |
|
300 | + { |
|
301 | + // Update existing auto group data |
|
300 | 302 | { |
301 | 303 | $sql = 'UPDATE ' . $this->autogroups_rules_table . ' |
302 | 304 | SET ' . $this->db->sql_build_array('UPDATE', $data) . ' |
303 | 305 | WHERE autogroups_id = ' . (int) $autogroups_id; |
306 | + } |
|
304 | 307 | $this->db->sql_query($sql); |
305 | 308 | } |
306 | 309 | else // Insert new auto group data |
@@ -177,15 +177,21 @@ |
||
177 | 177 | */ |
178 | 178 | public function membership_check($event) |
179 | 179 | { |
180 | - if ($event->offsetExists('session_data')) // core.session_create_after |
|
180 | + if ($event->offsetExists('session_data')) |
|
181 | + { |
|
182 | + // core.session_create_after |
|
181 | 183 | { |
182 | 184 | // only check session at login (this will prevent repeated excessive attempts at this check) |
183 | 185 | $users = strpos($event['session_data']['session_page'], 'login') !== false ? $event['session_data']['session_user_id'] : null; |
184 | 186 | } |
185 | - else if ($event->offsetExists('user_id_ary')) // core.user_add_after |
|
187 | + } |
|
188 | + else if ($event->offsetExists('user_id_ary')) |
|
189 | + { |
|
190 | + // core.user_add_after |
|
186 | 191 | { |
187 | 192 | $users = $event['user_id_ary']; |
188 | 193 | } |
194 | + } |
|
189 | 195 | else // core.user_active_flip_after |
190 | 196 | { |
191 | 197 | $users = $event['user_id']; |