fix($parse): set null reference properties to undefined

lgalfaso · lgalfaso · commit d19504a17935 · 2015-06-01T11:03:12.000+02:00
When there is an expression of the form `true && a.b` and where `a == null`, then set the value of `true && a.b` to `undefined`. Closes angular#11959
diff --git a/src/ng/parse.js b/src/ng/parse.js
@@ -983,6 +983,8 @@ ASTCompiler.prototype = {
             }
           }
           recursionFn(intoId);
+        }, function() {
+          self.assign(intoId, 'undefined');
         });
       }, !!create);
       break;
diff --git a/test/ng/parseSpec.js b/test/ng/parseSpec.js
@@ -1745,6 +1745,10 @@ describe('parser', function() {
         expect(scope.$eval("0&&2")).toEqual(0 && 2);
         expect(scope.$eval("0||2")).toEqual(0 || 2);
         expect(scope.$eval("0||1&&2")).toEqual(0 || 1 && 2);
+        expect(scope.$eval("true&&a")).toEqual(true && undefined);
+        expect(scope.$eval("true&&a.b")).toEqual(true && undefined);
+        expect(scope.$eval("false||a")).toEqual(false || undefined);
+        expect(scope.$eval("false||a.b")).toEqual(false || undefined);
       });
 
       it('should parse ternary', function() {

Original file line number	Diff line number	Diff line change
`@@ -983,6 +983,8 @@ ASTCompiler.prototype = {`
`983`	`983`	`}`
`984`	`984`	`}`
`985`	`985`	`recursionFn(intoId);`
	`986`	`+ }, function() {`
	`987`	`+ self.assign(intoId, 'undefined');`
`986`	`988`	`});`
`987`	`989`	`}, !!create);`
`988`	`990`	`break;`