Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#5350 closed enhancement (fixed)

Difficult to extend/handle the ext field in cometd

Reported by: gregwilkins Owned by: gregwilkins
Priority: high Milestone:
Component: Dojox Version: 1.0
Keywords: cometd Cc:
Blocked By: Blocking:

Description

The ext field is intended for extensions, but there are no simple extension points that allows easy adding of methods that can fill in or handle ext field.

Some extension methods need to be added

Change History (2)

comment:1 Changed 11 years ago by Greg Wilkins

Resolution: fixed
Status: newclosed

(In [11780]) fixes #5350 with restructure to allow extensions

comment:2 Changed 11 years ago by gregwilkins

I have committed a fix for this that:

+ creates dojox/cometd directory for extensions + moves dojox/_cometd/cometd.js to dojox/cometd/_cometd.js + adds the concepts of dojox.cometd._extendInList and dojox.cometd._extendOutList

to extend incoming and outgoing messages.

+ adds dojox/cometd/timestamp.js as an exemplar of an extension.

Note: See TracTickets for help on using tickets.