Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Enable multiple default values for XmlSelect

Browse source 
Useful for when a <select> tag with the 'multiple' attribute is
needed.

Bug: T93234
Change-Id: Ib83aed3392cf15cce4637fac56979a0f6b3d6b2f
This commit is contained in:
Geoffrey Mon 2015-09-13 22:52:33 +00:00
parent 6812ce1230
commit e36845aac6
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
includes/XmlSelect.php
View file

@ -43,7 +43,7 @@ class XmlSelect {
}
/**
* @param string $default
* @param string|array $default
*/
public function setDefault( $default ) {
$this->default = $default;
@ -95,7 +95,7 @@ class XmlSelect {
* label => ( label => value, label => value )
*
* @param array $options
* @param string $default
* @param string|array $default
* @return string
*/
static function formatOptions( $options, $default = false ) {
@ -106,7 +106,11 @@ class XmlSelect {
$contents = self::formatOptions( $value, $default );
$data .= Html::rawElement( 'optgroup', array( 'label' => $label ), $contents ) . "\n";
} else {
$data .= Xml::option( $label, $value, $value === $default ) . "\n";
// If $default is an array, then the <select> probably has the multiple attribute,
// so we should check if each $value is in $default, rather than checking if
// $value is equal to $default.
$selected = is_array( $default ) ? in_array( $value, $default ) : $value === $default;
$data .= Xml::option( $label, $value, $selected ) . "\n";
}
}