Merge pull request #244 from patrickhousley/bugfix/239-constructure-called-twice

19majkel94 · web-flow · commit ec00057e5a65 · 2017-08-15T13:39:06.000+02:00
Controller constructer called twice
diff --git a/src/metadata/ActionMetadata.ts b/src/metadata/ActionMetadata.ts
@@ -257,7 +257,8 @@ export class ActionMetadata {
      * Action method is an action defined in a user controller.
      */
     callMethod(params: any[]) {
-        return this.controllerMetadata.instance[this.method].apply(this.controllerMetadata.instance, params);
+        const controllerInstance = this.controllerMetadata.instance;
+        return controllerInstance[this.method].apply(controllerInstance, params);
     }
 
     // -------------------------------------------------------------------------

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,8 @@ export class ActionMetadata {`
`257`	`257`	`* Action method is an action defined in a user controller.`
`258`	`258`	`*/`
`259`	`259`	`callMethod(params: any[]) {`
`260`		`- return this.controllerMetadata.instance[this.method].apply(this.controllerMetadata.instance, params);`
	`260`	`+ const controllerInstance = this.controllerMetadata.instance;`
	`261`	`+ return controllerInstance[this.method].apply(controllerInstance, params);`
`261`	`262`	`}`
`262`	`263`
`263`	`264`	`// -------------------------------------------------------------------------`