Opened 12 years ago

Closed 12 years ago

#4946 closed defect (fixed)

Cometd does not load on IE, extra comma

Reported by: guest Owned by: Jared Jurkiewicz
Priority: high Milestone:
Component: General Version: 1.0
Keywords: Cc: marknesb@…
Blocked By: Blocking:

Description

Extra comma in cometd.js breaks IE.

Line 613:

this.openTunnelWith({

607 message: dojo.toJson([ 608 { 609 channel: "/meta/connect", 610 connectionType: "callback-polling", 611 clientId: this._cometd.clientId, 612 timestamp: this.lastTimestamp, 613 id: this._cometd.messageId++, 614 } 615 ]) 616 }); 617 }

Attachments (1)

dojox._cometd.cometd_20071030.patch​ (385 bytes) - added by Jared Jurkiewicz 12 years ago.
Minor patch to cometd.js

Download all attachments as: .zip

Change History (3)

comment:1 Changed 12 years ago by Jared Jurkiewicz

Owner: changed from anonymous to Jared Jurkiewicz

I'll fix this. Trivial update.

Changed 12 years ago by Jared Jurkiewicz

Attachment: dojox._cometd.cometd_20071030.patch​ added

Minor patch to cometd.js

comment:2 Changed 12 years ago by Jared Jurkiewicz

Resolution: → fixed
Status: new → closed

(In [11232]) Removing tailing comma. fixes #4946

Note: See TracTickets for help on using tickets.