Opened 10 years ago
Closed 5 years ago
#11873 closed defect (patchwelcome)
dojox.socket.LongPoll close() error
Reported by: | liucougar | Owned by: | Kris Zyp |
---|---|---|---|
Priority: | high | Milestone: | 1.13 |
Component: | Dojox | Version: | 1.5 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
deferred in last line of function close() of dojox.socket.LongPoll? is not defined:
fire("close", {wasClean:true}, deferred);
so if close is being called, an error will be thrown. My fix is to remove the deferred argument to fire.
Change History (2)
comment:1 Changed 10 years ago by
comment:2 Changed 5 years ago by
Milestone: | tbd → 1.12 |
---|---|
Resolution: | → patchwelcome |
Status: | new → closed |
Given that no one has shown interest in creating a patch in the past 4+ years, I'm closing this as patchwelcome.
Note: See
TracTickets for help on using
tickets.
Is this fixed? I don't see where deferred would be undefined in the current codebase (it is used just above that line).