makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-28 06:08:52 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/12109] Bug when setting permissions with "Select all users"

Browse source 
https://tracker.phpbb.com/browse/PHPBB3-12109

PHPBB3-12109
This commit is contained in:
Tristan Darricau 2014-05-10 01:53:51 +02:00
parent d4fc060bcd
commit 9b9c048702
1 changed files with 16 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
phpBB/includes/acp/acp_permissions.php
View file

@ -330,14 +330,6 @@ class acp_permissions
} }
} }
// Setting permissions screen
$s_hidden_fields = build_hidden_fields(array(
'user_id' => $user_id,
'group_id' => $group_id,
'forum_id' => $forum_id,
'type' => $permission_type)
);
// Go through the screens/options needed and present them in correct order // Go through the screens/options needed and present them in correct order
foreach ($permission_victim as $victim) foreach ($permission_victim as $victim)
{ {
@ -470,6 +462,14 @@ class acp_permissions
// If there are more than 5 forums selected the admin is not able to select all users/groups too. // If there are more than 5 forums selected the admin is not able to select all users/groups too.
// We need to see if the number of forums can be increased or need to be decreased. // We need to see if the number of forums can be increased or need to be decreased.
// Setting permissions screen
$s_hidden_fields = build_hidden_fields(array(
'user_id' => $user_id,
'group_id' => $group_id,
'forum_id' => $forum_id,
'type' => $permission_type)
);
$template->assign_vars(array( $template->assign_vars(array(
'U_ACTION' => $this->u_action, 'U_ACTION' => $this->u_action,
'ANONYMOUS_USER_ID' => ANONYMOUS, 'ANONYMOUS_USER_ID' => ANONYMOUS,
@ -506,6 +506,14 @@ class acp_permissions
return; return;
} }
// Setting permissions screen
$s_hidden_fields = build_hidden_fields(array(
'user_id' => $user_id,
'group_id' => $group_id,
'forum_id' => $forum_id,
'type' => $permission_type)
);
// Do not allow forum_ids being set and no other setting defined (will bog down the server too much) // Do not allow forum_ids being set and no other setting defined (will bog down the server too much)
if (sizeof($forum_id) && !sizeof($user_id) && !sizeof($group_id)) if (sizeof($forum_id) && !sizeof($user_id) && !sizeof($group_id))
{ {