Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

[regression] lang collection in metas is missing for controller #1215

Closed
caridy opened this issue Aug 12, 2013 · 1 comment
Closed

[regression] lang collection in metas is missing for controller #1215

caridy opened this issue Aug 12, 2013 · 1 comment

Comments

@caridy
Copy link
Contributor

caridy commented Aug 12, 2013

Reported by @ft thru the forum: http://developer.yahoo.com/forum/Yahoo-Mojito/Internationalization-on-client-side/1376040501301-15e8a4da-ac8c-4391-985f-093001c8b477/1376293135175

It seems like the list of supported langs for each controlled is not added correctly to the meta, so, for mojits running on the client side it will never load the corresponding lang bundle. Here is an example of the meta from the repro case (https://github.com/ftliou/mojito-test):

YUI.add("loader-app", function(Y) {
    Y.applyConfig({
        groups: {
            app: Y.merge(((Y.config.groups && Y.config.groups.app) || {}), {
                modules: {
                    "Master": {
                        "path": "Master.js",
                        "requires": ["mojito", "dump", "mojito-composite-addon", "mojito-intl-addon", "mojito-assets-addon", "intl"]
                    },
                    "lang/Master_es": {
                        "path": "Master_es.js",
                        "requires": ["intl"]
                    }
                }
            })
        }
    });
}, "", {
    requires: ["loader-base"]
});

In this case, Master should have the collection of langs right after requires.

lzhan pushed a commit to lzhan/mojito that referenced this issue Aug 27, 2013
lzhan pushed a commit that referenced this issue Sep 10, 2013
fix issue #1215 lang collection in metas is missing for controller
@caridy
Copy link
Contributor Author

caridy commented Sep 25, 2013

+mojito@0.7.5

@caridy caridy closed this as completed Sep 25, 2013
lzhan pushed a commit to lzhan/mojito that referenced this issue Nov 7, 2013
lzhan pushed a commit to lzhan/mojito that referenced this issue Nov 7, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant