Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#15886 closed defect (fixed)

Compatibility problem for dojox/mvc found with todomvc project.

Reported by: Ed Chatelain Owned by: Ed Chatelain
Priority: undecided Milestone: 1.8.1
Component: DojoX MVC Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description

Akira hit a problem moving the todomvc dojo code up to dojo 1.8. The previous code which was working with dojo 1.7 did not work on 1.8, because of a problem in mvc/StatefulModel.

Attachments (1)

mvc15886.patch (987 bytes) - added by Ed Chatelain 7 years ago.
Fix for StatefulModel?, thanks to Akira Sudoh (IBM CCLA)

Download all attachments as: .zip

Change History (7)

Changed 7 years ago by Ed Chatelain

Attachment: mvc15886.patch added

Fix for StatefulModel?, thanks to Akira Sudoh (IBM CCLA)

comment:1 Changed 7 years ago by cjolif

Milestone: tbd1.8.1

comment:2 Changed 7 years ago by cjolif

Milestone: 1.8.1tbd

actually looking at it, this "just" re-introduces an undocumented API that the app was probably using. Maybe a better solution would be for the app to avoid using it?

Last edited 7 years ago by cjolif (previous) (diff)

comment:3 Changed 7 years ago by cjolif

after discussion with edchat as some people already took model against the todomvc app, we will fix that "regression".

comment:4 Changed 7 years ago by cjolif

In [29550]:

refs #15886. Fixing "regression" on private (but advertised) API . Thanks Akira Sudoh (IBM, CCLA).

comment:5 Changed 7 years ago by cjolif

Resolution: fixed
Status: newclosed

In [29551]:

fixes #15886. Fixing "regression" on private (but advertised) API . Thanks Akira Sudoh (IBM, CCLA).

comment:6 Changed 7 years ago by bill

Milestone: tbd1.8.1
Note: See TracTickets for help on using tickets.