Number Parser: Remove unnecessary regular expression part

Ref #292 Ref #353
globalizejs · 1daba30d1ec7fdec0f5c172fca6bfc09a71dcb50 · rxaviers committed Dec 30, 2016 · Dec 30, 2016 · 1daba30
1 parent f76714a
commit 1daba30d1ec7fdec0f5c172fca6bfc09a71dcb50
Unified Split

Showing with 3 additions and 3 deletions.

+2 −2 src/number/parse-properties.js

+1 −1 test/unit/number/parse-properties.js
diff --git a/src/number/parse-properties.js b/src/number/parse-properties.js
@@ -111,7 +111,7 @@ return function( pattern, cldr, options ) {
 	//
 	// digits_w_1_grouping_separators =  regexp(\d{1,3}(,\d{3})+)
 	//
-	// digits_w_2_grouping_separators =  regexp(\d{1,2}((,\d{2})*(,\d{3}))?)
+	// digits_w_2_grouping_separators =  regexp(\d{1,2}((,\d{2})*(,\d{3})))
 
 	// Integer part
 	numberTokenizer = digitsRe + "+";
@@ -121,7 +121,7 @@ return function( pattern, cldr, options ) {
 		if ( secondaryGroupingSize ) {
 			aux = digitsRe + "{1," + secondaryGroupingSize + "}((" + groupingSeparatorRe +
 				digitsRe + "{" + secondaryGroupingSize + "})*(" + groupingSeparatorRe +
-				digitsRe + "{" + primaryGroupingSize + "}))?";
+				digitsRe + "{" + primaryGroupingSize + "}))";
 		} else {
 			aux = digitsRe + "{1," + primaryGroupingSize + "}(" + groupingSeparatorRe +
 				digitsRe + "{" + primaryGroupingSize + "})+";
diff --git a/test/unit/number/parse-properties.js b/test/unit/number/parse-properties.js
@@ -105,7 +105,7 @@ QUnit.test( "should return number tokenizer", function( assert ) {
 
 	assert.deepEqual(
 		properties( "#,##,##0.##", en )[ 2 ].number,
-		/^((\d{1,2}((,\d{2})*(,\d{3}))?|\d+))?(\.\d+)?/
+		/^((\d{1,2}((,\d{2})*(,\d{3}))|\d+))?(\.\d+)?/
 	);
 
 	assert.deepEqual(