Automatically patch the require.js module callbacks

[mattrobenolt]

This can be done by checking for define.amd.

https://github.com/jrburke/requirejs/blob/master/require.js#L1921

Possibly require() as well:

https://github.com/jrburke/requirejs/blob/master/require.js#L1325

[SBoudrias]

Hi @mattrobenolt, tested this build this morning and this was working great on my app.

Although, I'm concerned by the fact you infer if define is a function, then require is a function. If I'm correct, only RequireJS AMD loader use the require function, other like curl, lsjs and dojo may break (didn't test it though). But anyway, I guess the point is mostly that inferring is a - most of the time - bad thing. Any though?

[mattrobenolt]

I'll look into this today with the other AMD libs and see if it'll hurt. Thanks for the feedback. :)

And, this is probably the direction things are going to go: #69 FYI

[mattrobenolt]

Ok, so according to the spec, you're right. It's not required to exist. I'll adjust this. http://wiki.commonjs.org/wiki/Modules/AsynchronousDefinition

[mattrobenolt]

This will be a plugin. So closing this.