Ticket #4994 (closed defect: fixed)

Opened 9 months ago

Last modified 9 months ago

Clicking on a column header leads to exception

Reported by: peller Owned by: sjmiles
Priority: normal Milestone: 1.0
Component: DojoX Grid Version: 0.9
Severity: major Keywords:
Cc: sjmiles, alex

Description (last modified by peller) (diff)

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

Changed 9 months ago by peller

  • cc alex added
  • description modified (diff)
  • summary changed from Resizing a column leads to exception to Clicking on a column header leads to exception

Changed 9 months ago by sjmiles

  • owner changed from sorvell to sjmiles
  • status changed from new to assigned

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

Changed 9 months ago by sjmiles

  • status changed from assigned to closed
  • resolution set to fixed

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

Changed 9 months ago by peller

  • status changed from closed to reopened
  • resolution deleted

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.

Changed 9 months ago by peller

  • status changed from reopened to closed
  • resolution set to fixed
  • milestone changed from 1.0.1 to 1.0

nm. this made it in 1.0. changing milestone.

Note: See TracTickets for help on using tickets.