Opened 14 years ago

Last modified 13 years ago

#5503 closed enhancement

dojox.grid.data.DojoData markupFactory could use constructorFunction param — at Version 1

Reported by: guest Owned by: alex
Priority: high Milestone: 1.2
Component: DojoX Grid Version: 1.0
Keywords: Cc: [email protected]
Blocked By: Blocking:

Description (last modified by sorvell)

Currently DojoData? class name is hard-coded in its markupFactory. This seems unnecessary and makes inheriting DojoData? class more complicated. There's an easy solution: the parser passes the current class to the markupFactory method. See http://localhost/dojo.ssh/dojox/grid/tests/test_tundra_edit.html for discussion.

Change History (1)

comment:1 Changed 14 years ago by sorvell

Description: modified (diff)
Milestone: 1.11.2
Owner: changed from sorvell to alex

So it looks like changing the markupFctory function in dojox.grid.data.DojoData? will work:

markupFactory: function(args, node, constructorFunction){
  return new constructorFunction(null, null, args);
}

Passing this to Alex to confirm and implement. Thanks.

Note: See TracTickets for help on using tickets.