openerp-mexico-localization

Merge lp://staging/~vauxoo/openerp-mexico-localization/oml-trunk-certificate_link-to-report-l10n_mx_facturae_report-dev-jorge into lp://staging/openerp-mexico-localization

oml-trunk-certificate_link-to-report-l10n_mx_facturae_report-dev-jorge
Merge into trunk

Proposed by Jorge Angel Naranjo Rogel - http://www.vauxoo.com on 2013-06-10

Status:	Merged
Merged at revision:	228
Proposed branch:	lp://staging/~vauxoo/openerp-mexico-localization/oml-trunk-certificate_link-to-report-l10n_mx_facturae_report-dev-jorge
Merge into:	lp://staging/openerp-mexico-localization
Diff against target:	90 lines (+32/-1) 2 files modified l10n_mx_facturae_report/report/invoice_facturae_html.mako (+7/-1) l10n_mx_facturae_report/report/invoice_facturae_html.py (+25/-0)
To merge this branch:	bzr merge lp://staging/~vauxoo/openerp-mexico-localization/oml-trunk-certificate_link-to-report-l10n_mx_facturae_report-dev-jorge
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Moisés López - http://www.vauxoo.com	2013-06-10	Approve on 2013-06-11
Jorge Angel Naranjo Rogel - http://www.vauxoo.com		Needs Resubmitting on 2013-06-11
Rodolfo Lopez	2013-06-11	Pending
Review via email: mp+168542@code.staging.launchpad.net

Description of the change

[REF][l10n_mx_facturae_report] Add certificate_link to report in CFD-I

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2013-06-10:

En donde usas
invoice.company_emitter_id.id
Deberias de usar el de la compania del usuario.

Revision history for this message

Moisés López - http://www.vauxoo.com (moylop260) wrote on 2013-06-11:

Cambiar esta línea
+ data_pac = pac_params_obj.browse(self.cr, self.uid, pac_params_ids[0])

Por
pac_params_id = pac_params_ids and pac_params_ids[0] or False
if pac_params_id:
+ data_pac = pac_params_obj.browse(self.cr, self.uid, pac_params_id)

Ya que si no tienes ningún params.pac configurado hará un trace.

review: Needs Fixing

lp://staging/~vauxoo/openerp-mexico-localization/oml-trunk-certificate_link-to-report-l10n_mx_facturae_report-dev-jorge updated on 2013-06-11

226. By Jorge Angel Naranjo Rogel - http://www.vauxoo.com on 2013-06-11: [FIX][l10n_mx_facturae_report] function _get_data_certificate of invoice_facturae_html.py