fix($parse): treat falsy values as defined in assignment expressions

Closes angular#14990
Closes angular#14994

Author: m-amr

src/ng/parse.js

@@ -944,7 +944,7 @@ ASTCompiler.prototype = {
           self.if_(self.stage === 'inputs' || 's', function() {
             if (create && create !== 1) {
               self.if_(
-                self.not(self.nonComputedMember('s', ast.name)),
+                self.isNull(self.nonComputedMember('s', ast.name)),
                 self.lazyAssign(self.nonComputedMember('s', ast.name), '{}'));
             }
             self.assign(intoId, self.nonComputedMember('s', ast.name));
@@ -973,7 +973,7 @@ ASTCompiler.prototype = {
             }
           } else {
             if (create && create !== 1) {
-              self.if_(self.not(self.nonComputedMember(left, ast.property.name)), self.lazyAssign(self.nonComputedMember(left, ast.property.name), '{}'));
+              self.if_(self.isNull(self.nonComputedMember(left, ast.property.name)), self.lazyAssign(self.nonComputedMember(left, ast.property.name), '{}'));
             }
             expression = self.nonComputedMember(left, ast.property.name);
             self.assign(intoId, expression);
@@ -1155,6 +1155,10 @@ ASTCompiler.prototype = {
     return '!(' + expression + ')';
   },
 
+  isNull: function(expression) {
+    return expression + '==null';
+  },
+
   notNull: function(expression) {
     return expression + '!=null';
   },
@@ -1546,7 +1550,7 @@ ASTInterpreter.prototype = {
   identifier: function(name, context, create, expression) {
     return function(scope, locals, assign, inputs) {
       var base = locals && (name in locals) ? locals : scope;
-      if (create && create !== 1 && base && !(base[name])) {
+      if (create && create !== 1 && base && base[name] == null) {
         base[name] = {};
       }
       var value = base ? base[name] : undefined;
@@ -1583,7 +1587,7 @@ ASTInterpreter.prototype = {
     return function(scope, locals, assign, inputs) {
       var lhs = left(scope, locals, assign, inputs);
       if (create && create !== 1) {
-        if (lhs && !(lhs[right])) {
+        if (lhs && lhs[right] == null) {
           lhs[right] = {};
         }
       }

test/ng/parseSpec.js

@@ -3155,6 +3155,45 @@ describe('parser', function() {
           expect(isFunction(s.toString)).toBe(true);
           expect(l.toString).toBe(1);
         }));
+
+        it('should assign a property to scope properties with null or undefined value', inject(function($parse) {
+          var scope = {a:null};
+          $parse('a.b =1')(scope);
+          expect(scope.a).toEqual({b:1});
+
+          scope.a = undefined;
+          $parse('a.b =1')(scope);
+          expect(scope.a).toEqual({b:1});
+        }));
+
+        they('should not assign a property to scope properties with $prop value', [0, false, ''],
+          function(falsyValue) {
+            inject(function($parse) {
+              var scope = {a:falsyValue};
+              expect(function() {
+                $parse('a.b=1')(scope);
+              }).toThrow();
+              expect(scope.a).toBe(falsyValue);
+
+              scope = {a:falsyValue};
+              expect(function() {
+                $parse('a[\'b\']=1')(scope);
+              }).toThrow();
+              expect(scope.a).toBe(falsyValue);
+
+              scope = {a:{b:falsyValue}};
+              expect(function() {
+                $parse('a.b.c=1')(scope);
+              }).toThrow();
+              expect(scope.a.b).toBe(falsyValue);
+
+              scope = {a:{b:falsyValue}};
+              expect(function() {
+                $parse('a.b[\'c\']=1')(scope);
+              }).toThrow();
+              expect(scope.a.b).toBe(falsyValue);
+            });
+          });
       });
 
       describe('literal', function() {