Opened 7 years ago

Closed 7 years ago

Last modified 7 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 7 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 7 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 7 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 7 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 7 years ago by bill

Milestone: tbd1.8

comment:4 Changed 7 years ago by cjolif

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

comment:5 Changed 7 years ago by cjolif

In [28544]:

refs #15331.

comment:6 Changed 7 years ago by cjolif

In [28545]:

refs #15331.

Note: See TracTickets for help on using tickets.