Merge pull request #591 from eugenio165/bugfix/double-decorators

fix(crud): Custom route decorators applied twice
nestjsx · Aug 18, 2020 · 77bb83a · 77bb83a
2 parents 644acfd + d905b66
commit 77bb83a
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/packages/crud/src/crud/reflection.helper.ts b/packages/crud/src/crud/reflection.helper.ts
@@ -84,25 +84,16 @@ export class R {
     target: object,
     name: string,
   ) {
-    // this makes proxy decorator works
-    Reflect.defineProperty(
-      target,
-      name,
-      Reflect.decorate(
-        decorators,
-        target,
-        name,
-        Reflect.getOwnPropertyDescriptor(target, name),
-      ),
-    );
-
     // this makes metadata decorator works
-    Reflect.decorate(
+    const decoratedDescriptor = Reflect.decorate(
       decorators,
       target,
       name,
       Reflect.getOwnPropertyDescriptor(target, name),
     );
+
+    // this makes proxy decorator works
+    Reflect.defineProperty(target, name, decoratedDescriptor);
   }
 
   static setParsedRequestArg(index: number) {