Opened 12 years ago
Closed 9 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.
(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 12 years ago by
Component: | General → DojoX Grid |
---|---|
Owner: | changed from anonymous to Bryan Forbes |
comment:2 Changed 12 years ago by
comment:3 Changed 9 years ago by
Owner: | changed from Bryan Forbes to Phil DeJarnett |
---|---|
Status: | new → pending |
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 9 years ago by
Status: | pending → new |
---|
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 9 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
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.