[5.0] Product change on invoice line

Bug #616202 reported by Quentin THEURET @Amaris
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Status tracked in Trunk
5.0
Won't Fix
Undecided
Unassigned
Trunk
Fix Released
Undecided
JMA(Open ERP)

Bug Description

On invoice line, when you change the product, the UoM and the description aren't updated.

Revision history for this message
Quentin THEURET @Amaris (qtheuret) wrote :

I attach a patch for the modification of the UoM when the product change.

For the update of the description, the behavior of the system for sale is to replace the description by the product name on sale order line and on invoice line, the behavior is to no update the description.

Should the behavior be always same ?

Changed in openobject-addons:
status: New → Confirmed
Changed in openobject-addons:
assignee: nobody → JMA(Open ERP) (jma-openerp)
Revision history for this message
JMA(Open ERP) (jma-openerp) wrote :

Hello Quentin,

Would you please apply this patch and notify us?

Thanks.

Revision history for this message
JMA(Open ERP) (jma-openerp) wrote :

We would like to ask our usability experts about what should be the actual behaviour before & after the patch, Experts please guide us.

Thank you.

Revision history for this message
Quentin THEURET @Amaris (qtheuret) wrote :

Hello JMA,

You patch works fine when you clear the product_id field. But if you change the product without cleared the field, the problem occurs again.

My patch fix the problem for the UoS but not fix the problem for the description because I wait for the response about the unicity of behavior about lines (in invoice, in order, in purchase, ...)

A combinaison of your patch and my patch fix the problem when clear the field and when modify the product without clear the field before.

Revision history for this message
JMA(Open ERP) (jma-openerp) wrote :

Hello Quentin,

Thank you for pointing it out.
Would you please apply the patch and notify us?

Thank you.

Revision history for this message
Quentin THEURET @Amaris (qtheuret) wrote :

Hello JMA,

The patch works fine.

Thank you.

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Hello Usability Experts,

I would like request you to share your views here.
This is really an important issue.

1. Do we really require 'name' to pass on onchange_product_id?
2. Same thing applies for uom.

Thanks.

Revision history for this message
Ferdinand (office-chricar) wrote :

IMHO changing the product should fetch the values for the new product, regardless what was entered before.
Nevertheless I admit that in some cases the user might have to reenter some data, but in doubt the in the majority of cases fetching the new values will be correct.

I just want to point to the fact that the uom is later stored in the stock_move and I have had already some bad issues having more than ONE uom in stock moves (product_uom) - there are some "group by uom" returning more than one record giving wrong figures.
but that might be another issue that the product_uom must not change over time = it shouldn't be stored in stock_move at all.

Revision history for this message
JMA(Open ERP) (jma-openerp) wrote :

Hello Quentin,

Things work pretty well with the latest trunk version.
It has been resolved.
Thanks.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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