diff options
author | Neil Drumm <drumm@3064.no-reply.drupal.org> | 2006-11-16 08:45:49 +0000 |
---|---|---|
committer | Neil Drumm <drumm@3064.no-reply.drupal.org> | 2006-11-16 08:45:49 +0000 |
commit | 7fb6b1ac8083bb7211e6395b68e062a5fa25580b (patch) | |
tree | 2ea81116086c8dabc3c3fd00eb8672f76d8ec44d /modules/user/user.module | |
parent | 5278747df5b6b9b0a645093cd4ea08fea4a675aa (diff) | |
download | brdo-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.module | 6 |
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]))); } |