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

[Alejandro Santana]

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

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.

[Alejandro Santana (alejandrosantana)]

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.

[lp:~alejandrosantana/openerp-spain/6.1_account_balance_reporting_imp_fix_report_non_zero]

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.

[Alejandro Santana (alejandrosantana)]

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)]

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

[Alejandro Santana (alejandrosantana)]

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.

=== modified file 'account_balance_reporting/report/generic_non_zero_report.rml'
--- account_balance_reporting/report/generic_non_zero_report.rml	2014-01-10 10:36:59 +0000
+++ account_balance_reporting/report/generic_non_zero_report.rml	2014-06-20 13:41:14 +0000
@@ -90,6 +90,7 @@
     <paraStyle name="Bal3" fontName="Helvetica" fontSize="8.0" leading="10" alignment="LEFT"/>
     <paraStyle name="Bal4" fontName="Helvetica" fontSize="8.0" leading="10" alignment="LEFT"/>
     <paraStyle name="Bal5" fontName="Helvetica-Oblique" fontSize="8.0" leading="10" alignment="LEFT"/>
+    <paraStyle name="zero" fontName="Helvetica" fontSize="0.0" leading="0" alignment="LEFT" spaceBefore="0.0" spaceAfter="0.0"/>
   </stylesheet>
   <images/>
   <story>
@@ -131,8 +132,9 @@
     <blockTable colWidths="229.0,59.0,57.0,86.0,80.0" style="Tabla9">
        
         <tr>
-        [[repeatIn(report.line_ids,'line')]]
-        [[ line.current_value==0.0 and line.previous_value==0.0 and removeParentNode('tr')]]
+        [[ report.line_ids or removeParentNode('blockTable') ]]
+        [[ repeatIn(report.line_ids,'line') ]]
+        [[ line and line.current_value==0.0 and line.previous_value==0.0 and removeParentNode('tr') ]]
         <td>
           <para style="Bal1">
             <font face="Times-Bold" size="10.0">[[(line.css_class=='l1') and line.name or removeParentNode('font')]]</font>
@@ -166,6 +168,13 @@
           <para style="P2">[[report.previous_fiscalyear_id and formatLang(line.previous_value)]]</para>
         </td>
       </tr>
+      <tr>
+          <td><para style="zero"></para></td>
+          <td><para style="zero"></para></td>
+          <td><para style="zero"></para></td>
+          <td><para style="zero"></para></td>
+          <td><para style="zero"></para></td>
+        </tr>
     </blockTable>
     <para style="P4">
       <font color="white"> </font>