Skip to content
Snippets Groups Projects
Commit b283ca75 authored by Marcin Kolanko's avatar Marcin Kolanko
Browse files

Merge branch 'bugfix/select-multiple' into 'master'

Bugfix/select multiple

See merge request !8
parents 5ba28f5d 9d0a8887
Branches
Tags 2.2.1
1 merge request!8Bugfix/select multiple
Pipeline #5249 failed
# Changelog
## [2.2.1] - 2020-08-18
### Changed
- Fiexed bug with the "select multiple" field
## [2.2.0] - 2020-07-30
### Added
- Date picker field
......
......@@ -2,13 +2,13 @@
/**
* @var \WPDesk\Forms\Field $field
* @var string $name_prefix
* @var string $value
* @var mixed $value
*/
?>
<select
id="<?php echo esc_attr( $field->get_id() ); ?>"
<?php if ($field->has_classes()): ?>class="<?php echo esc_attr( $field->get_classes() ); ?>"<?php endif; ?>
name="<?php echo esc_attr( $name_prefix ); ?>[<?php echo esc_attr( $field->get_name() ); ?>]"
name="<?php echo esc_attr( $name_prefix ); ?>[<?php echo esc_attr( $field->get_name() ); ?>]<?php echo $field->is_multiple()? '[]' : ''; ?>"
<?php foreach ($field->get_attributes() as $key => $attr_val): ?>
<?php echo esc_attr($key); ?>="<?php echo esc_attr($attr_val); ?>"
<?php endforeach; ?>
......@@ -22,7 +22,7 @@
<?php foreach ( $field->get_possible_values() as $possible_value => $label ): ?>
<option
<?php if ( $possible_value === $value || (is_numeric($possible_value) && is_numeric($value) && (int) $possible_value === (int) $value )): ?>selected="selected"<?php endif; ?>
<?php if ( $possible_value === $value || (is_array($value) && in_array($possible_value, $value)) || (is_numeric($possible_value) && is_numeric($value) && (int) $possible_value === (int) $value )): ?>selected="selected"<?php endif; ?>
value="<?php echo esc_attr( $possible_value ); ?>"
><?php echo esc_html( $label ); ?></option>
<?php endforeach; ?>
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment