Invalid AMD module identifier about scroller HOT 5 CLOSED

Damn! That is very frustrating! Oh well - so it goes. I'll need to update all of the extensions. Fortunately I've still not got around to publicising the AMP module aspect of them in any documentation yet, so I guess that's sort of good...

from scroller.

It's probably not so bad. Initially I assumed that RequireJS was not able to handle the current identifier, however, it turned out that it works nevertheless.

from scroller.

Just to check, it does work with the current formatting for you? I don't have a demo setup at the moment to test it myself, but I was sure I had tested it when I implemented it... However, it probably is better to get it right before I broadcast that this feature is in there... :-)

from scroller.

I am not that familiar with AMD and RequireJS, however, it definitely works using the following configuration:

require.config({
    baseUrl: 'lib',
    paths: {
        datatables: 'datatables/datatables',
        'datatables-scroller': 'datatables/scroller/js/dataTables.scroller',
        // …
    },
    // …
})

// Load the main app module to start the app
requirejs(["app/main"]);

I also use some other libraries that haven't built-in support for AMD and they require some special configuration (http://requirejs.org/docs/api.html#config-shim). On the other hand, for DataTables (and Scroller) the shim configuration is not necessary. They just work "out of the box".

from scroller.

Fixed by this commit: f32487b. I've removed the named AMD loader for all DataTables extensions. It remains for DataTables core, since the extensions need to be able to reference DataTables by a name, but the extensions are all anonymous now.

from scroller.