plain-object-merge - npm Package Compare versions

Comparing version 1.0.0 to 1.0.1

dist/index.js

@@ -17,3 +17,5 @@ "use strict";
         if (isPrimitive(value)) {
-            target[key] = value;
+            if (value !== undefined || !(key in target)) {
+                target[key] = value;
+            }
         }
@@ -20,0 +22,0 @@ else if (!target[key] || Array.isArray(value)) {

package.json

 {
   "name": "plain-object-merge",
   "description": "Extremely fast function optimized for deep merging json-serializable plain objects.",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "main": "dist/index.js",
@@ -6,0 +6,0 @@ "types": "dist/index.d.ts",

src/index.ts

@@ -31,4 +31,8 @@
 
-      target[key] = value;
+      if ( value !== undefined || !(key in target) ) {
 
+        target[key] = value;
+
+      }
+
     } else if ( !target[key] || Array.isArray ( value ) ) {
@@ -35,0 +39,0 @@

test/fixtures.js

@@ -21,3 +21,3 @@
       null: null,
-      undefined: undefined,
+      undefined: 123,
       arr: ['1', '2', '3'],
@@ -24,0 +24,0 @@ deep: {

test/index.js

@@ -12,3 +12,3 @@
 
-  it ( 'works', t => {
+  it ( 'can merge complext objects', t => {
 
@@ -19,2 +19,12 @@ t.deepEqual ( merge ( Fixtures.parts ), Fixtures.result );
 
+  it ( 'supports undefined', t => {
+
+    t.deepEqual ( merge ([ {}, { foo: undefined } ]), { foo: undefined } );
+
+    t.deepEqual ( merge ([ { foo: 0 }, { foo: undefined } ]), { foo: 0 } );
+
+    t.deepEqual ( merge ([ { foo: {} }, { foo: undefined } ]), { foo: {} } );
+
+  });
+
 });

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

8174

increased by6.63%

Number of package files

12

increased by9.09%

Lines of code

202

increased by4.66%

No dependency changes