openerp-mexico-localization

Merge lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_pac_sf-get-rate-dev-yzk into lp://staging/openerp-mexico-localization

  1. addons-mx-trunk-facturae_pac_sf-get-rate-dev-yzk
  2. Merge into trunk
Proposed by Isaac López Zúñiga
Status: Merged
Merged at revision: 107
Proposed branch: lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_pac_sf-get-rate-dev-yzk
Merge into: lp://staging/openerp-mexico-localization
Diff against target: 102 lines (+27/-15)
2 files modified
l10n_mx_facturae_pac_sf/wizard/wizard_export_invoice_pac_sf.py (+13/-5)
l10n_mx_facturae_pac_sf/wizard/wizard_export_invoice_pac_sf_v6.py (+14/-10)
To merge this branch: bzr merge lp://staging/~vauxoo/openerp-mexico-localization/addons-mx-trunk-facturae_pac_sf-get-rate-dev-yzk
Reviewer Review Type Date Requested Status
Moisés López - http://www.vauxoo.com Pending
Review via email: mp+88586@code.staging.launchpad.net

This proposal supersedes a proposal from 2012-01-14.

Description of the change

Correction on rate

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

La linea
31 + rate = invoice.currency_id.rate and invoice.currency_id.rate or '1'

Que diferencia tiene con la siguiente sentencia?
31 + rate = invoice.currency_id.rate or '1'

O tal vez quisiste poner
31 + rate = invoice.currency_id.rate and str(invoice.currency_id.rate) or '1'
Notese el str

review: Needs Fixing
Revision history for this message
Moisés López - http://www.vauxoo.com (moylop260) : Posted in a previous version of this proposal
review: Approve

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.