summaryrefslogtreecommitdiff
path: root/modules/user/user.module
diff options
context:
space:
mode:
authorNeil Drumm <drumm@3064.no-reply.drupal.org>2006-11-16 08:45:49 +0000
committerNeil Drumm <drumm@3064.no-reply.drupal.org>2006-11-16 08:45:49 +0000
commit7fb6b1ac8083bb7211e6395b68e062a5fa25580b (patch)
tree2ea81116086c8dabc3c3fd00eb8672f76d8ec44d /modules/user/user.module
parent5278747df5b6b9b0a645093cd4ea08fea4a675aa (diff)
downloadbrdo-7fb6b1ac8083bb7211e6395b68e062a5fa25580b.tar.gz
brdo-7fb6b1ac8083bb7211e6395b68e062a5fa25580b.tar.bz2
#97686 by chx. Replace a loop with a standard PHP function.
Diffstat (limited to 'modules/user/user.module')
-rw-r--r--modules/user/user.module6
1 files changed, 1 insertions, 5 deletions
diff --git a/modules/user/user.module b/modules/user/user.module
index 92e0cee34..4ee3ae7ae 100644
--- a/modules/user/user.module
+++ b/modules/user/user.module
@@ -1851,11 +1851,7 @@ function user_admin_perm_submit($form_id, $form_values) {
// Delete, so if we clear every checkbox we reset that role;
// otherwise permissions are active and denied everywhere.
db_query('DELETE FROM {permission} WHERE rid = %d', $role->rid);
- foreach ($form_values[$role->rid] as $key => $value) {
- if (!$value) {
- unset($form_values[$role->rid][$key]);
- }
- }
+ $form_values[$role->rid] = array_filter($form_values[$role->rid]);
if (count($form_values[$role->rid])) {
db_query("INSERT INTO {permission} (rid, perm) VALUES (%d, '%s')", $role->rid, implode(', ', array_keys($form_values[$role->rid])));
}