diff options
| author | Dries Buytaert <dries@buytaert.net> | 2010-10-28 02:27:09 +0000 |
|---|---|---|
| committer | Dries Buytaert <dries@buytaert.net> | 2010-10-28 02:27:09 +0000 |
| commit | 1d0f6479da4cdbd14b747ead6cb934a769a9579d (patch) | |
| tree | 3875e9403a28aac4477a3531379c85b6b10cd660 /modules/field | |
| parent | 0344a78d9643d97e1aff8ab94fec551bc091bfbc (diff) | |
| download | brdo-1d0f6479da4cdbd14b747ead6cb934a769a9579d.tar.gz brdo-1d0f6479da4cdbd14b747ead6cb934a769a9579d.tar.bz2 | |
- Patch #839556 by jrchamp, dmitrig01: remove pointless usage of array_key_exists().
Diffstat (limited to 'modules/field')
| -rw-r--r-- | modules/field/modules/list/list.module | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/field/modules/list/list.module b/modules/field/modules/list/list.module index 3ad913e58..7addd362d 100644 --- a/modules/field/modules/list/list.module +++ b/modules/field/modules/list/list.module @@ -261,7 +261,7 @@ function list_field_validate($entity_type, $entity, $field, $instance, $langcode $allowed_values = list_allowed_values($field); foreach ($items as $delta => $item) { if (!empty($item['value'])) { - if (count($allowed_values) && !array_key_exists($item['value'], $allowed_values)) { + if (!empty($allowed_values) && !isset($allowed_values[$item['value']])) { $errors[$field['field_name']][$langcode][$delta][] = array( 'error' => 'list_illegal_value', 'message' => t('%name: illegal value.', array('%name' => t($instance['label']))), |