Load locales with webpack

Hi,

I am trying to use this module with webpack. Sadly, it doesn’t work as expected. When I init the plugin with:

aurelia.use
        .standardConfiguration()
        .developmentLogging()
        .plugin('aurelia-validation');

I get validation-locale.js:78 Uncaught (in promise) ReferenceError: System is not defined.

I tracked the bug and it seems that System.import is explicitly used in the code to load the locale files here preventing the module to work with anything else than SystemJS.

I tried to load the locale manually. To do that, I used this code in main.js:

import {bootstrap} from 'aurelia-bootstrapper-webpack';
import {ValidationLocale} from '../node_modules/aurelia-validation/src/validation-locale';
import { data } from '../node_modules/aurelia-validation/src/resources/en-US';

bootstrap(function (aurelia) {
    aurelia.use
        .standardConfiguration()
        .developmentLogging();

    ValidationLocale.Repository.addLocale('en-US', data)

    aurelia.start().then(() => aurelia.setRoot('app', document.body));
});

Once I did that,

console.log(ValidationLocale.Repository.instances.has('en-US'))

does respond true. However, I still have the following errors:

validation-locale.js:78 Uncaught (in promise) ReferenceError: System is not defined
validation-property.js:86 Uncaught (in promise) Error: An exception occurred while trying to load the locale(…)

To make it work, I had to modify the modules loaders in webpack and add this line:

{ test: /aurelia-validation\/src\/.*\.js$/, loader: 'babel', query: {stage: 0} },

Would it be possible to remove the need for System.import or allow the user to load a locale manually?

[jsobell]

For those trying to get it working in the meantime: npm install Zensavona/validation#use-aurelia-loader-instead-of-systemjs Thanks @Zensavona

[EisenbergEffect]

@PWKad This is a similar issue that was reported the other day. We need to switch this to using the Loader abstraction otherwise it won’t work with Webpack, require.js, dojo or any non-system.js loader. I’d put this fix at a pretty high priority. @zewa666