Opened 9 years ago

Closed 4 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 8 years ago by Kris Zyp

Is this fixed? I don't see where deferred would be undefined in the current codebase (it is used just above that line).

comment:2 Changed 4 years ago by dylan

Milestone: tbd1.12
Resolution: patchwelcome
Status: newclosed

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.