Merge pull request vuejs#1949 from rhyzx/inline-statement-variable

yyx990803 · yyx990803 · commit f819514e0ed7 · 2015-12-02T11:09:34.000-05:00
Add $arguments variable in component events inline statement vuejs#1945
diff --git a/src/instance/api/data.js b/src/instance/api/data.js
@@ -1,5 +1,5 @@
 import Watcher from '../../watcher'
-import { del } from '../../util/index'
+import { del, toArray } from '../../util/index'
 import { parseText } from '../../parsers/text'
 import { parseDirective } from '../../parsers/directive'
 import { getPath } from '../../parsers/path'
@@ -23,7 +23,9 @@ export default function (Vue) {
       if (asStatement && !isSimplePath(exp)) {
         var self = this
         return function statementHandler () {
+          self.$arguments = toArray(arguments)
           res.get.call(self, self)
+          self.$arguments = null
         }
       } else {
         try {
diff --git a/test/unit/specs/instance/events_spec.js b/test/unit/specs/instance/events_spec.js
@@ -237,6 +237,24 @@ describe('Instance Events', function () {
       expect(vm.a).toBe(1)
     })
 
+    it('passing $arguments', function () {
+      new Vue({
+        el: document.createElement('div'),
+        template: '<comp @ready="onReady($arguments[1])"></comp>',
+        methods: {
+          onReady: spy
+        },
+        components: {
+          comp: {
+            compiled: function () {
+              this.$emit('ready', 123, 1234)
+            }
+          }
+        }
+      })
+      expect(spy).toHaveBeenCalledWith(1234)
+    })
+
     describe('attached/detached', function () {
 
       it('in DOM', function () {
