The dojox.layout.GridContainer's default constructor does not check for null or undefined properties
UseCase? Extend the GridContainer? and construct it without any properties. This is a valid use case since the properties will be added later.
Result: An error is thrown to the effect that props has no values.
Examining the constructor showed that it was checking the properties on the assumption they would exist or be defined. This logic is brittle, since there may well be a case when the properties are not defined. In this case, the default properties should be assigned.
Changed the code to handle undefined properties in the constructor.
Attachments (1)
-
GridContainer.js (31.0 KB) - added by spidey2099 12 years ago.
-
Changed the grid container's constructor. Please see comment : 6/27/2009 JB: Changed to handle undefined props
Download all attachments as: .zip
Change History (5)
Component: |
General →
Dojox
|
Owner: |
changed from anonymous to dante
|
Milestone: |
tbd →
1.4
|
Status: |
new →
assigned
|
Resolution: |
→ fixed
|
Status: |
assigned →
closed
|
Component: |
Dojox →
DojoX Layout
|
Changed the grid container's constructor. Please see comment : 6/27/2009 JB: Changed to handle undefined props