Merge branch 'feature/empty-array' into main

Author: oleg-yasenytsky

README.md

@@ -1,4 +1,4 @@
-# Js object utils
+# Js object utils [![NPM Module](https://img.shields.io/npm/v/@idapgroup/js-object-utils.svg)](https://www.npmjs.com/package/@idapgroup/js-object-utils)
 
 Javascript functions for transform or mutate object
 
@@ -49,7 +49,7 @@ const options = {
   booleanMapper: (val: boolean) => (val ? '1' : '0'),
 
   /**
-   * allow empty string
+   * allow empty string or array
    * defaults to false
    */
   allowEmptyValues: false,

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@idapgroup/js-object-utils",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "description": "Javascript functions for transform or mutate object",
   "main": "build/main/index.js",
   "typings": "build/main/index.d.ts",

src/lib/create-form-data.ts

@@ -18,7 +18,7 @@ const fillFormData = (
     index,
     booleanMapper,
     allowNullableValues,
-    allowEmptyValues,
+    allowEmptyValues
   } = config;
 
   if (value === undefined || value === null) {
@@ -30,13 +30,17 @@ const fillFormData = (
   } else if (value instanceof Date) {
     formData.append(keyPrefix, value.toISOString());
   } else if (Array.isArray(value)) {
-    value.forEach((item, index) => {
-      fillFormData(
-        item,
-        { ...config, keyPrefix: `${keyPrefix}[${index}]` },
-        formData
-      );
-    });
+    if (value.length === 0 && allowEmptyValues) {
+      formData.append(`${keyPrefix}[]`, '');
+    } else {
+      value.forEach((item, index) => {
+        fillFormData(
+          item,
+          { ...config, keyPrefix: `${keyPrefix}[${index}]` },
+          formData
+        );
+      });
+    }
   } else if (typeof value === 'object') {
     if (value instanceof File || value instanceof Blob) {
       formData.append(
@@ -77,7 +81,7 @@ export const createFormData = (
       index: null,
       booleanMapper: (val: boolean) => (val ? '1' : '0'),
       allowNullableValues: false,
-      allowEmptyValues: false,
+      allowEmptyValues: false
     },
     options || {}
   );

test/create form-data.spec.ts

@@ -137,10 +137,12 @@ describe('createFormData function', () => {
     expect(fd).to.have.been.equal(initial);
   });
   it('should be success with empty values', () => {
-    const fd = createFormData({name: ''}, {allowEmptyValues: true});
+    const fd = createFormData({name: '', products: []}, {allowEmptyValues: true});
     expect(fd.get(`name`)).to.have.been.equal('');
-    const fd2 = createFormData({name: ''}, {allowEmptyValues: false});
+    expect(fd.get(`products[]`)).to.have.been.equal('');
+    const fd2 = createFormData({name: '', products: []}, {allowEmptyValues: false});
     expect(fd2.has(`name`)).to.have.been.false;
+    expect(fd2.has(`products[]`)).to.have.been.false;
     const fd3 = createFormData({name: '', email: null}, {allowEmptyValues: true, allowNullableValues: true});
     expect(fd3.get(`name`)).to.have.been.equal('');
     expect(fd3.get(`email`)).to.have.been.equal('');