Fixes inversify/InversifyJS/issues/527

inversify · Apr 6, 2017 · bd16e7a · bd16e7a
1 parent ace36cc
commit bd16e7a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "inversify-express-utils",
-  "version": "3.5.0",
+  "version": "3.5.1",
   "description": "Some utilities for the development of express applications with Inversify",
   "main": "lib/index.js",
   "jsnext:main": "es/index.js",
@@ -40,7 +40,7 @@
     "gulp-mocha": "^4.0.1",
     "gulp-tslint": "^7.0.1",
     "gulp-typescript": "^3.1.3",
-    "inversify": "^3.2.0",
+    "inversify": "^3.3.0",
     "mocha": "^3.2.0",
     "publish-please": "^2.2.0",
     "reflect-metadata": "^0.1.8",

diff --git a/src/decorators.ts b/src/decorators.ts
@@ -80,15 +80,14 @@ export function Params(type: PARAMETER_TYPE, parameterName: string) {
         };
         if (!Reflect.hasOwnMetadata(METADATA_KEY.controllerParameter, target.constructor)) {
             parameterMetadataList.unshift(parameterMetadata);
-            metadataList[methodName] = parameterMetadataList;
-            Reflect.defineMetadata(METADATA_KEY.controllerParameter, metadataList, target.constructor);
         } else {
             metadataList = Reflect.getOwnMetadata(METADATA_KEY.controllerParameter, target.constructor);
             if (metadataList.hasOwnProperty(methodName)) {
                 parameterMetadataList = metadataList[methodName];
             }
             parameterMetadataList.unshift(parameterMetadata);
         }
-
+        metadataList[methodName] = parameterMetadataList;
+        Reflect.defineMetadata(METADATA_KEY.controllerParameter, metadataList, target.constructor);
     };
-};
+}