merge: release 1.0.1 (#17)

Author: NoNameProvided

CHANGELOG.md

@@ -2,12 +2,18 @@
 
 _This changelog follows the [keep a changelog][keep-a-changelog]_ format to maintain a human readable changelog.
 
-## [1.0.0][v1.0.0] - 2020-07-31 [BREAKING CHANGE]
+## [1.0.1][v1.0.1] - 2020-07-31
 
 > This is the **final release of the library**, if you lack some functionality please use [class-transformer][ct] and [class-validator][cv] instead.
 
 #### Fixed
 
+- sanitization won't skip properties with empty string values anymore (`''`)
+
+## [1.0.0][v1.0.0] - 2020-07-31 [BREAKING CHANGE]
+
+#### Fixed
+
 - passing `{ each: true }` into decorators don't crashes the library anymore
 - fixed security warning due to out-of-date dev dependencies
 
@@ -50,6 +56,7 @@ _This changelog follows the [keep a changelog][keep-a-changelog]_ format to main
 
 Initial version.
 
+[v1.0.0]: https://github.com/typestack/class-sanitizer/compare/v1.0.0...v1.0.1
 [v1.0.0]: https://github.com/typestack/class-sanitizer/compare/v0.0.5...v1.0.0
 [v0.0.5]: https://github.com/typestack/class-sanitizer/compare/v0.0.4...v0.0.5
 [v0.0.4]: https://github.com/typestack/class-sanitizer/compare/v0.0.3...v0.0.4

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "class-sanitizer",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Decorator based class property sanitation in Typescript.",
   "license": "MIT",
   "author": {

package.json

@@ -120,7 +120,7 @@ export class Sanitizer {
   public sanitize<T = Record<string, any>>(classInstance: InstanceType<any>): T {
     this.metadataStorage
       .getSanitizeMetadatasForClassInstance(classInstance)
-      .filter(metadata => !!classInstance[metadata.propertyName])
+      .filter(mt => classInstance[mt.propertyName] !== undefined && classInstance[mt.propertyName] !== null)
       .forEach(metadata => {
         /** If `each` is set we validate the values of the array.  */
         if (metadata.each) {

src/sanitizer.class.ts

@@ -1,5 +1,5 @@
 import { defaultMetadataStorage } from '../src/default-storage.const';
-import { Trim, Ltrim, Rtrim, SanitizeNested } from '../src/decorators';
+import { Trim, Ltrim, Rtrim, ToInt, SanitizeNested } from '../src/decorators';
 import { sanitize, Sanitizer, sanitizeAsync } from '../src';
 
 describe('Basic Functionality', () => {
@@ -109,4 +109,28 @@ describe('Basic Functionality', () => {
 
     expect(result).toEqual(input);
   });
+
+  it(`should skip null and undefined properties but not ''`, () => {
+    class TestClass {
+      @ToInt()
+      emptyStringValue: string;
+
+      @ToInt()
+      undefinedValue: string;
+
+      @ToInt()
+      nullValue: string;
+    }
+
+    const instance = new TestClass();
+    instance.emptyStringValue = '';
+    instance.undefinedValue = undefined;
+    instance.nullValue = null;
+
+    sanitize(instance);
+
+    expect(instance.emptyStringValue).toBeNaN();
+    expect(instance.undefinedValue).toBeUndefined();
+    expect(instance.nullValue).toBeNull();
+  });
 });