Merge lp:~openerp-dev/openerp-web/trunk-qwebreport-pos-sle into lp:openerp-web

Proposed by sle-openerp
Status: Merged
Merged at revision: 3994
Proposed branch: lp:~openerp-dev/openerp-web/trunk-qwebreport-pos-sle
Merge into: lp:openerp-web
Diff against target: 23 lines (+5/-8)
1 file modified
addons/web/static/src/js/core.js (+5/-8)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/trunk-qwebreport-pos-sle
Reviewer Review Type Date Requested Status
OpenERP R&D Web Team Pending
Review via email: mp+216030@code.launchpad.net
To post a comment you must log in.
3977. By sle-openerp

[MERGE] trunk

3978. By sle-openerp

[MERGE] trunk

3979. By sle-openerp

[MERGE] trunk

3980. By sle-openerp

[MERGE] trunk

3981. By sle-openerp

[MERGE] trunk

3982. By sle-openerp

[MERGE] trunk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'addons/web/static/src/js/core.js'
2--- addons/web/static/src/js/core.js 2014-04-18 15:25:11 +0000
3+++ addons/web/static/src/js/core.js 2014-04-29 12:43:38 +0000
4@@ -460,14 +460,11 @@
5 .appendTo(document.body)
6 .load(function () {
7 try {
8- if (options.error) {
9- if (!this.contentDocument.body.childNodes[1]) {
10- options.error(this.contentDocument.body.childNodes);
11- }
12- else {
13- options.error(JSON.parse(this.contentDocument.body.childNodes[1].textContent));
14- }
15- }
16+ if (options.error) {
17+ var body = this.contentDocument.body;
18+ var node = body.childNodes[1] || body.childNodes[0];
19+ options.error(JSON.parse(node.textContent));
20+ }
21 } finally {
22 complete();
23 }