Breaking: Convert 'null' and 'undefined' to null

Author: doesdev

README.md

@@ -13,15 +13,17 @@
 > A tiny URL param parser, suitable for server or browser
 
 ## highlights
-- it's tiny (< 20 sloc)
+- it's tiny (< 25 sloc)
 - it's highly browser compatible (IE6 and everything else)
 - it ships with both ES6 and CommonJS modules
-- it's competitively fast (use memoization in front of it to get really fast)
 - it makes repeated keys an array `?a=1&a=2` -> `{ a: [1, 2] }`
 - it handles explicit arrays `?a[]=1` -> `{ a: [1] }`
 - it handles flags (key with no value) `?a=b&flag` -> `{ a: 'b', flag: true }`
 - it decodes `encodeURIComponent` encoded items
 - it converts booleans and numbers from strings
+- it converts the values `'null'` and `'undefined'` to `null`
+- it respects the hash (as per [RFC 3986](https://tools.ietf.org/html/rfc3986#section-3.4))
+- it's competitively fast (not the fastest, not far from it)
 
 ## lowlights
 - it looks like code golf, kind of is

index.js

@@ -2,15 +2,17 @@
 
 function decode (s) { return s.indexOf('%') === -1 ? s : decodeURIComponent(s) }
 function num (v) { return Number.isNaN(+v) ? v : +v }
+function nil (v) { return v === 'null' || v === 'undefined' ? null : v }
 function bool (v) { return v === 'false' ? false : (v === 'true' ? true : v) }
-function conv (v) { return bool(num(v)) }
+function conv (v) { return bool(nil(num(v))) }
 
 module.exports = function (u) {
   var qI, qU, hI
   if (!u || (qI = u.indexOf('?')) === -1 || !(qU = u.slice(qI + 1))) return {}
   if ((hI = qU.indexOf('#')) !== -1 && !(qU = qU.slice(0, hI))) return {}
   var obj = {}
   qU.split('&').forEach(function (q) {
+    if (!q) return
     q = ((q = q.split('=')) && q.length === 2 ? q : [q[0], 'true']).map(decode)
     if (q[0].slice(-2) === '[]') obj[q[0] = q[0].slice(0, -2)] = obj[q[0]] || []
     if (!obj[q[0]]) return (obj[q[0]] = conv(q[1]))

module.mjs

@@ -2,15 +2,17 @@
 
 function decode (s) { return s.indexOf('%') === -1 ? s : decodeURIComponent(s) }
 function num (v) { return Number.isNaN(+v) ? v : +v }
+function nil (v) { return v === 'null' || v === 'undefined' ? null : v }
 function bool (v) { return v === 'false' ? false : (v === 'true' ? true : v) }
-function conv (v) { return bool(num(v)) }
+function conv (v) { return bool(nil(num(v))) }
 
 export default function (u) {
   var qI, qU, hI
   if (!u || (qI = u.indexOf('?')) === -1 || !(qU = u.slice(qI + 1))) return {}
   if ((hI = qU.indexOf('#')) !== -1 && !(qU = qU.slice(0, hI))) return {}
   var obj = {}
   qU.split('&').forEach(function (q) {
+    if (!q) return
     q = ((q = q.split('=')) && q.length === 2 ? q : [q[0], 'true']).map(decode)
     if (q[0].slice(-2) === '[]') obj[q[0] = q[0].slice(0, -2)] = obj[q[0]] || []
     if (!obj[q[0]]) return (obj[q[0]] = conv(q[1]))

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tiny-params",
-  "version": "3.0.1",
+  "version": "4.0.0",
   "description": "A tiny URL param parser, suitable for server or browser",
   "main": "index.js",
   "module": "module.mjs",

test.js

@@ -11,6 +11,8 @@ const q = [
   `zip=37615`,
   `last=carpenter`,
   `zip=37601&q=%3F`,
+  `number=22.55`,
+  `nil=undefined`,
   `flag`,
   `amp=%26`,
   `eq=%3D`,
@@ -25,6 +27,8 @@ runTests('Testing tiny-params CommonJS module', () => {
   test('Encoded question mark is correctly parsed', p.q, '?')
   test('Encoded ampersand is correctly parsed', p.amp, '&')
   test('Encoded equal sign is correctly parsed', p.eq, '=')
+  test('Numeric value parses as such', p.number, 22.55)
+  test('Null and undefined stings parse as null', p.nil, null)
   test('Boolean value parses as such', p.boolval, false)
   test('Flag (key with no value) parses to true', p.flag, true)