Merge pull request #354 from Consdata/IKC-383-event-preview

IKC-383 Event preview
Consdata · May 24, 2024 · 4b52d43 · 4b52d43
2 parents 2333109 + 65caf69
commit 4b52d43
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/kouncil-frontend/libs/common-utils/src/lib/util/object-utils.spec.ts b/kouncil-frontend/libs/common-utils/src/lib/util/object-utils.spec.ts
@@ -22,4 +22,12 @@ describe('ObjectUtils', () => {
       }
     });
   });
+
+  it('should return value when it is null', () => {
+    expect(ObjectUtils.removeNull(null)).toBeNull();
+  });
+
+  it('should return value when it is undefined', () => {
+    expect(ObjectUtils.removeNull(undefined)).toBeUndefined();
+  });
 });
diff --git a/kouncil-frontend/libs/common-utils/src/lib/util/object-utils.ts b/kouncil-frontend/libs/common-utils/src/lib/util/object-utils.ts
@@ -2,13 +2,15 @@ export class ObjectUtils {
 
   // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
   public static removeNull(value: any): string {
-    Object.keys(value).forEach(key => {
-      if (value[key] && typeof value[key] === 'object') {
-        this.removeNull(value[key]);
-      } else if (value[key] === null) {
-        delete value[key];
-      }
-    });
+    if (value) {
+      Object.keys(value).forEach(key => {
+        if (value[key] && typeof value[key] === 'object') {
+          this.removeNull(value[key]);
+        } else if (value[key] === null) {
+          delete value[key];
+        }
+      });
+    }
     return value;
   }
 }