Changeset 7590 for branches/0.4/buildscripts

Timestamp:

03/14/07 12:42:45 (22 months ago)

Author:

jburke

Message:

Fix for xdomain 18n resource loading. Previous change to get local i18n bundles to work was flawed. Also rolling back previous buildUtilXd.js since I do not think it is part of the problem. Fixed a test for IE.

Files:

• branches/0.4/buildscripts/buildUtilXd.js (modified) (1 diff)

branches/0.4/buildscripts/buildUtilXd.js

@@ -113 +113 @@
 
         //Final XD file contents.
-        var provideString = 'dojo.provide("' + moduleName + '.nls.' + (localeName ? localeName + '.' : '') + bundleName + '");';
-
-        fileContents = provideString
+        fileContents = 'dojo.provide("' + moduleName + '.nls.' + (localeName ? localeName + '.' : '') + bundleName + '");'
                          + 'dojo.hostenv.xdLoadFlattenedBundle("' + moduleName + '", "' + bundleName
                    + '", "' + localeName + '", ' + fileContents + ');';
 
-        //Now make a proper xd.js file out of the content, but
-        //only include the dojo.provide statement. We'll leave the bundle contents outside of the
-        //xd wrapper.
-        var bundleContents = buildUtilXd.makeXdContents(provideString, baseRelativePath, prefixes);
-        
-        //Just inline the real contents right in the file. Its ok if bundles are evaluated right
-        //away since they don't have dependencies.
-        bundleContents += fileContents;
-        
-        return bundleContents;
+        //Now make a proper xd.js file out of the content.
+        return buildUtilXd.makeXdContents(fileContents, baseRelativePath, prefixes);
 }
 //END makeXdBundleContents function