search

OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
22 stars 51 forks source link

Update sale.py #8

Closed lbaldi closed 9 years ago

lbaldi commented 9 years ago

El chequeo de la posición fiscal en la orden de venta se tiene que realizar antes de obtener la denominación. No sirve tenerla dentro de la función _get_pos_ar

oca-clabot commented 9 years ago

Hey @lbaldi, 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:

Appreciation of efforts, OCA CLAbot

sebastiken commented 9 years ago

Gracias @lbaldi por tu aporte. Por lo que veo del código, si la Orden no tiene Posición Fiscal llamaba a get_posar con el parámetro _denomid en False cosa que es inconsistente. Con tu modificación, la llamada a get_posar directamente no se hace si no tiene la Posición Fiscal seteada en la Orden. Perfecto!