qs-kit - npm Package Compare versions

Comparing version 0.6.2 to 0.6.3

lib/qs.js

@@ -116,3 +116,8 @@ /*
 
-			if ( options.autoNumber && numberRe.test( value ) ) { value = parseFloat( value ) ; }
+			if ( options.autoNumber && numberRe.test( value ) ) {
+				value = parseFloat( value ) ;
+			}
+			else {
+				value = decodeURIComponent( value ) ;
+			}
 
@@ -119,0 +124,0 @@ indexOfDotDollar = key.lastIndexOf( '.$' ) ;

package.json

 {
   "name": "qs-kit",
-  "version": "0.6.2",
+  "version": "0.6.3",
   "description": "A query string manipulation toolbox.",
@@ -5,0 +5,0 @@ "main": "lib/qs.js",

test/qs-test.js

@@ -97,2 +97,3 @@ /*
 		var options = { restQueryFilter: true , autoNumber: true } ;
+		expect( qs.parse( ".prop=value" , options ) ).to.equal( { prop: "value" } ) ;
 		expect( qs.parse( ".path.to.prop=value" , options ) ).to.equal( { "path.to.prop": "value" } ) ;
@@ -104,4 +105,7 @@ expect( qs.parse( ".path.to.prop.$lt=10" , options ) ).to.equal( { "path.to.prop": { $lt: 10 } } ) ;
 
-	it( "'restQueryFilter' option with a string should nest those filters" , () => {
-		var options = { restQueryFilter: 'filter' , autoNumber: true } ;
+	it( "'restQueryFilter' option set to a string should use it as the property which the filters will be nested into" , () => {
+		var value , encoded ,
+			options = { restQueryFilter: 'filter' , autoNumber: true } ;
+		
+		expect( qs.parse( ".prop=value" , options ) ).to.equal( { filter: { prop: "value" } } ) ;
 		expect( qs.parse( ".path.to.prop=value" , options ) ).to.equal( { filter: { "path.to.prop": "value" } } ) ;
@@ -111,2 +115,17 @@ expect( qs.parse( ".path.to.prop.$lt=10" , options ) ).to.equal( { filter: { "path.to.prop": { $lt: 10 } } } ) ;
 		expect( qs.parse( ".path.to.prop.$lt=10&.path.to.prop.$gt=5&.path.to.prop2.$ne=10" , options ) ).to.equal( { filter: { "path.to.prop": { $lt: 10 , $gt: 5 } , "path.to.prop2": { $ne: 10 } } } ) ;
+		
+		value = "éµ%!:&=?#«»" ;
+		encoded = encodeURIComponent( value ) ;
+		expect( qs.parse( ".prop=" + encoded , options ) ).to.equal( { filter: { prop: value } } ) ;
+		expect( qs.parse( ".path.to.prop=" + encoded , options ) ).to.equal( { filter: { "path.to.prop": value } } ) ;
+
+		value = "إختبار" ;
+		encoded = encodeURIComponent( value ) ;
+		expect( qs.parse( ".prop=" + encoded , options ) ).to.equal( { filter: { prop: value } } ) ;
+		expect( qs.parse( ".path.to.prop=" + encoded , options ) ).to.equal( { filter: { "path.to.prop": value } } ) ;
+
+		value = "测试" ;
+		encoded = encodeURIComponent( value ) ;
+		expect( qs.parse( ".prop=" + encoded , options ) ).to.equal( { filter: { prop: value } } ) ;
+		expect( qs.parse( ".path.to.prop=" + encoded , options ) ).to.equal( { filter: { "path.to.prop": value } } ) ;
 	} ) ;
@@ -130,2 +149,14 @@ } ) ;
 
+describe( "encodeURIComponent" , () => {
+
+	it( "should encode all QS Kit special characters" , () => {
+		expect( encodeURIComponent( "=" ) ).to.be( "%3D" ) ;
+		expect( encodeURIComponent( "&" ) ).to.be( "%26" ) ;
+		expect( encodeURIComponent( "[]" ) ).to.be( "%5B%5D" ) ;
+		expect( encodeURIComponent( "," ) ).to.be( "%2C" ) ;
+	} ) ;
+} ) ;
+
+
+
 describe( "Historical bugs" , () => {
@@ -132,0 +163,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

25270

increased by6.22%

Lines of code

436

increased by6.86%

No dependency changes