Merge pull request #3 from aha-app/ACT-31-fix-double-call-destroy-applicationcontroller

ACT-31: Fix double call to destroy in ApplicationController

Author: jemmyw

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aha-app/mvc",
-  "version": "0.11.1",
+  "version": "0.11.2",
   "description": "Simple MVC framework using React for the view and GraphQL for the models.",
   "main": "dist/index.js",
   "type": "module",

src/controller/ApplicationController.tsx

@@ -146,6 +146,10 @@ class ApplicationController<
    */
   destroy() {}
 
+  /**
+   * Internal destroy function. Do not override
+   * @private
+   */
   internalDestroy() {
     this.destroy();
   }
@@ -326,7 +330,6 @@ function Controller<Props = {}>({
   useEffect(() => {
     return () => {
       debug('Destroying controller');
-      if (controller.destroy) controller.destroy();
       controller.internalDestroy();
     };
   }, [controller]);