OnEvent decorator does not triggered if list of events provided about event-emitter

[kamilmysliwiec]

Would you like to create a PR for this issue?

[benkroeger]

You can not use the OnEvent decorator to subscribe to multiple events at once (except for when you use wildcards.
The event argument from OnEvent decorator is passed to the EventEmitter2.on() method - which treats the event argument of type Array as a namespaced event (i.e. 'foo.bar' equals ['foo', 'bar']) or in this case (@Kopleman ) ['some.event1', 'some.event2'] would subscribe to the event 'some.event1.some.event2' - which I assume would never be emitted.

Note: For using wildcards or namespaces, you must activate the wildcard option on the module.

Namespaces with Wildcards: To use namespaces/wildcards, pass the wildcard option into the EventEmitter constructor. When namespaces/wildcards are enabled, events can either be strings (foo.bar) separated by a delimiter or arrays (['foo', 'bar']). The delimiter is also configurable as a constructor option.

[micalevisk]

if we change from
@OnEvent(['some.event1', 'some.event2']) to
@OnEvent(['some.event1'])
it will work. Should be easy to fix

[inceenes10]

In the example below doesn't work

@Injectable()
export class Indexer {

    @OnEvent("auction.created")
    created() {
        console.log("auction created 2")
    }
}

but this work as expected in my nest app

@Injectable()
export class Indexer {


    constructor(
        private readonly auctionContract: AuctionContract,
        private readonly eventEmitter: EventEmitter2
    ) {
        this.eventEmitter.addListener("auction.created", this.created.bind(this))
    }

    created() {
        console.log("auction created 2")
    }
}

I don't know why the first approach is not working. But I'd like to create a PR for this. I don't know where to start.