diff options
| author | Henry Sudhof <kellanved@phpbb.com> | 2007-06-04 10:44:09 +0000 |
|---|---|---|
| committer | Henry Sudhof <kellanved@phpbb.com> | 2007-06-04 10:44:09 +0000 |
| commit | 22f5b1aff683b98a3b989567517e7d15ad9ad1ce (patch) | |
| tree | 5d003a8931b0be1563d07b73d22dd6b37d324104 /phpBB/includes/ucp | |
| parent | 498b87df106597c4d7bf236561fa76e99eeeb30f (diff) | |
| download | forums-22f5b1aff683b98a3b989567517e7d15ad9ad1ce.tar forums-22f5b1aff683b98a3b989567517e7d15ad9ad1ce.tar.gz forums-22f5b1aff683b98a3b989567517e7d15ad9ad1ce.tar.bz2 forums-22f5b1aff683b98a3b989567517e7d15ad9ad1ce.tar.xz forums-22f5b1aff683b98a3b989567517e7d15ad9ad1ce.zip | |
Adding a few safeguards to stop people from breaking their own boards ;)
Translators, I'm sorry but this means some work for you.
git-svn-id: file:///svn/phpbb/trunk@7713 89ea8834-ac86-4346-8a33-228a782c2dd0
Diffstat (limited to 'phpBB/includes/ucp')
| -rw-r--r-- | phpBB/includes/ucp/ucp_groups.php | 29 |
1 files changed, 22 insertions, 7 deletions
diff --git a/phpBB/includes/ucp/ucp_groups.php b/phpBB/includes/ucp/ucp_groups.php index 99371d2024..4789da1f5a 100644 --- a/phpBB/includes/ucp/ucp_groups.php +++ b/phpBB/includes/ucp/ucp_groups.php @@ -943,14 +943,14 @@ class ucp_groups $user->add_lang(array('acp/groups', 'acp/common')); - $name_ary = request_var('usernames', '', true); + $names = request_var('usernames', '', true); if (!$group_id) { trigger_error($user->lang['NO_GROUP'] . $return_page); } - if (!$name_ary) + if (!$names) { trigger_error($user->lang['NO_USERS'] . $return_page); } @@ -966,15 +966,30 @@ class ucp_groups trigger_error($user->lang['NOT_LEADER_OF_GROUP'] . $return_page); } - $name_ary = array_unique(explode("\n", $name_ary)); + $name_ary = array_unique(explode("\n", $names)); $group_name = ($group_row['group_type'] == GROUP_SPECIAL) ? $user->lang['G_' . $group_row['group_name']] : $group_row['group_name']; $default = request_var('default', 0); - - // Add user/s to group - if ($error = group_user_add($group_id, false, $name_ary, $group_name, $default, 0, 0, $group_row)) + + if (confirm_box(true)) + { + // Add user/s to group + if ($error = group_user_add($group_id, false, $name_ary, $group_name, $default, 0, 0, $group_row)) + { + trigger_error($user->lang[$error] . $return_page); + } + } + else { - trigger_error($user->lang[$error] . $return_page); + $s_hidden_fields = array( + 'default' => $default, + 'usernames' => $names, + 'g' => $group_id, + 'i' => $id, + 'mode' => $mode, + 'action' => $action + ); + confirm_box(false, sprintf($user->lang['GROUP_CONFIRM_ADD_USER' . ((sizeof($name_ary) == 1) ? '' : 'S')], implode(', ', $name_ary)), build_hidden_fields($s_hidden_fields)); } trigger_error($user->lang['GROUP_USERS_ADDED'] . '<br /><br />' . sprintf($user->lang['RETURN_PAGE'], '<a href="' . $this->u_action . '&action=list&g=' . $group_id . '">', '</a>')); |