account_invoice_posting

[openmindsystems]

This new module adds a field to save the accounting date in the invoice form.

[oca-clabot]

Hey @openmindsystems, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

• @openmindsystems (login unknown in OCA database)

Appreciation of efforts,
OCA CLAbot

[pedrobaeza]

Buenas, ¿podéis comentar cuál es el objetivo de ese campo?

[AlbertCabedo]

Buenas, para determinadas empresas que reciben subvenciones o han de justificar facturas para distintos organismos oficiales.

[alejandrosantana]

Se recomienda la forma: from . import account_invoice
Y falta un salto del línea al final.

[alejandrosantana]

PEP8 me devuelve estos avisos:

./__init__.py:25:23: W292 no newline at end of file
./__openerp__.py:36:17: E203 whitespace before ':'
./account_invoice.py:31:1: E302 expected 2 blank lines, found 1
./account_invoice.py:33:1: W293 blank line contains whitespace
./account_invoice.py:37:1: W293 blank line contains whitespace
./account_invoice.py:38:18: W292 no newline at end of file

El código, por lo demás, OK. En la utilidad/necesidad del campo ya no me meto. ¿Tendría sentido que la fecha se autorellenara con el botón de "Enviar factura"?

[pedrobaeza]

Pero es que no es necesario crear un campo per se. Directamente, esa información la tienes con la fecha de creación del asiento contable. Se puede hacer dos cosas:

• Consultar esa información directamente con las herramientas del sistema, lo cual reconozco que no es directo.
• Poner que el campo sea un related a move_id.create_date.

Me decanto por esta segunda opción.

Además, este módulo debería ir destinado a un repositorio más genérico, ya que es una utilidad general. Yo diría de meterlo en https://github.com/OCA/account-financial-tools.

Siento tantas pegas, pero es por hacerlo lo mejor posible.

[openmindsystems]

Buenas, hemos corregido los puntos que nos habéis comentado.

Un saludo.

[pedrobaeza]

Mis comentarios no están atendidos.