Skip to content

Commit

Permalink
fix(select): object as value
Browse files Browse the repository at this point in the history
fixes: #11413
  • Loading branch information
manucorporat committed Apr 27, 2017
1 parent b114427 commit 4c8efc2
Showing 1 changed file with 14 additions and 7 deletions.
21 changes: 14 additions & 7 deletions src/components/select/select.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { Config } from '../../config/config';
import { DeepLinker } from '../../navigation/deep-linker';
import { Form } from '../../util/form';
import { BaseInput } from '../../util/base-input';
import { isCheckedProperty, isTrueProperty, deepCopy, deepEqual } from '../../util/util';
import { isCheckedProperty, isTrueProperty, deepCopy, deepEqual, assert } from '../../util/util';
import { Item } from '../item/item';
import { NavController } from '../../navigation/nav-controller';
import { Option } from '../option/option';
Expand Down Expand Up @@ -147,13 +147,12 @@ import { SelectPopover, SelectPopoverOption } from './select-popover-component';
providers: [ { provide: NG_VALUE_ACCESSOR, useExisting: Select, multi: true } ],
encapsulation: ViewEncapsulation.None,
})
export class Select extends BaseInput<string[]|string> implements AfterViewInit, OnDestroy {
export class Select extends BaseInput<any> implements AfterViewInit, OnDestroy {

_multi: boolean = false;
_options: QueryList<Option>;
_texts: string[] = [];
_text: string = '';
_values: string[] = [];

/**
* @input {string} The text to display on the cancel button. Default: `Cancel`.
Expand Down Expand Up @@ -222,6 +221,15 @@ export class Select extends BaseInput<string[]|string> implements AfterViewInit,
this.open();
}

/**
* @hidden
*/
getValues(): any[] {
const values = Array.isArray(this._value) ? this._value : [this._value];
assert(!this._multi && values.length <= 1, 'single only can have one value');
return values;
}

/**
* Open the select interface.
*/
Expand Down Expand Up @@ -394,8 +402,8 @@ export class Select extends BaseInput<string[]|string> implements AfterViewInit,
@ContentChildren(Option)
set options(val: QueryList<Option>) {
this._options = val;

if (this._values.length === 0) {
const values = this.getValues();
if (values.length === 0) {
// there are no values set at this point
// so check to see who should be selected
// we use writeValue() because we don't want to update ngModel
Expand All @@ -422,12 +430,11 @@ export class Select extends BaseInput<string[]|string> implements AfterViewInit,
*/
_inputUpdated() {
this._texts.length = 0;
this._values = Array.isArray(this._value) ? this._value : [this._value + ''];

if (this._options) {
this._options.forEach(option => {
// check this option if the option's value is in the values array
option.selected = this._values.some(selectValue => {
option.selected = this.getValues().some(selectValue => {
return isCheckedProperty(selectValue, option.value);
});

Expand Down

0 comments on commit 4c8efc2

Please sign in to comment.