Make StringParser parse empty string as null

JonasKress · JonasKress · commit eba3bb3a9bcd · 2015-07-30T18:24:44.000+02:00
This patch changes the StringParser to return null when the given string
is empty.

Bug: T93572
diff --git a/src/valueParsers/parsers/StringParser.js b/src/valueParsers/parsers/StringParser.js
@@ -20,7 +20,9 @@ vp.StringParser = util.inherit( PARENT, {
 	 * @param {string} rawValue
 	 */
 	parse: function( rawValue ) {
-		return $.Deferred().resolve( new dv.StringValue( rawValue ) ).promise();
+		return $.Deferred().resolve(
+			rawValue === '' ? null : new dv.StringValue( rawValue )
+		).promise();
 	}
 } );
 
diff --git a/tests/src/valueParsers/parsers/StringParser.tests.js b/tests/src/valueParsers/parsers/StringParser.tests.js
@@ -38,7 +38,8 @@ define( [
 				[ '42', new dv.StringValue( '42' ) ],
 				[ ' foo ', new dv.StringValue( ' foo ' ) ],
 				[ ' Baa', new dv.StringValue( ' Baa' ) ],
-				[ 'xXx ', new dv.StringValue( 'xXx ' ) ]
+				[ 'xXx ', new dv.StringValue( 'xXx ' ) ],
+				[ '', null ]
 			];
 		}
 
