ajacksified / ajacksified/Mediator.js

Difference in removing all / single subscriber

Aperta
#30 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
460
Fork
75
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

I was looking at the `removeSubscriber` method of the `Channel` object and I noticed something that I can't explain.

When removing a specific subscriber you set the `channel` property of the subscriber to `null` before removing it from the array of subscribers. When all the subscribers get cleared due to the `id` param being `null` the array is simply cleared and the `channel` property on all the subscribers don't get reset to `null`.

Is there a specific reason for this or is it an oversight?

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.