Skip to content
This repository was archived by the owner on Apr 20, 2018. It is now read-only.

Use EventEmitter.removeListener instead of the nonexistent 'off' method#52

Closed
duncanmak wants to merge 1 commit intoReactive-Extensions:masterfrom
duncanmak:fix-remove-listener
Closed

Use EventEmitter.removeListener instead of the nonexistent 'off' method#52
duncanmak wants to merge 1 commit intoReactive-Extensions:masterfrom
duncanmak:fix-remove-listener

Conversation

@duncanmak
Copy link

method.

Without this patch, you get an error that looks like this:

TypeError: Object # has no method 'off'
at null.action (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.node.js:145:24)
at Disposable.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:612:18)
at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)
at AutoDetachObserverPrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:4668:20)
at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)
at AutoDetachObserverPrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:4668:20)
at CompositeDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:554:39)
at null.action (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.binding.js:524:49)
at Disposable.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:612:18)
at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)

method.

Without this patch, you get an error that looks like this:

TypeError: Object #<ReadStream> has no method 'off'
    at null.action (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.node.js:145:24)
    at Disposable.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:612:18)
    at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)
    at AutoDetachObserverPrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:4668:20)
    at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)
    at AutoDetachObserverPrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:4668:20)
    at CompositeDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:554:39)
    at null.action (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.binding.js:524:49)
    at Disposable.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:612:18)
    at SingleAssignmentDisposablePrototype.dispose (/Users/duncan/git/duncanmak/test-project/node_modules/rx/rx.js:701:17)
mattpodwysocki added a commit that referenced this pull request Oct 22, 2013
@mattpodwysocki
Copy link
Member

Fixed as of 7265876

@mattpodwysocki
Copy link
Member

Thanks for the PR, I just realized how many other people had noticed the same thing. ugh!

@duncanmak duncanmak deleted the fix-remove-listener branch October 30, 2013 02:46
bouzuya pushed a commit to bouzuya/RxJS that referenced this pull request Mar 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments