fix expressions on repeated items

vuejs · Jan 15, 2014 · 08ba942 · 08ba942
1 parent 0dcf28e
commit 08ba942
Show file tree

Hide file tree

Showing 4 changed files with 53 additions and 4 deletions.
diff --git a/src/compiler.js b/src/compiler.js
@@ -530,9 +530,7 @@ CompilerProto.markComputed = function (binding) {
         vm    = this.vm
     binding.isComputed = true
     // bind the accessors to the vm
-    if (binding.isFn) {
-        binding.value = utils.bind(value, vm)
-    } else {
+    if (!binding.isFn) {
         value.$get = utils.bind(value.$get, vm)
         if (value.$set) {
             value.$set = utils.bind(value.$set, vm)

diff --git a/src/directives/on.js b/src/directives/on.js
@@ -29,6 +29,7 @@ module.exports = {
 
         var compiler = this.compiler,
             event    = this.arg,
+            isExp    = this.binding.isExp,
             ownerVM  = this.binding.compiler.vm
 
         if (compiler.repeat &&
@@ -51,7 +52,7 @@ module.exports = {
                 if (target) {
                     e.el = target
                     e.targetVM = target.vue_viewmodel
-                    handler.call(ownerVM, e)
+                    handler.call(isExp ? e.targetVM : ownerVM, e)
                 }
             }
             dHandler.event = event

diff --git a/test/functional/fixtures/repeated-exp.html b/test/functional/fixtures/repeated-exp.html
@@ -0,0 +1,31 @@
+<!DOCTYPE html>
+<html lang="en">
+    <head>
+        <title>Repeated Expressions</title>
+        <meta charset="utf-8">
+        <script src="../../../dist/vue.js"></script>
+    </head>
+    <body>
+        <ul id="test">
+            <li
+                v-repeat="items"
+                v-on="click:n++"
+                class="item-{{$index}}"
+            >
+                {{n}}
+            </li>
+        </ul>
+        <script>
+            var app = new Vue({
+                el: '#test',
+                data: {
+                    items: [
+                        { n: 1 },
+                        { n: 2 },
+                        { n: 3 }
+                    ]
+                }
+            })
+        </script>
+    </body>
+</html>
diff --git a/test/functional/specs/repeated-exp.js b/test/functional/specs/repeated-exp.js
@@ -0,0 +1,19 @@
+casper.test.begin('Repeated Expressions', 3, function (test) {
+
+    casper
+    .start('./fixtures/repeated-exp.html')
+    .then(function () {
+        this.click('.item-0')
+        this.click('.item-1')
+        this.click('.item-2')
+    })
+    .then(function () {
+        test.assertSelectorHasText('.item-0', '2')
+        test.assertSelectorHasText('.item-1', '3')
+        test.assertSelectorHasText('.item-2', '4')
+    })
+    .run(function () {
+        test.done()
+    })
+
+})