Opened 9 years ago

Closed 7 years ago

Last modified 7 years ago

#11572 closed defect (fixed)

DataGrid generates runtime errors when you delete rows in a particular order

Reported by: philip Owned by: Evan
Priority: high Milestone: 1.7.4
Component: DojoX Grid Version: 1.5
Keywords: Cc:
Blocked By: Blocking:

Description

I found that if you delete grid items in a particular order, you can get Javascript errors. Firebug reports the error as,

this._by_idx[_4bf] is undefined

I think this is coming from the _onDelete method of DataGrid?. I'm attaching a test case that shows the error. A grid with 2 rows is created. When you click on the checkbox, the row is deleted. If you delete the second row and then the first row, you get an error. If you do it in the other direction, everything is fine.

Attachments (1)

r1.html (2.3 KB) - added by philip 9 years ago.
Example that shows DataGrid? deletion bug.

Download all attachments as: .zip

Change History (9)

Changed 9 years ago by philip

Attachment: r1.html added

Example that shows DataGrid? deletion bug.

comment:1 Changed 8 years ago by Adam Peller

Component: GeneralDojoX Grid
Owner: changed from anonymous to bryanforbes

comment:2 Changed 8 years ago by bill

Owner: changed from bryanforbes to Bryan Forbes

comment:3 Changed 8 years ago by evan

Milestone: tbd1.7
Priority: normalhigh

comment:4 Changed 8 years ago by Adam Peller

Owner: changed from Bryan Forbes to Evan

comment:5 Changed 8 years ago by Chris Mitchell

Priority: highnormal

not blocker for 1.7rc1, lowering priority

comment:6 Changed 7 years ago by Evan

Resolution: fixed
Status: newclosed

In [29251]:

Fixes #11572 !strict, do not apply edit if the row does not exist anymore, also back port to 1.7.x branch

comment:7 Changed 7 years ago by Evan

Milestone: 1.81.7.4

comment:8 Changed 7 years ago by Evan

In [29290]:

Refs #11572 !strict, fix regression, a better way to check cell node

Note: See TracTickets for help on using tickets.