Ticket #4994 (closed defect: fixed)

Opened 20 months ago

Last modified 20 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 20 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 20 months ago by sjmiles

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

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

Changed 20 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 20 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 20 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.