[FIX] Error al crear el asiento de cierre

[cubells]

Detectado error en los asientos de cierre.

Las cuentas con el método de cierre 'unreconcilied' deben de añadirse al cierre si existe balance, porque si no es así, se añaden lineas al asiento de cierre redundantes (porque el balance es cero) y/o otras que no deberían de estar como 477, 472 etc.

Probado en clientes en producción.

[cubells]

Esta modificación es válida tanto para la 6.1, la 7 y la 8.

@pedrobaeza: hago el pull en las otras, no?

[pedrobaeza]

Creo que es mejor hacer el parche poniendo la condición del balance al principio del todo, en la línea:

if account.type != 'view':

y haciendo la comparación de forma correcta para evitar posibles errores de redondeo, quedando así:

from openerp.tools import float_is_zero
...
precision = self.pool['decimal.precision'].precision_get(cr, uid, 'Account')
if (account.type != 'view' and 
        not float_is_zero(account.balance, precision_rounding=precision)):
...

y así se puede quitar también esa condición de la parte que se hace con el método de cierre 'balance'.

[cubells]

perfecto.

Lo modifico y pruebo y subo el pull. Te parece?

[pedrobaeza]

Perfecto.

[cubells]

@pedrobaeza: por cierto.

En la linea donde se realiza el diccionario de las lineas con el map, aquí:
https://github.com/OCA/l10n-spain/blob/8.0/l10n_es_fiscal_year_closing/wizard/wizard_run.py#L402

en una 6.1 en producción, es imposible realizar el cierre. Se queda pensando en esa linea.

Leí que esta solución gasta mucha menos memoria:

'line_id': [(0, 0, x) for x in move_lines],

Lo he probado y he podido realizar el cierre.

Qué opinas?

[pedrobaeza]

En teoría, las "list comprehension" son más lentas que los map, aunque hay algunas discrepancias (https://www.google.es/search?client=ubuntu&channel=fs&q=list+comprehension+versus+map&ie=utf-8&oe=utf-8&gfe_rd=cr&ei=QiNBVYz-DdKp8weNsYGQDg), pero si la experiencia empírica te dice que es mejor la otra opción, adelante, cámbialo.

[cubells]

Va perfecto y más rápido.

[pedrobaeza]

👍 Esperamos a Travis, que en la rama 7.0 debe estar correcto.

[cubells]

No problem.

Cuando pase hago el pr en la 8.0

En la 6.1 lo hago en bazar, no?

[pedrobaeza]

No, también está aquí en GitHub. Migré todo de hosting.

[coveralls]

Coverage increased (+0.01%) to 18.71% when pulling 32a77c4 on cubells:7.0 into cf193a6 on OCA:7.0.