Call service or repository in event dispatcher about express-typescript-boilerplate

[dweber019]

Hi @jasonvillalon

Generally it's fine to do db calls in the event dispatcher.
Use Case: Gathering data for email which will be dispatched over the event dispatcher.

If you provide your use case, we could give you more help.

[jasonvillalon]

Hi @dweber019 thanks for reply.

actually right now im calling database queries in EventSubcriber like this:

@On(events.postActions.created)
    public async onPostActionsCreate(postActions: PostActions): Promise<void> {
        try {
            const connection: Connection = Container.get('connection');
            const em = connection.createEntityManager();
            const post: any = await em.findOne<Posts>(Posts, {
                where: {id: postActions.postId},
                relations: ['user'],
            });
            console.log(post);
        } catch (err) {
            console.log('error', err);
        }
    }

actually i dont like it, is it possible to call service or repository directly here?

my use case is when when post action is created im dispatching email and emitting socket io

Thanks in advance

[dweber019]

@jasonvillalon you could try to use the normal typedi annotation for injecting a service.

constructor(
	private postService: PostService
) { }

Never tested but maybe it works as event-dispatch does some things with a container and a meta registry.

[dweber019]

@jasonvillalon did it work? If you don't mind I close this issue.
We/you can reopen if needed.

[jasonvillalon]

@dweber019 no it didnt work. when im injecting dependencies in subscribers its always undefined.

[jasonvillalon]

@dweber019 can we reopen this? i really need this feature now :) my event dispatcher modules are so bloated now.

[jasonvillalon]

ok i figure out a way to call service inside event dispatcher. i think this will work for me:

const em = connection.createEntityManager();
const myService = new MyService(em.getCustomRepository(MyRepository), log);

is this the only way?

[LogansUA]

@dweber019 It didn't work for me too. Can you reopen it? It's really must have thing.

[LogansUA]

@jasonvillalon Maybe this can help you? Directly getting service from container, instead of building it buy yourself.