Ticket #5054 (closed defect: fixed)

Opened 14 months ago

Last modified 14 months ago

Grid c.editor has no properties after trying to resize a header

Reported by: guest Owned by: sorvell
Priority: high Milestone: 1.0.1
Component: DojoX Grid Version: 1.0
Severity: critical Keywords: Grid c.editor resize header
Cc:

Description

I do not know how you rate your severity levels, but this problem causes the dojo grid to error out and, to the user of the grid, appears to "destroy" their grid data. I flagged it critical because of this.

You can use your test located here as an example of this problem: http://download.dojotoolkit.org/release-1.0.0/dojo-release-1.0.0/dojox/grid/tests/test_events.html

You can recreate the error "c.editor has no properties" when attempting to resize a grid header after double clicking on a row. I have had this issue when reloading data from a datastore, but had no way to give you an example without giving you my code until I found this issue also resides when resizing a header and not just re setting the grid model.

Steps to recreate: 1) Double click on row 2 (any row will work, but 2 is this example) 2) Make sure that the double click highlighted the row "green" as if it was going to be edited 3) Resize any of the headers

The error should show in firebug as "c.editor has no properties". In addition to the error, everything after the selection you made will not render in the grid and it will appear to be missing everything past that row.

Change History

Changed 14 months ago by sjmiles

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

[11459]

Grid should not put editor-less cells in edit mode. Also (likely redundant) protection against referencing null c.editor. fixes #5054

Changed 14 months ago by peller

  • status changed from closed to reopened
  • resolution deleted

has to go in 1.0 branch as well

Changed 14 months ago by peller

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

Fixed on 1.0 branch also: [11460]

Changed 14 months ago by peller

oops. meant [11461]

Changed 14 months ago by peller

(In [11468]) mirror checkin [11460] in 1.0 branch. Refs #5054

Note: See TracTickets for help on using tickets.