Hopiu/angular.js
1
0
Fork 0
mirror of https://github.com/Hopiu/angular.js.git synced 2026-03-17 07:40:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(select): double array issue with multislect and jQuery

Browse source
This commit is contained in:
Misko Hevery 2012-02-03 10:34:16 -08:00
parent 13b21aaf5a
commit 85b2084f57
2 changed files with 25 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/widget/select.js
View file

@ -243,9 +243,8 @@ var selectDirective = ['$formFactory', '$compile', '$parse',
widget.$apply(function() {
var optionGroup,
collection = valuesFn(modelScope) || [],
key = selectElement.val(),
tempScope = inherit(modelScope),
value, optionElement, index, groupIndex, length, groupLength;
key, value, optionElement, index, groupIndex, length, groupLength;
if (multiple) {
value = [];
@ -257,13 +256,15 @@ var selectDirective = ['$formFactory', '$compile', '$parse',
for(index = 1, length = optionGroup.length; index < length; index++) {
if ((optionElement = optionGroup[index].element)[0].selected) {
key = optionElement.val();
if (keyName) tempScope[keyName] = key;
tempScope[valueName] = collection[optionElement.val()];
tempScope[valueName] = collection[key];
value.push(valueFn(tempScope));
}
}
}
} else {
key = selectElement.val();
if (key == '?') {
value = undefined;
} else if (key == ''){

21
test/widget/selectSpec.js
View file

@ -591,6 +591,27 @@ describe('select', function() {
browserTrigger(select, 'change');
expect(scope.selected).toEqual([scope.values[0]]);
});
it('should select from object', function() {
createSelect({
'ng:model':'selected',
'multiple':true,
'ng:options':'key as value for (key,value) in values'
});
scope.values = {'0':'A', '1':'B'};
scope.selected = ['1'];
scope.$digest();
expect(select.find('option')[1].selected).toBe(true);
select.find('option')[0].selected = true;
browserTrigger(select, 'change');
expect(scope.selected).toEqual(['0', '1']);
select.find('option')[1].selected = false;
browserTrigger(select, 'change');
expect(scope.selected).toEqual(['0']);
});
});
});
});