fix($parse): Copy inputs for expressions with expensive checks

lgalfaso · gkalpak · commit 0b7fff303f46 · 2016-01-28T21:13:48.000+02:00
Closes: angular#13871
diff --git a/src/ng/parse.js b/src/ng/parse.js
@@ -1797,6 +1797,7 @@ function $ParseProvider() {
       for (var i = 0; fn.inputs && i < fn.inputs.length; ++i) {
         fn.inputs[i] = expensiveChecksInterceptor(fn.inputs[i]);
       }
+      expensiveCheckFn.inputs = fn.inputs;
 
       return expensiveCheckFn;
 
diff --git a/test/ng/parseSpec.js b/test/ng/parseSpec.js
@@ -2429,6 +2429,13 @@ describe('parser', function() {
                 expect($parse.$$runningExpensiveChecks()).toEqual(false);
               });
             });
+
+            it('should handle `inputs` when running with expensive checks', inject(function($parse) {
+              expect(function() {
+                scope.$watch($parse('a + b', null, true), noop);
+                scope.$digest();
+              }).not.toThrow();
+            }));
           });
         });
 

Original file line number	Diff line number	Diff line change
`@@ -1797,6 +1797,7 @@ function $ParseProvider() {`
`1797`	`1797`	`for (var i = 0; fn.inputs && i < fn.inputs.length; ++i) {`
`1798`	`1798`	`fn.inputs[i] = expensiveChecksInterceptor(fn.inputs[i]);`
`1799`	`1799`	`}`
	`1800`	`+ expensiveCheckFn.inputs = fn.inputs;`
`1800`	`1801`
`1801`	`1802`	`return expensiveCheckFn;`
`1802`	`1803`