Skip to content

Handle empty events array#32

Open
jasonkarns wants to merge 1 commit intoccampbell:masterfrom
jasonkarns:off-guard
Open

Handle empty events array#32
jasonkarns wants to merge 1 commit intoccampbell:masterfrom
jasonkarns:off-guard

Conversation

@jasonkarns
Copy link

It's common to pre-emptively call off with an event and handler prior
to any handlers being registered. This is a safeguard to ensure that the
same handler isn't attached multiple times. In this scenario, the events
array isn't checked for existence prior to the accessor call (for the
given selector). Thus a TypeError is raised.

Closes #31

It's common to pre-emptively call `off` with an event and handler prior
to any handlers being registered. This is a safeguard to ensure that the
same handler isn't attached multiple times. In this scenario, the events
array isn't checked for existence prior to the accessor call (for the
given selector). Thus a TypeError is raised.

Closes ccampbell#31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant