From aa112244fd7bac268c65abc2be6178abacb0cff4 Mon Sep 17 00:00:00 2001
From: Hristo Deshev <hristo@deshev.com>
Date: Fri, 5 Jun 2015 14:40:38 +0300
Subject: [PATCH] Parse out attributes without values.

Treat #myattr as #myAttr=''.
---
 js-libs/easysax/easysax.js     | 65 +++++++++++++++++++++-------------
 node-tests/test-angular-xml.ts | 36 ++++++++++++++++++-
 2 files changed, 75 insertions(+), 26 deletions(-)

diff --git a/js-libs/easysax/easysax.js b/js-libs/easysax/easysax.js
index 7b6f7f10e..324627adf 100644
--- a/js-libs/easysax/easysax.js
+++ b/js-libs/easysax/easysax.js
@@ -237,12 +237,12 @@ function EasySAXParser() {
 		, attr_list = hasSurmiseNS ? [] : false
 		, name, value = ''
 		, ok = false
+        , noValueAttribute = false
 		, j, w, nn, n
 		, hasNewMatrix
 		, alias, newalias
 		; 
 
-
 		aa: 
 		for(; i < l; i++) {
 			w = s.charCodeAt(i);
@@ -251,7 +251,7 @@ function EasySAXParser() {
 				continue
 			};
 
-			if ((w < 65 && w !== 40 && w !== 41) || // allow parens () -- used for angular syntax
+			if ((w < 65 && w !== 40 && w !== 41 && w !== 35) || // allow parens () -- used for angular syntax
                     w > 122 || (w === 92 || (w > 93 && w < 97)) ) { // ожидаем символ
 				return attr_res = false; // error. invalid char
 			};
@@ -260,21 +260,28 @@ function EasySAXParser() {
 				w = s.charCodeAt(j);
 
 				if (w > 96 && w < 123 || w > 64 && w < 91 || w > 47 && w < 59 || w === 45 || w === 95 || w === 46 /* https://github.com/telerik/xPlatCore/issues/179 */) {
-					continue;
+                    if (noValueAttribute) {
+                        j--; //Started next attribute. Get back and break out of the loop.
+                        break;
+                    } else {
+                        continue;
+                    }
 				};
 
-                if (w === 32 || (w > 8 && w < 14) ) {  // \f\n\r\t\v пробел
-                    continue;
-                };
-
-                if (w === 91 || w === 93 || w === 40 || w === 41 || w === 94) { // Angular special attribute chars:[]()^
+                if (w === 91 || w === 93 || w === 40 || w === 41 || w === 94 || w === 35) { // Angular special attribute chars:[]()^
                     continue;
                 }
 
-
-				if (w !== 61) { // "=" == 61
+                if (w === 32 || (w > 8 && w < 14) ) {  // \f\n\r\t\v пробел
+                    noValueAttribute = true;
+                    continue;
+                } else if (w === 61) { // "=" == 61
+                    noValueAttribute = false;
+                    break;
+                } else {
 					//console.log('error 2');
-					return attr_res = false; // error. invalid char
+                    if (!noValueAttribute)
+                        return attr_res = false; // error. invalid char
 				};
 
 				break;
@@ -298,18 +305,19 @@ function EasySAXParser() {
                 }
             }
 
-			if (w === 34) {  // '"'
-				j = s.indexOf('"', i = j+2 );
+            if (!noValueAttribute) {
+                if (w === 34) {  // '"'
+                    j = s.indexOf('"', i = j+2 );
 
-			} else {
-				if (w === 39) {
-					j = s.indexOf('\'', i = j+2 );
+                } else {
+                    if (w === 39) {
+                        j = s.indexOf('\'', i = j+2 );
 
-				} else {  // "'"
-					//console.log('error 3')
-					return attr_res = false; // error. invalid char
-				};
-			};
+                    } else {  // "'"
+                        return attr_res = false; // error. invalid char
+                    };
+                };
+            }
 
 			if (j === -1) {
 				//console.log('error 4')
@@ -317,10 +325,10 @@ function EasySAXParser() {
 			};
 
 
-			if (j+1 < l) {
+			if (j+1 < l && !noValueAttribute) {
 				w = s.charCodeAt(j+1);
 
-				if (w > 32 || w < 9 || (w<32 && w>13)) {
+				if (w > 32 || w < 9 || (w < 32 && w > 13)) {
 					// error. invalid char
 					//console.log('error 5')
 					return attr_res = false;
@@ -328,8 +336,14 @@ function EasySAXParser() {
 			};
 
 
-			value = s.substring(i, j);
-			i = j + 1; // след. семвол уже проверен потому проверять нужно следуюший
+            if (noValueAttribute) {
+                value = '';
+            } else {
+                value = s.substring(i, j);
+            }
+
+            //i = j + 1; // след. семвол уже проверен потому проверять нужно следуюший
+            i = j; // след. семвол уже проверен потому проверять нужно следуюший
 
 			if (isNamespace) { // 
 				if (hasSurmiseNS) {
@@ -382,6 +396,7 @@ function EasySAXParser() {
 			};
 
 			res[name] = value;
+            noValueAttribute = false;
 		};
 
 
diff --git a/node-tests/test-angular-xml.ts b/node-tests/test-angular-xml.ts
index 3c65eddd5..72bb3ff52 100644
--- a/node-tests/test-angular-xml.ts
+++ b/node-tests/test-angular-xml.ts
@@ -31,10 +31,44 @@ describe("angular xml parser", () => {
         assert.equal(last_attrs['(tap)'], 'onTap(blah)');
     });
 
-    it("parsers (^event) binding", () => {
+    it("parses (^event) binding", () => {
         parser.parse("<TextField (^tap)='onTap(blah)' />");
 
         assert.equal('TextField', last_element);
         assert.equal(last_attrs['(^tap)'], 'onTap(blah)');
     });
+
+    it("parses #id attribute", () => {
+        parser.parse("<TextField #firstName />");
+
+        assert.equal('TextField', last_element);
+        assert.equal(last_attrs['#firstName'], '');
+    });
+
+    it("parses #id attribute followed by another", () => {
+        parser.parse("<TextField #firstName text='Name' />");
+
+        assert.equal('TextField', last_element);
+        assert.equal(last_attrs['#firstName'], '');
+        assert.equal(last_attrs['text'], 'Name');
+    });
+
+    return
+    it("detects equals without value", () => {
+        parser.parse("<TextField brokenTag= />");
+
+        assert.isFalse(last_attrs);
+    });
+
+    it("detects no equals with quoted value", () => {
+        parser.parse("<TextField noEquals 'value' />");
+
+        assert.isFalse(last_attrs);
+    });
+
+    it("detects unclosed tag after no value attribute", () => {
+        parser.parse("<TextField #myId");
+
+        assert.isFalse(last_attrs);
+    });
 });