Code review comment for lp:~alejandrosantana/openerp-spain/6.1_account_balance_reporting_imp_fix_report_non_zero

Revision history for this message
Alejandro Santana (alejandrosantana) wrote :

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.

« Back to merge proposal