summaryrefslogtreecommitdiff
path: root/addons/web_process/static
diff options
context:
space:
mode:
authorXavier Morel <xmo@openerp.com>2012-01-13 16:01:10 +0100
committerXavier Morel <xmo@openerp.com>2012-01-13 16:01:10 +0100
commita76f6d2a4efd17906c6ef1b80e5e5e9830c9307a (patch)
tree532bcb269eef5f56808530db49a873e12533e249 /addons/web_process/static
parentdb7470757392baf9da571fd86a0e17f354328fcd (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')
0 files changed, 0 insertions, 0 deletions