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

lgalfaso · lgalfaso · commit 37cb9ad43619 · 2016-01-28T15:33:39.000+01:00
Closes: angular#13871
diff --git a/src/ng/parse.js b/src/ng/parse.js
@@ -1819,6 +1819,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
@@ -2457,6 +2457,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
`@@ -1819,6 +1819,7 @@ function $ParseProvider() {`
`1819`	`1819`	`for (var i = 0; fn.inputs && i < fn.inputs.length; ++i) {`
`1820`	`1820`	`fn.inputs[i] = expensiveChecksInterceptor(fn.inputs[i]);`
`1821`	`1821`	`}`
	`1822`	`+ expensiveCheckFn.inputs = fn.inputs;`
`1822`	`1823`
`1823`	`1824`	`return expensiveCheckFn;`
`1824`	`1825`