Opened 9 years ago

Closed 8 years ago

#13795 closed defect (patchwelcome)

dojox.grid.DataGrid does not implement set() API across the board

Reported by: Karl Tiedt Owned by: Evan
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.6.1
Keywords: grid setStore set('store') Cc:
Blocked By: Blocking:

Description

DataGrid? never implemented _setStoreAttr so... this example fails to set store2 properly AFTER startup has been called

​http://jsfiddle.net/K5VtK/3/

(Ran into some confusing issues with a user in #dojo today regarding this)

Change History (4)

comment:1 Changed 9 years ago by Karl Tiedt

Also worth noting... (the users use case) -- They were calling grid.set('store', store) in postCreate() which worked by virtue of it setting the property prior to it being used by grid... however if they swapped this out with setStore() in postCreate it throws an exception... I informed them this is likely not to get fixed, but is it worth throwing a warning or deferring setStore() until after .startup() to avoid this problem?

comment:2 Changed 9 years ago by Adam Peller

Component: General → DojoX Grid
Owner: set to Evan

comment:3 Changed 8 years ago by bill

DojoX Grid and EnhancedGrid are deprecated in favor of ​dgrid and ​gridx.

You should upgrade your code to use one of those two grids.

We will consider patches to the old DojoX Grid code though.

comment:4 Changed 8 years ago by bill

Resolution: → patchwelcome
Status: new → closed
Note: See TracTickets for help on using tickets.