Opened 10 years ago

Closed 7 years ago

#9181 closed defect (invalid)

dojox.grid.cells.Bool no longer renders as alwaysEditing when declared in markup

Reported by: Phil DeJarnett Owned by: Phil DeJarnett
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.3.0
Keywords: Cc:
Blocked By: Blocking:

Description

There appears to be a bug in the way DataGrid? 1.3 converts a declared structure over.

When testing to see why checkboxes were no longer showing up correctly (they rendered as the text "true" or "false"), I discovered that the inProps mixin for the Bool (on line 40 of dojox/grid/cells/_base.js) is receiving a bunch of undefined values for fields such as "alwaysEditing".

I tried tracking it back, but the incorrect values persisted all the way to the _Widget setting attrs.

I'll look into this some more, to see if I can track it down.

Forum discussion

(Note: there really should be some declarative tests in the grid, since I think many people use it that way.)

Change History (5)

comment:1 Changed 10 years ago by Adam Peller

Component: GeneralDojoX Grid
Owner: changed from anonymous to Bryan Forbes

comment:2 Changed 10 years ago by Phil DeJarnett

Update: apparently, adding alwaysEditing="true" DOES work-around the problem. I must have had a caching issue before.

I'm still uncertain where this is getting set incorrectly.

comment:3 Changed 7 years ago by ben hockey

Owner: changed from Bryan Forbes to Phil DeJarnett
Status: newpending

OverZealous? is there still an issue here? if so, could you attach a test case? if there is no response within 14 days, this issue will automatically close.

comment:4 Changed 7 years ago by Phil DeJarnett

Status: pendingnew

It's been so long since I wrote this, I don't even know where the code was that this problem is from. Since it's from 4 minor versions ago (1.3), I'd just go ahead and close it.

comment:5 Changed 7 years ago by Adam Peller

Resolution: invalid
Status: newclosed
Note: See TracTickets for help on using tickets.