[7.0] update taxes in invoices do ignore fiscal positions

Bug #1154594 reported by Frederic Clementi - Camptocamp
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Low
OpenERP Publisher's Warranty Team

Bug Description

Hello,

create a new DB with l10n_fr
standard VAT at nornal rate is 19.6.
with l10n_fr I also have the fiscal position for export or export with european union.

So here is how to reproduce
create a product with defaul sale vat set
Create a customer invoice
Do not set any fiscal position
create a invoice line with this product
save your invoice with this VAT

Now, change the fiscal position to export
and click on Update (beside taxes)

Problem : vat is not replaced by the destination vat set in the fiscal position

Note that if you set the fiscal position before creating your invoice line, it works as expected.

see video : https://docs.google.com/file/d/0B4GKXfLLiGNtNTlQWjhmMVp3c3c/edit?usp=sharing

Thanks

Frederic
Camptocamp

Tags: maintenance

Related branches

Revision history for this message
Frederic Clementi - Camptocamp (frederic-clementi) wrote :

By the way, we hame the same problem on supplier invoice

Changed in openobject-addons:
assignee: nobody → OpenERP Publisher's Warranty Team (openerp-opw)
tags: added: maintenance
Revision history for this message
Antoine(OpenERP) (ahu-openerp) wrote :

Here's the thing we discussed with Phuong and Quentin

For version 7, we could display a message as soon as the fiscal is position is modified while there's at least one line in the invoice.

For the trunk version, we could add an onchange function on the field fiscal_position so that it will change all the taxes defined in every lines of the invoice by the matching tax in the fiscal position, if there's a source tax being equal to the tax of the invoice line for that fiscal position.

Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote : Re: [Bug 1154594] Re: [7.0] update taxes in invoices do ignore fiscal positions

Hello,

if you attempt such on_change, make sure it's extensible enough. Because in
some countries like Brazil (you have 10 "partners" there), we use a fiscal
position by line, because for some products it has to be different. It
means we should have a way to stop or override a blind propagation to the
lines.

Thanks.

On Thu, Apr 4, 2013 at 1:24 PM, Antoine(OpenERP) <<email address hidden>
> wrote:

> Here's the thing we discussed with Phuong and Quentin
>
> For version 7, we could display a message as soon as the fiscal is
> position is modified while there's at least one line in the invoice.
>
> For the trunk version, we could add an onchange function on the field
> fiscal_position so that it will change all the taxes defined in every
> lines of the invoice by the matching tax in the fiscal position, if
> there's a source tax being equal to the tax of the invoice line for that
> fiscal position.
>
> --
> You received this bug notification because you are subscribed to OpenERP
> Addons.
> https://bugs.launchpad.net/bugs/1154594
>
> Title:
> [7.0] update taxes in invoices do ignore fiscal positions
>
> Status in OpenERP Addons (modules):
> New
>
> Bug description:
> Hello,
>
> create a new DB with l10n_fr
> standard VAT at nornal rate is 19.6.
> with l10n_fr I also have the fiscal position for export or export with
> european union.
>
> So here is how to reproduce
> create a product with defaul sale vat set
> Create a customer invoice
> Do not set any fiscal position
> create a invoice line with this product
> save your invoice with this VAT
>
> Now, change the fiscal position to export
> and click on Update (beside taxes)
>
> Problem : vat is not replaced by the destination vat set in the fiscal
> position
>
> Note that if you set the fiscal position before creating your invoice
> line, it works as expected.
>
> see video :
>
> https://docs.google.com/file/d/0B4GKXfLLiGNtNTlQWjhmMVp3c3c/edit?usp=sharing
>
> Thanks
>
> Frederic
> Camptocamp
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/openobject-addons/+bug/1154594/+subscriptions
>

Revision history for this message
Antoine(OpenERP) (ahu-openerp) wrote :

You mean a fiscal position for every invoice line?
I tried the l10n_br and couldn't see that on invoices view.

Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

what does that is l10n_br_account
from
http://bazaar.launchpad.net/~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0/files

But don't loose just time trying to understand all that. Just take care of
what I said please.

Thanks.

On Mon, Apr 8, 2013 at 8:50 AM, Antoine(OpenERP) <<email address hidden>
> wrote:

> You mean a fiscal position for every invoice line?
> I tried the l10n_br and couldn't see that on invoices view.
>
> --
> You received this bug notification because you are subscribed to OpenERP
> Addons.
> https://bugs.launchpad.net/bugs/1154594
>
> Title:
> [7.0] update taxes in invoices do ignore fiscal positions
>
> Status in OpenERP Addons (modules):
> New
>
> Bug description:
> Hello,
>
> create a new DB with l10n_fr
> standard VAT at nornal rate is 19.6.
> with l10n_fr I also have the fiscal position for export or export with
> european union.
>
> So here is how to reproduce
> create a product with defaul sale vat set
> Create a customer invoice
> Do not set any fiscal position
> create a invoice line with this product
> save your invoice with this VAT
>
> Now, change the fiscal position to export
> and click on Update (beside taxes)
>
> Problem : vat is not replaced by the destination vat set in the fiscal
> position
>
> Note that if you set the fiscal position before creating your invoice
> line, it works as expected.
>
> see video :
>
> https://docs.google.com/file/d/0B4GKXfLLiGNtNTlQWjhmMVp3c3c/edit?usp=sharing
>
> Thanks
>
> Frederic
> Camptocamp
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/openobject-addons/+bug/1154594/+subscriptions
>

Changed in openobject-addons:
status: New → Fix Committed
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.