Opened 7 years ago

Closed 7 years ago

#15009 closed defect (fixed)

Dojox StatefulModel getStateful

Reported by: Matej Owned by: Ed Chatelain
Priority: undecided Milestone: 1.8
Component: DojoX MVC Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

When I use dojox/mvc/getStateful modul with object var obj = {

prop1 : "String", prop2 : 10, prop3 : undefined, check this prop4 : null check this };

IE7 has different result. The problem is with prop3 and prop4 where is no value. See attached example and console.log in IE and Chrome.

Attachments (3)

statefulmodel.html (1.1 KB) - added by Matej 7 years ago.
example
mvc.html (1.2 KB) - added by Matej 7 years ago.
correct sample
15009-tweak.patch (4.6 KB) - added by Ed Chatelain 7 years ago.
This patch has the tweak to the fix which went into 14681, and fix for 15011

Download all attachments as: .zip

Change History (8)

Changed 7 years ago by Matej

Attachment: statefulmodel.html added

example

Changed 7 years ago by Matej

Attachment: mvc.html added

correct sample

comment:1 Changed 7 years ago by Matej

use trunk version or download StatefulArray?.js and getStateful.js

http://svn.dojotoolkit.org/src/dojox/trunk/mvc/StatefulArray.js http://svn.dojotoolkit.org/src/dojox/trunk/mvc/getStateful.js

IE9 / IE 8 result

prop1 : "String", prop2 : 10, prop3 : undefined, prop4 : null,

IE7 result

prop1 : "String", prop2 : 10, prop3 : [object Object], prop4 : [object Object],

please fix it.

comment:2 Changed 7 years ago by Matej

Check getType() method pls.

comment:3 Changed 7 years ago by Ed Chatelain

Milestone: tbd1.8
Status: newassigned

comment:4 Changed 7 years ago by Ed Chatelain

The fix was made by #14681, the fix works, but I am going to be tweaking it a little, so I will close this out after those updates are in.

Changed 7 years ago by Ed Chatelain

Attachment: 15009-tweak.patch added

This patch has the tweak to the fix which went into 14681, and fix for 15011

comment:5 Changed 7 years ago by cjolif

Resolution: fixed
Status: assignedclosed

In [28183]:

fixes #15009, #15011. Fix typo in test + tweak to 15009 fix. IE tests still fail. !strict.

Note: See TracTickets for help on using tickets.