Fix NPE when reviver function encounters a null value. (#2)

Author: kshetline

CHANGELOG.md

@@ -1,3 +1,7 @@
+### 4.1.1
+
+* Fix NPE when reviver function encounters a null value.
+
 ### 4.1.0
 
 * Added a new replacer function feature, `JSONZ.LITERALLY_AS`, allowing a replacer function to explicitly determine how a value will be rendered when stringified.

lib/parse.js

@@ -1197,7 +1197,7 @@ module.exports = function parse(text, reviver, newOptions) {
   function internalize(holder, name, reviver) {
     const value = holder[name];
 
-    if (typeof value === 'object') {
+    if (value && typeof value === 'object') {
       const keys = Object.keys(value);
 
       for (const key of keys) {

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "json-z",
-  "version": "4.1.0",
+  "version": "4.1.1",
   "description": "JSON for everyone.",
   "main": "lib/index.min.js",
   "types": "lib/index.d.ts",

package.json

@@ -556,8 +556,8 @@ describe('JSONZ', () => {
 
 it('parse(text, reviver)', () => {
   expect(
-    JSONZ.parse('{a:1,b:2}', (k, v) => (k === 'a') ? 'revived' : v)).to.deep.equal(
-    { a: 'revived', b: 2 },
+    JSONZ.parse('{a:1,b:2,c:null}', (k, v) => (k === 'a') ? 'revived' : v)).to.deep.equal(
+    { a: 'revived', b: 2, c: null },
     'modifies property values'
   );