Code review comment for lp://qastaging/~carlos-realsystems/openerp-mexico-localization/addons-mx-trunk-CAMPO_DATETIME_FE-dev-carlos-realsystems

Hola Carlos,
Muchas gracias por tu propuesta de merge.

Si partimos de que este cambio es para conservar el campo de date_invoice, tal cual del tipo "date" (como está originalmente) y no del tipo "datetime" como se heredaba antes, y por lo tanto requería de un parche...

Hago aquí mis comentarios:
En la línea
65 - self.write(cr, uid, [inv.id], {'date_invoice': time.strftime('%Y-%m-%d %H:%M:%S')})
66 + t=time.strftime('%Y-%m-%d %H:%M:%S')
67 + self.write(cr, uid, [inv.id], {'date_invoice': t, 'invoice_datetime': t })

El campo date_invoice ya no debería de tener la sección " %H:%M:%S"
El campo date_invoice, ya no se debería de redefinir a datetime en _columns, es decir, eliminar esa línea para que se conserve el original.
El parche del sql y del .patch ya no será necesario, por lo que ya se puede hacer un
bzr remove l10n_mx_invoice_datetime/patchs