diff options
| author | Xavier Morel <xmo@openerp.com> | 2012-01-13 16:01:10 +0100 |
|---|---|---|
| committer | Xavier Morel <xmo@openerp.com> | 2012-01-13 16:01:10 +0100 |
| commit | a76f6d2a4efd17906c6ef1b80e5e5e9830c9307a (patch) | |
| tree | 532bcb269eef5f56808530db49a873e12533e249 /addons/web_process/static/src/xml/web_process.xml | |
| parent | db7470757392baf9da571fd86a0e17f354328fcd (diff) | |
[FIX] concatenated JS files blowing up when not terminated with a semicolon
If the final semicolon of an openerp module is forgotten and the next
file starts with an expression (such as a parens, because it's a
third-party module using the module pattern, see Backbone.js or jQuery
for examples), the JS parser will not perform semicolon insertion and
will instead try to execute the module's wrapper/initializer function
by passing it whatever follows it (generally an other function).
This usually results in an error and stuff blowing out everywhere for
no obvious reason.
Concatenation of JS files now adds an explicit semicolon between files
(ideally it should only add one if it's missing, but that would
require backtracking in the file while skipping comments &etc, can't
be arsed and double-semicolons don't hurt much) to avoid this issue.
bzr revid: xmo@openerp.com-20120113150110-47j90oishtjrib7s
Diffstat (limited to 'addons/web_process/static/src/xml/web_process.xml')
0 files changed, 0 insertions, 0 deletions
