Merge lp://qastaging/~carlos-realsystems/openerp-mexico-localization/addons-mx-trunk-CAMPO_DATETIME_FE-dev-carlos-realsystems into lp://qastaging/openerp-mexico-localization/6.1

Proposed by Carlos Contreras
Status: Needs review
Proposed branch: lp://qastaging/~carlos-realsystems/openerp-mexico-localization/addons-mx-trunk-CAMPO_DATETIME_FE-dev-carlos-realsystems
Merge into: lp://qastaging/openerp-mexico-localization/6.1
Diff against target: 262 lines (+12/-182)
4 files modified
l10n_mx_invoice_datetime/__openerp__.py (+4/-2)
l10n_mx_invoice_datetime/invoice.py (+8/-5)
l10n_mx_invoice_datetime/patchs/account.patch (+0/-42)
l10n_mx_invoice_datetime/patchs/account_invoice_report.sql (+0/-133)
To merge this branch: bzr merge lp://qastaging/~carlos-realsystems/openerp-mexico-localization/addons-mx-trunk-CAMPO_DATETIME_FE-dev-carlos-realsystems
Reviewer Review Type Date Requested Status
Moisés López - http://www.vauxoo.com Needs Fixing
Review via email: mp+146957@code.qastaging.launchpad.net

Description of the change

Se agrega campo invoice_datetime tipo Fecha y hora, así como su muestra en la vista de la factura en el tab otros datos.

To post a comment you must log in.
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

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

review: Needs Fixing
217. By Carlos Contreras

 Se corrige el formato datetime y para evitar que agregue el tiempo al campo date_invoice por observaciones de Moises Lopez

Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

El comment "and add hour & minute to field date_invoice" en el __openerp__.py
ahora está de más.

review: Needs Fixing
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) wrote :

Ahora que el campo "date_invoice" es field.date
Y el nuevo campo "invoice_datetime" es el que tiene el field.datetime
Ahora hay que reemplazar en todos los módulos donde se usaba el campo "date_invoice" al nuevo campo "invoice_datetime"

Para esto te puede servir un
rgrep "date_invoice" addons-mx-folder

review: Needs Fixing

Unmerged revisions

217. By Carlos Contreras

 Se corrige el formato datetime y para evitar que agregue el tiempo al campo date_invoice por observaciones de Moises Lopez

216. By Carlos Contreras

 Se grega a la vista el campo a mostrar en pestaña otros.

215. By Carlos Contreras

 Se agrega campo invoice_datetime tipo datetime

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.