[dojox/app] Adding top-level nls file breaks build system

[Paul Christopher]

Description

See attached extra files for the existing test case dojox/app/test/multiSceneApp: Adding translations for the tab bar in application.html seems to break the build system:

discovering resources...
starting reading resources...
starting processing raw resource content...
starting tokenizing resource...
starting processing resource tokens...
starting parsing resource...
js: "<eval'ed string>#1(Function)#1(eval)", line 96: uncaught JavaScript runtime
 exception: ReferenceError: "params" is not defined.
        at <eval'ed string>#1(Function)#1(eval):96
        at <eval'ed string>#1(Function)#1(eval):277
        at <eval'ed string>#1(Function)#1(eval):277
        at <eval'ed string>#1(Function)#1(eval):248 (doBuild)
        at <eval'ed string>#1(Function)#1(eval):283
        at <eval'ed string>#1(Function)#1(eval):83

I.e. the unbuilt version with translations just works fine, but as soon as you try to build the application, the build fails with the error message shown above.

Steps to reproduce the issue

• Add/ replace the files in folder dojox/app/tests/multiSceneApp with the files provided in the attached zip archive.

• Run index.html: Translations should be fine.

• Now try to build the application. This fails with the above error message.

Discussion

I ran into this issue with my own multiSceneApp like application. The build gets broken when adding general, top-level translations for the tab bar. I could now reproduce the issue with dojox/app/tests/multiSceneApp. There might be still other issues with building dojox/app/tests/multiSceneApp (such as the build tool does not like the location folder "test"), but this is not the aim of this ticket.

[dylan]

Given the lack of attention to this ticket, we would accept a pull request. Closing as patchwelcome.