fix($parse): do not pass scope/locals to interceptors of one-time bindings

jbedard · jbedard · commit 87a586eb9a23 · 2017-07-21T20:56:14.000-07:00
diff --git a/src/ng/parse.js b/src/ng/parse.js
@@ -1907,7 +1907,7 @@ function $ParseProvider() {
         if (isDone(lastValue)) {
           scope.$$postDigest(unwatchIfDone);
         }
-        return post(lastValue, scope, locals);
+        return post(lastValue);
       }
     }
 
diff --git a/test/ng/parseSpec.js b/test/ng/parseSpec.js
@@ -3246,6 +3246,20 @@ describe('parser', function() {
             expect(args).toEqual([1]);
           }));
 
+          it('should only be passed the intercepted value when wrapping one-time', inject(function($parse) {
+            var args;
+            function interceptor(v) {
+              args = sliceArgs(arguments);
+              return v;
+            }
+
+            scope.$watch($parse('::a', interceptor));
+
+            scope.a = 1;
+            scope.$digest();
+            expect(args).toEqual([1]);
+          }));
+
           it('should only be passed the intercepted value when double-intercepted',
               inject(function($parse) {
             var args1;

Original file line number	Diff line number	Diff line change
`@@ -1907,7 +1907,7 @@ function $ParseProvider() {`
`1907`	`1907`	`if (isDone(lastValue)) {`
`1908`	`1908`	`scope.$$postDigest(unwatchIfDone);`
`1909`	`1909`	`}`
`1910`		`- return post(lastValue, scope, locals);`
	`1910`	`+ return post(lastValue);`
`1911`	`1911`	`}`
`1912`	`1912`	`}`
`1913`	`1913`