-
Notifications
You must be signed in to change notification settings - Fork 124
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
retain the default listeners for future unsubscribing #5781
Conversation
|
||
## | ||
# @return Array[Object] the listeners Hyrax subscribes by default. | ||
def default_listeners |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about adding a simple test since this is a public thing which shouldn't go away without red flags?
Maybe something ensuring it returns an array?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests are in. thanks!
37f73d6
to
141d96d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
subscribe the hyrax default listeners in batch. provide a hook for looking up the `Hyrax.publisher.default_listeners`. this makes it possible to unsubscribe individual listeners at a later time. these instances have to live in memory for the life of the Publisher anyway, so this shouldn't impact when they are freed for garbage collection.
141d96d
to
5af0f63
Compare
this one had fallen way behind, but i just rebased it and will follow through on any issues in the tests. |
subscribe the hyrax default listeners in batch. provide a hook for looking up
the
Hyrax.publisher.default_listeners
. this makes it possible to unsubscribeindividual listeners at a later time.
these instances have to live in memory for the life of the Publisher anyway, so
this shouldn't impact when they are freed for garbage collection.
@samvera/hyrax-code-reviewers