Opened 12 years ago

Closed 12 years ago

#4994 closed defect (fixed)

Clicking on a column header leads to exception

Reported by: Adam Peller Owned by: sjmiles
Priority: high Milestone: 1.0
Component: DojoX Grid Version: 0.9
Keywords: Cc: sjmiles, alex
Blocked By: Blocking:

Description (last modified by Adam Peller)

dojox/grid/tests/test_edit_dijit.html

Click on a header like "Mark". This leads to an exception on FF2:

151 var ineq = inCompare(a[inField], b[inField]);

call stack

generateComparator(["note", false, "replied", 5 more...], ["normal", false, "new", 5 more...])../../../dojo/../... (line 151)
sort()../../../dojo/../... (line 242)
sort()../../../dojo/../... (line 124)
setSortInfo(4)../../../dojo/../... (line 510)
setSortIndex(3, undefined)../../../dojo/../... (line 505)
onHeaderCellClick(click clientX=407, clientY=134)../../../dojo/../... (line 234)
doheaderclick(click clientX=407, clientY=134)../../../dojo/../... (line 561)
_dispatch("doheaderclick", click clientX=407, clientY=134)../../../dojo/../... (line 523)
dispatchHeaderEvent(click clientX=407, clientY=134)../../../dojo/../... (line 533)
onHeaderEvent(click clientX=407, clientY=134)../../../dojo/../... (line 19)
doHeaderEvent(click clientX=407, clientY=134)../../../dojo/../... (line 235)
(?)()

Change History (5)

comment:1 Changed 12 years ago by Adam Peller

Cc: alex added
Description: modified (diff)
Summary: Resizing a column leads to exceptionClicking on a column header leads to exception

comment:2 Changed 12 years ago by sjmiles

Owner: changed from sorvell to sjmiles
Status: newassigned

I've been working on the sorting algorithm, so it's likely I just broke this. Will check it out asap.

comment:3 Changed 12 years ago by sjmiles

Resolution: fixed
Status: assignedclosed

(In [11346]) Fix typo in model. Add code to prevent bogus click events after column resize on FF. Fixes #4994.

comment:4 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: closedreopened

This went in the trunk for 1.1. If it belongs in 1.0.1 it needs to go in the new 1.0 branch, else change the milestone. Thanks.

comment:5 Changed 12 years ago by Adam Peller

Milestone: 1.0.11.0
Resolution: fixed
Status: reopenedclosed

nm. this made it in 1.0. changing milestone.

Note: See TracTickets for help on using tickets.