[regression] mdnd test failure in test_dnd_dndFromDojo.html, included patch

[wsargent]

When you run the test by hitting the page, you cannot accept Type1 items into the "Source/Target? dojox" box, and you see the exception as follows:

Uncaught TypeError?: Object #<Object> has no method 'manager' at line 51 in DndFromDojo?.js.

[wsargent]

patch to change Manager reference

[Adam Peller]

patch file does not apply. what format is this?

I'm not familiar with the code. Why isn't a 'new' keyword needed here? Why not also convert over the AreaManager? line above?

[bill]

manager() and areaManager() are both factory functions, hence no new keyword.

I'll check in the fix to call Manager.manager() along with the fix to #16036. Can't call AreaManager.areaManager() though without fixing the code in dojox/dnd/AreaManager.js, so I won't bother with that. (You can update it if you like.)

[bill]

In [29755]:

For back-compat, make sure client code can call dojox.dnd.manager() in addition to Manager.manager(). Also fixing one of the AMD conversion errors in dojox/mdnd. Refs #16029, #16036, !strict, fixing them on trunk.

[bill]

In [29807]:

Merge [29755] to 1.8 branch, making dojox.dnd.manager() available again, fixes #16029, #16036 !strict.