Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#9146 closed defect (fixed)

Error in gfx utils.js deserialize

Reported by: chris de jong Owned by: Eugene Lazutkin
Priority: high Milestone: 1.3.1
Component: DojoX GFX Version: 1.3.0
Keywords: gfx utils.js deserialize Cc: chris@…
Blocked By: Blocking:

Description

The deserialize function in dojox.gfx.utils calls gu.serialize in the case of an array. Should be gu.deserialize.

corrected code: if(object instanceof Array){ return d.map(object, d.hitch(null, gu.deserialize, parent)); Array

}

browser Firefox 3.0.8 Os Windows XP mail chris@…

The attached file will demonstrate the malfunction (it will show a blank page without the fix).

Attachments (1)

bugreport.html (2.2 KB) - added by chris de jong 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by chris de jong

Attachment: bugreport.html added

comment:1 Changed 10 years ago by Eugene Lazutkin

Component: GeneralDojoX GFX
Milestone: tbd1.3.1
Owner: changed from anonymous to Eugene Lazutkin
Priority: normalhigh
Status: newassigned

Excellent bug report: the fix, the test, which fails "before" and works "after"!

comment:2 Changed 10 years ago by Eugene Lazutkin

Resolution: fixed
Status: assignedclosed

(In [17302]) gfx: typo fix, thx Chris De Jong, !strict, fixes #9146.

comment:3 Changed 10 years ago by Adam Peller

Resolution: fixed
Status: closedreopened

leaving open until patch is applied to 1.3.1

comment:4 Changed 10 years ago by Eugene Lazutkin

Resolution: fixed
Status: reopenedclosed

(In [17338]) gfx: typo fix, thx Chris De Jong, fixes #9146.

comment:5 Changed 10 years ago by Eugene Lazutkin

(In [17342]) gfx: one more typo fix, !strict, fixes #9146.

comment:6 Changed 10 years ago by Eugene Lazutkin

(In [17343]) gfx: one more typo fix, !strict, fixes #9146.

Note: See TracTickets for help on using tickets.