fix vuejs#670 honor number option for select with multiple attribute

yyx990803 · yyx990803 · commit 97041547f83c · 2015-02-19T12:25:54.000-08:00
diff --git a/src/directives/model/select.js b/src/directives/model/select.js
@@ -18,7 +18,9 @@ module.exports = {
         ? getMultiValue(el)
         : el.value
       value = self.number
-        ? _.toNumber(value)
+        ? _.isArray(value)
+          ? value.map(_.toNumber)
+          : _.toNumber(value)
         : value
       self.set(value, true)
     }
diff --git a/test/unit/specs/directives/model_spec.js b/test/unit/specs/directives/model_spec.js
@@ -286,6 +286,22 @@ if (_.inBrowser) {
       expect(vm.test).toBe(1)
     })
 
+    it('select + number + multiple', function () {
+      var vm = new Vue({
+        el: el,
+        data: {
+          test: []
+        },
+        template: '<select v-model="test" multiple number><option>1</option><option>2</option></select>'
+      })
+      ;[].forEach.call(el.querySelectorAll('option'), function (o) {
+        o.selected = true
+      })
+      trigger(el.firstChild, 'change')
+      expect(vm.test[0]).toBe(1)
+      expect(vm.test[1]).toBe(2)
+    })
+
     it('select + number initial value', function () {
       var vm = new Vue({
         el: el,
