fix(): allow spaces around dots for identifiers

Fixes angular#4613

Author: pkozlowski-opensource

src/ng/parse.js

@@ -263,15 +263,17 @@ Lexer.prototype = {
     var ident = '';
     var start = this.index;
 
-    var lastDot, peekIndex, methodName, ch;
+    var lastDot, justSeenADot, peekIndex, methodName, ch;
 
     while (this.index < this.text.length) {
       ch = this.text.charAt(this.index);
       if (ch === '.' || this.isIdent(ch) || this.isNumber(ch)) {
-        if (ch === '.') lastDot = this.index;
+        if (justSeenADot = ch === '.') lastDot = this.index;
         ident += ch;
       } else {
-        break;
+        if (!(this.isWhitespace(ch) && justSeenADot)) {
+          break;
+        }
       }
       this.index++;
     }

test/ng/parseSpec.js

@@ -71,6 +71,10 @@ describe('parser', function() {
       expect(tokens[i].string).toEqual('d"e');
     });
 
+    it('should tokenize identifiers with spaces after dots', function () {
+      expect(lex('foo. bar')[0].text).toEqual('foo.bar');
+    });
+
     it('should tokenize undefined', function() {
       var tokens = lex("undefined");
       var i = 0;
@@ -349,6 +353,14 @@ describe('parser', function() {
         expect(scope.$eval("x.y.z", scope)).not.toBeDefined();
       });
 
+      it('should handle white-spaces around dots in paths', function () {
+        scope.a = {b: 4};
+        expect(scope.$eval("a . b", scope)).toEqual(4);
+        expect(scope.$eval("a. b", scope)).toEqual(4);
+        expect(scope.$eval("a .b", scope)).toEqual(4);
+        expect(scope.$eval("a    . \nb", scope)).toEqual(4);
+      });
+
       it('should resolve deeply nested paths (important for CSP mode)', function() {
         scope.a = {b: {c: {d: {e: {f: {g: {h: {i: {j: {k: {l: {m: {n: 'nooo!'}}}}}}}}}}}}};
         expect(scope.$eval("a.b.c.d.e.f.g.h.i.j.k.l.m.n", scope)).toBe('nooo!');