refactor(@nestjs/common): rewrite the ValidationPipe constructor

initialize the properties in a more elegant way
nestjs · Feb 4, 2018 · aaab2bc · aaab2bc
1 parent ff229b5
commit aaab2bc
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/common/pipes/validation.pipe.ts b/src/common/pipes/validation.pipe.ts
@@ -14,12 +14,12 @@ export class ValidationPipe implements PipeTransform<any> {
 
   private returnTransformed: boolean;
 
-  private validatorOptions: ValidatorOptions = {};
+  private validatorOptions: ValidatorOptions;
 
-  constructor(options?: ValidationPipeOptions) {
-    this.returnTransformed = (options && 'transform' in options) ? options.transform : true;
-    if (options) delete options.transform;
-    this.validatorOptions = { ...options } as ValidatorOptions;
+  constructor(options: ValidationPipeOptions = {}) {
+    const { transform, ...validatorOptions } = options;
+    this.returnTransformed = transform != null ? transform : true;
+    this.validatorOptions = validatorOptions;
   }
 
   public async transform(value, metadata: ArgumentMetadata) {