Opened 14 years ago

Closed 13 years ago

#6351 closed defect (wontfix)

_getRowId() in needs to check for existence of _rowIdentities entry

Reported by: guest Owned by: sorvell
Priority: high Milestone: tbd
Component: DojoX Grid Version: 1.0
Keywords: Cc: [email protected]
Blocked By: Blocking:


When used with an identiy capable store (eg ItemfileWriteStore?) data.DojoData?._getRowId() causes a 'nullpointer' upon updating items in the store that have not been loaded by the Grid yet.

Cause: data.DojoData?._getRowId() does a:

rowId = this._rowIdentities[].rowId;

without prior checking whether the entry in _rowIdentities already exists. This is not the case if the row has not been populated by the Grid yet.


Simply check whether the entry exists in _rowIdenties prior to accessing .rowId.

Something like: rowId = this._rowIdentities[];


rowId = rowId.rowId;

(Sorry for not providing a patch but 1, its such a simple fix 2, someone with more indepth knowledge of the code and understanding of the internals of data.DojoData? will have to check this anyways.



Change History (2)

comment:1 Changed 14 years ago by bill

Milestone: tbd

mark all (open) tickets w/blank milestones to be "tbd"; their milestones need to be set to a version number or to "future"

comment:2 Changed 13 years ago by Bryan Forbes

Resolution: wontfix
Status: newclosed is no longer supported.

Note: See TracTickets for help on using tickets.