Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#15331 closed defect (fixed)

Change the debugDataBinding support to use a flag set in data-dojo-config

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

Description

Set the flag to debugDataBinding in data-dojo-config instead of using dojox.debugDataBinding = true;

How about debugMvcBindings as the flag?

Attachments (1)

mvc-15331-debugflag.patch (80.7 KB) - added by Ed Chatelain 10 years ago.
Along with this patch we also need to rename ParserExtension?.js to parserExtension.js, but that was not picked up by the patch.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 years ago by cjolif

Even better we might provision for other flags and have a hash object on dojoConfig that contains all potential MVC flags:

mvc: {
  debugBindings: true,
  ...
}

Or something along those lines.

Changed 10 years ago by Ed Chatelain

Attachment: mvc-15331-debugflag.patch added

Along with this patch we also need to rename ParserExtension?.js to parserExtension.js, but that was not picked up by the patch.

comment:2 Changed 10 years ago by cjolif

Resolution: fixed
Status: newclosed

In [28543]:

fixes #15331. Change the debugDataBinding support to use a flag set in data-dojo-config and rename ParserExtension? to parserExtension as this is not a class. Thanks Ed Chatelain & Akira Sudoh (IBM, CCLA). !strict.

comment:3 Changed 10 years ago by bill

Milestone: tbd1.8

comment:4 Changed 10 years ago by cjolif

Actually ParserExtension renaming did not come trough working on fixing this.

comment:5 Changed 10 years ago by cjolif

In [28544]:

refs #15331.

comment:6 Changed 10 years ago by cjolif

In [28545]:

refs #15331.

Note: See TracTickets for help on using tickets.