Merge pull request #25 from arshadkazmi42/handler-for-not-stirng

Add support for handling non string values in json

Author: arshadkazmi42

index.js

@@ -31,14 +31,15 @@ const jnestedReplace = (input, searchValue, newValue, skipKeys=[]) => {
     // for every element of array
     if (isArray(input[key])) {
       for (let i=0; i<input[key].length; i++) {
+
         input[key][i] = jnestedReplace(input, searchValue, newValue, skipKeys);
       }
       continue;
     }
 
     // If the key needs to be skipped.
     // Do not process and continue to next element
-    if (skipKeys.indexOf(key) === -1) {
+    if (skipKeys.indexOf(key) === -1 && isString(input[key])) {
       input[key] = input[key].replace(searchValue, newValue);
     }
   }
@@ -57,5 +58,11 @@ const isArray = (data) => {
   return data instanceof Array;
 };
 
+// check id the data is string
+const isString = (data) => {  
+  return typeof data === 'string' 
+    && Object.prototype.toString.call(data) === '[object String]';
+};
+
 
 module.exports = jnestedReplace;

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "json-nested-replace",
-  "version": "1.0.4",
+  "version": "1.1.0",
   "description": "Searches and replace values at every level of nested json ",
   "main": "index.js",
   "scripts": {

package.json

@@ -101,4 +101,34 @@ describe('replace in nested json', () => {
     expect(replacedValue.repository.url).to.equal(INPUT_JSON.repository.url);
     expect(replacedValue.name).to.be.equal('jnested-replace');
   });
+  it('show skip the values which are not string', () => {
+    const input = {
+      'name': 'json-nested-replace',
+      'author': 'Arshad Kazmi',
+      'number': 1234,
+      'boolean': true,
+      'repository': {
+        'url': 'https://github.com/arshadkazmi42/json-nested-replace',
+        'language': 'js',
+        'numberString': '1234',
+        'booleanString': 'true'
+      }
+    };
+
+    const expected = {
+      'name': 'json-nested-replace',
+      'author': 'Arshad Kazmi',
+      'number': 1234,
+      'boolean': true,
+      'repository': {
+        'url': 'https://github.com/arshadkazmi42/json-nested-replace',
+        'language': 'js',
+        'numberString': '4321',
+        'booleanString': 'true'
+      }
+    };
+
+    const updatedInput = jnestedReplace(input, '1234', '4321');
+    expect(updatedInput).to.be.deep.equals(expected);
+  });
 });