fix(core): do not fire onInit event twice

Closes #900
mikro-orm · Oct 5, 2020 · 9485f48 · 9485f48
1 parent 5612759
commit 9485f48
Showing 1 changed file with 1 addition and 12 deletions.
diff --git a/packages/core/src/entity/EntityFactory.ts b/packages/core/src/entity/EntityFactory.ts
@@ -53,7 +53,7 @@ export class EntityFactory {
       this.unitOfWork.registerManaged(entity, data, options.refresh && options.initialized, options.newEntity);
     }
 
-    this.runHooks(entity, meta);
+    this.eventManager.dispatchEvent(EventType.onInit, { entity, em: this.em });
 
     return entity as New<T, P>;
   }
@@ -189,15 +189,4 @@ export class EntityFactory {
     });
   }
 
-  private runHooks<T>(entity: T, meta: EntityMetadata<T>): void {
-    /* istanbul ignore next */
-    const hooks = meta.hooks?.onInit || [];
-
-    if (hooks.length > 0) {
-      hooks.forEach(hook => (entity[hook] as unknown as () => void)());
-    }
-
-    this.eventManager.dispatchEvent(EventType.onInit, { entity, em: this.em });
-  }
-
 }