Merge pull request #20 from storyofams/fix/method-decorators-order

fix: execution order of method decorators

Author: ggurkal

lib/e2e.test.ts

@@ -53,8 +53,8 @@ class TestHandler {
     return { contentType, id, step, redirect, test: this.testField };
   }
 
-  @Post()
   @HttpCode(201)
+  @Post()
   @SetHeader('X-Method', 'create')
   public create(@Header('Content-Type') contentType: string, @Body() body: CreateDto) {
     return { contentType, receivedBody: body, test: this.testField, instanceOf: body instanceof CreateDto };

lib/internals/handler.ts

@@ -42,17 +42,16 @@ export function Handler(method?: HttpVerb): MethodDecorator {
   }
 
   return function (target: object, propertyKey: string | symbol, descriptor: TypedPropertyDescriptor<any>) {
-    const httpCode: number | undefined = Reflect.getMetadata(HTTP_CODE_TOKEN, target.constructor, propertyKey);
-    const metaParameters: Array<MetaParameter> = (
-      Reflect.getMetadata(PARAMETER_TOKEN, target.constructor, propertyKey) ?? []
-    ).sort((a: MetaParameter, b: MetaParameter) => a.index - b.index);
-
     const originalHandler = descriptor.value;
     descriptor.value = async function (req: NextApiRequest, res: NextApiResponse) {
       if (req.method !== method) {
         return notFound(req, res);
       }
 
+      const httpCode: number | undefined = Reflect.getMetadata(HTTP_CODE_TOKEN, target.constructor, propertyKey);
+      const metaParameters: Array<MetaParameter> = (
+        Reflect.getMetadata(PARAMETER_TOKEN, target.constructor, propertyKey) ?? []
+      ).sort((a: MetaParameter, b: MetaParameter) => a.index - b.index);
       const classHeaders: Map<string, string> | undefined = Reflect.getMetadata(HEADER_TOKEN, target.constructor);
       const methodHeaders: Map<string, string> | undefined = Reflect.getMetadata(
         HEADER_TOKEN,