openerp-community-reviewer team mailing list archive

[Graeme Gellatly <gdgellatly@xxxxxxxxx>]

Review: Approve (code review, no test)

To the best of my knowledge a fiscal position is not dependent on a product and nor is an invoice line.  It would be nice if this dependency wasn't there but recognise the difficulties in implementing consistently if an existing fiscal position has already remapped accounts and taxes before changing it.  Also recognise that most invoices contain products.  

So only comment would be to document the limitation or raise an error if none of the lines contain product_ids.  Not sure how it would happen with a mix of both, but maybe it would pay to play it safe and raise an error if this case ever happens.

But otherwise LGTM.

-- 
https://code.launchpad.net/~akretion-team/account-invoicing/70-add-invoice_fiscal_position_update/+merge/200358
Your team Account Core Editors is requested to review the proposed merge of lp:~akretion-team/account-invoicing/70-add-invoice_fiscal_position_update into lp:account-invoicing.