Merge lp:~alejandrosantana/openerp-spain/6.1_account_balance_reporting_imp_fix_report_non_zero into lp:~pexego/openerp-spain/6.1_account_balance_reporting_imp

Proposed by Alejandro Santana on 2014-06-18
Status: Needs review
Proposed branch: lp:~alejandrosantana/openerp-spain/6.1_account_balance_reporting_imp_fix_report_non_zero
Merge into: lp:~pexego/openerp-spain/6.1_account_balance_reporting_imp
Diff against target: 37 lines (+11/-2)
1 file modified
account_balance_reporting/report/generic_non_zero_report.rml (+11/-2)
To merge this branch: bzr merge lp:~alejandrosantana/openerp-spain/6.1_account_balance_reporting_imp_fix_report_non_zero
Reviewer Review Type Date Requested Status
Pexego 2014-06-18 Pending
Review via email: mp+223536@code.launchpad.net

Commit message

[FIX] Corrige el error en el informe sin líneas a cero cuando no hay líneas distintas de cero.

Description of the change

Corregido el informe generic_non_zero_report.rml para contemplar el caso de que no haya ninguna línea distinta de cero, en cuyo caso daba un error que impedía la generación del pdf.

To post a comment you must log in.

Ufff, perdón, me acabo de dar cuenta de que en el MP vas a ver un montón de líneas cambiadas, porque usé el indentador automático de Kate.

Sólo hay un cambio en dos líneas, realmente:
---En la 310 añado:
310 + <font color="white" face="Times-Roman" size="1.0">[[ report.line_ids and removeParentNode('font') or removeParentNode('blockTable') ]]</font>

---En la 312 añado al principio una comprobación de que existe la línea: 'line and '
312 + [[ line and (line.current_value==0.0 and line.previous_value==0.0 and removeParentNode('tr')) or removeParentNode('tr') ]]

Si hace falta sobreescribo mi rama para que sea más claro el cambio, como veas.

400. By Alejandro Santana on 2014-06-20

[FIX] Corrige el error en el informe sin líneas a cero cuando no hay líneas distintas de cero.

Corregido el MP para que sólo haya cambios en las líneas que realmente importan (Más corrección de un error en el anterior).

Pedro Manuel Baeza (pedro.baeza) wrote :

Alejandro, ¿este MP todavía aplicaría?

Pues sí, al menos contra la rama de Pexego, ya que fue la que probé y donde detecté el error que daba al generar un informe sin líneas.

Unmerged revisions

400. By Alejandro Santana on 2014-06-20

[FIX] Corrige el error en el informe sin líneas a cero cuando no hay líneas distintas de cero.

399. By Alejandro Santana on 2014-06-18

[FIX] Corrige el error en el informe sin líneas a cero cuando no hay líneas distintas de cero.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account_balance_reporting/report/generic_non_zero_report.rml'
2--- account_balance_reporting/report/generic_non_zero_report.rml 2014-01-10 10:36:59 +0000
3+++ account_balance_reporting/report/generic_non_zero_report.rml 2014-06-20 13:41:14 +0000
4@@ -90,6 +90,7 @@
5 <paraStyle name="Bal3" fontName="Helvetica" fontSize="8.0" leading="10" alignment="LEFT"/>
6 <paraStyle name="Bal4" fontName="Helvetica" fontSize="8.0" leading="10" alignment="LEFT"/>
7 <paraStyle name="Bal5" fontName="Helvetica-Oblique" fontSize="8.0" leading="10" alignment="LEFT"/>
8+ <paraStyle name="zero" fontName="Helvetica" fontSize="0.0" leading="0" alignment="LEFT" spaceBefore="0.0" spaceAfter="0.0"/>
9 </stylesheet>
10 <images/>
11 <story>
12@@ -131,8 +132,9 @@
13 <blockTable colWidths="229.0,59.0,57.0,86.0,80.0" style="Tabla9">
14
15 <tr>
16- [[repeatIn(report.line_ids,'line')]]
17- [[ line.current_value==0.0 and line.previous_value==0.0 and removeParentNode('tr')]]
18+ [[ report.line_ids or removeParentNode('blockTable') ]]
19+ [[ repeatIn(report.line_ids,'line') ]]
20+ [[ line and line.current_value==0.0 and line.previous_value==0.0 and removeParentNode('tr') ]]
21 <td>
22 <para style="Bal1">
23 <font face="Times-Bold" size="10.0">[[(line.css_class=='l1') and line.name or removeParentNode('font')]]</font>
24@@ -166,6 +168,13 @@
25 <para style="P2">[[report.previous_fiscalyear_id and formatLang(line.previous_value)]]</para>
26 </td>
27 </tr>
28+ <tr>
29+ <td><para style="zero"></para></td>
30+ <td><para style="zero"></para></td>
31+ <td><para style="zero"></para></td>
32+ <td><para style="zero"></para></td>
33+ <td><para style="zero"></para></td>
34+ </tr>
35 </blockTable>
36 <para style="P4">
37 <font color="white"> </font>

Subscribers

People subscribed via source and target branches

to all changes: