dojo/i18n doesn't load dojo/xhr (but it needs to)

[bill]

dojo.requireLocalization() ends up calling dojo.xhr() but without loading the dojo/_base/xhr.

Apply attached patch to make i18n.js baseless, and then run util/doh/runner.html?test=dojo/tests/date/locale, which does a dojo.requireLocalization() call. It will get an exception.

The issue is that i18n.js has a dependency:

"./has!dojo-v1x-i18n-Api?./_base/xhr"

but dojo-v1x-i18n-Api isn't defined until later in that i18n.js file:

has.add("dojo-v1x-i18n-Api",
	// if true, define the v1.x i18n functions
	1
);

Chicken and egg problem.

Broke in [25690] so assigning to Ben.

[bill]

patch moving towards i18n.js being baseless (but it still has some dojo references)

[bill]

In [26965]:

Modifying files that call lang.getObject("foo", true, dojo) to remove unneeded references to dojo variable, refs #13959 !strict.

Skipping i18n.js since it has a dojo.xhr() missing dependency issue, refs #14211.

[ben hockey]

In [26968]:

since r26879 fixed the unguarded reference to window, there's no longer a need
to conditionally load xhr. refs #14211 !strict

[ben hockey]

bill, should r26968 be enough to close this ticket?

[bill]

Yes, after it's backported to the 1.7 branch.

There's still the issue of converting i18n.js to baseless, which I'll do on trunk, and also a missing reference to dojo/_base/config, but that missing reference is benign since kernel.js pulls in config.js.

[bill]

In [26969]:

Updates i18n.js module to not use dojo global variable, except for exporting the module's symbols. Refs #14220, #14211 !strict. Apparently it's no good to reference config.locale, (tests/date/locale fails), so still referencing dojo.locale. That should be fixed at some point.

[ben hockey]

In [26979]:

applying r26968 to 1.7 branch. fixes #14211 !strict