phpbb-extensions / autogroups

ext.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
 	 * Notification handler to call notification enable/disable/purge steps
 	 *
 	 * @param string $step               The step (enable, disable, purge)
-	 * @param array  $notification_types The notification type names
+	 * @param string[]  $notification_types The notification type names
 	 * @return string Return notifications as temporary state
 	 * @access protected
 	 */

conditions/type/base.php

Braces +1 added lines, -2 removed lines

@@ -258,8 +258,7 @@
 			if ($group_rule['autogroups_group_id'] == $current_rule['autogroups_group_id'] &&
 				$group_rule['autogroups_type_id'] != $current_rule['autogroups_type_id'] &&
 				sizeof($user_id_ary)
-			)
-			{
+			) {
 				// Load other auto group rule's condition type and get new data for our user(s)
 				$condition = $this->container->get($group_rule['autogroups_type_name']);
 				$condition_user_data = $condition->get_users_for_condition(array(

controller/admin_controller.php

Braces +4 added lines, -1 removed lines

@@ -254,11 +254,14 @@
 			trigger_error($this->user->lang('ACP_AUTOGROUPS_INVALID_RANGE') . adm_back_link($this->u_action), E_USER_WARNING);
 		}
 
-		if ($autogroups_id != 0) // Update existing auto group data
+		if ($autogroups_id != 0)
+		{
+			// Update existing auto group data
 		{
 			$sql = 'UPDATE ' . $this->autogroups_rules_table . '
 				SET ' . $this->db->sql_build_array('UPDATE', $data) . '
 				WHERE autogroups_id = ' . (int) $autogroups_id;
+		}
 			$this->db->sql_query($sql);
 		}
 		else // Insert new auto group data