From 7fb6b1ac8083bb7211e6395b68e062a5fa25580b Mon Sep 17 00:00:00 2001 From: Neil Drumm <drumm@3064.no-reply.drupal.org> Date: Thu, 16 Nov 2006 08:45:49 +0000 Subject: [PATCH] #97686 by chx. Replace a loop with a standard PHP function. --- modules/user/user.module | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/modules/user/user.module b/modules/user/user.module index 92e0cee34db3..4ee3ae7ae5f5 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]))); } -- GitLab