module delivery, create invoice from packing don't recalculate invoice taxes after add delivery lines

Bug #625235 reported by Àngel Àlvarez http://www.nan-tic.com
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
Unassigned

Bug Description

Invoiced created it's invalid, must press "calculate totals" button to recalculate taxes and totals.

Related branches

Revision history for this message
Àngel Àlvarez http://www.nan-tic.com (angel-nan) wrote :
Revision history for this message
JMA(Open ERP) (jma-openerp) wrote :

Hello Angel Alvarez,

Things seem working pretty normal for us.
The amount value is preserved correctly with the taxes.

Could you please specify the exact procedure(steps) you follow to generate the scenario?

Are you having problems with the 'compute taxes' button once the invoice is created?

Let us know exactly what the problem is.

Thank you.

Revision history for this message
Albert Cervera i Areny - http://www.NaN-tic.com (albert-nan) wrote :

Hi JMA,
I haven't tried Àngel's patch, but the issue seems to be pretty obvious from the code itself. The invoicing process always requires that once linies have been created, 'button_compute()' function must be called. That's what the 'stock' module does in 'action_invoice_create()' function, for example.

The 'delivery' module overrides stock.picking's action_invoice_create() function in order to add the new delivery line but forgets to call 'button_compute()' again after the line has been added. This means that if this new delivery line has taxes associated with it, those taxes are not updated in 'account.invoice.tax' table of the invoice.

Revision history for this message
Vinay Rana (OpenERP) (vra-openerp) wrote :

Hello Angel,

Our R&D Teams are focused on the latest OpenERP version, and this issue does not affect it.
Our policy is to keep the changes applied on stable branches to a minimum, in order to limit the regression risks for customers that are in production. This means that bugs reported on Launchpad are fixed in the trunk branch only by default, even if they were reported against other stable versions.
We stand of course ready to backport the change to stable releases if it has an impact on any customer. In this case please report it to our maintenance team via the OpenERP Publisher's Warranty. They will quickly help solve the issue and backport the fix if needed.
Thank you for your understanding!

Changed in openobject-addons:
status: New → Won't Fix
Revision history for this message
Manu (manu-tiedra) wrote :

Vinay,

this issue also happens in trunk so it should be fixed in trunk even if it is not backported.

Thank you.

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

For the record, this was fixed along with bug 816138

Changed in openobject-addons:
importance: Undecided → Low
milestone: none → 6.1
status: Won't Fix → Fix Released
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.