Sale Order: Change in pricelist should affect order lines

Bug #435291 reported by forstera
42
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Won't Fix
Wishlist
Dhruti Shastri(OpenERP)

Bug Description

Hello,
When we duplicate a sale order, the values on the bottom of the form (Untaxed Amount, Taxes, Total) looked like to be freezed.
I can add new lines, remove all lines to my sale order and click the button 'Compute', no value changes.

In the same way, but all the time (not only duplicate sale orders) if I change my price list of my sale order and click the 'Compute' button, my prices dont change. I need to re-introduce my articles to affect the new pricelist.

Revision history for this message
Numérigraphe (numerigraphe) wrote : Re: [Bug 435291] [NEW] When duplicating a sale order, total fields not updated

forstera a écrit :
> Public bug reported:
>
> Hello,
> When we duplicate a sale order, the values on the bottom of the form (Untaxed Amount, Taxes, Total) looked like to be freezed.
> I can add new lines, remove all lines to my sale order and click the button 'Compute', no value changes.
>
FYI I couldn't reproduce this behavior with yesterday's v5.0-bzr.
> In the same way, but all the time (not only duplicate sale orders) if I
> change my price list of my sale order and click the 'Compute' button, my
> prices dont change. I need to re-introduce my articles to affect the
> new pricelist.
As far as I can tell, this is the "expected" behavior - I mean it's
designed to do just that.
Lionel.

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote : Re: When duplicating a sale order, total fields not updated

Hello Arnaud,

Are you sure you are using the latest versions of server and addons codes?

This is neither 'producible' nor 'expected' here.

Those functional fields should get updated when SO Line(s) change(s)(without regard to its a new SO or duplicated). And they are updated normally.

I would like to request you to check again.

Thanks.

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

Ideally,

One should select a pricelist first and then add order lines.
Agreeing to your point, Let us try to sort out this issue.

Thank you for the report.

Revision history for this message
forstera (arnaud-forster-deactivatedaccount) wrote : Re: [Bug 435291] Re: When duplicating a sale order, total fields not updated

Hello Jay,

I'm just upgrading right now. But the problem is that my table (which
contains data) cannot be upgrated ... it seems to be the source of the
problem.

2009-09-24 08:07:16,262] WARNING:orm:column 'amount_tax' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !
[2009-09-24 08:07:16,287] WARNING:orm:column 'amount_untaxed' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !
[2009-09-24 08:07:16,290] WARNING:orm:column 'amount_total' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !
[2009-09-24 08:07:16,346] WARNING:orm:column 'amount_untaxed' in table
'purchase_order' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,359] WARNING:orm:column 'amount_tax' in table
'purchase_order' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,381] WARNING:orm:column 'amount_total' in table
'purchase_order' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,442] WARNING:orm:column 'amount_untaxed' in table
'account_invoice' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,450] WARNING:orm:column 'amount_tax' in table
'account_invoice' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,472] WARNING:orm:column 'residual' in table
'account_invoice' has changed type (DB = float8, def = float) but unable
to migrate this change !
[2009-09-24 08:07:16,489] WARNING:orm:column 'amount_total' in table
'account_invoice' has changed type (DB = float8, def = float) but unable
to migrate this change !

[2009-09-24 08:07:17,891] WARNING:orm:column 'amount_tax' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !
[2009-09-24 08:07:17,916] WARNING:orm:column 'amount_untaxed' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !
[2009-09-24 08:07:17,919] WARNING:orm:column 'amount_total' in table
'sale_order' has changed type (DB = float8, def = float) but unable to
migrate this change !

Meilleures salutations.

____________________________
MW Programmation SA
Arnaud Forster
IT Manager

Rue Charles Schäublin 2 | CH-2735 Malleray | Suisse
phone:+41(0)32 491 65 30 | fax:+41(0)32 491 65 35
email: <email address hidden> | web: www.mwprogrammation.ch

Jay (Open ERP) a écrit :
> Hello Arnaud,
>
> Are you sure you are using the latest versions of server and addons
> codes?
>
> This is neither 'producible' nor 'expected' here.
>
> Those functional fields should get updated when SO Line(s)
> change(s)(without regard to its a new SO or duplicated). And they are
> updated normally.
>
> I would like to request you to check again.
>
> Thanks.
>
>

Revision history for this message
forstera (arnaud-forster-deactivatedaccount) wrote :

Hello Jay,

I just upgraded to the last version and this solved the problem. I
tested the patch concerning the warnings during the update (
https://bugs.launchpad.net/openobject-server/+bug/433886

and this seems to work, too.

.. so, the last thing is the problem of the change of the pricelist

Thanks for your efforts

Regards

Meilleures salutations.

____________________________
MW Programmation SA
Arnaud Forster
IT Manager

Rue Charles Schäublin 2 | CH-2735 Malleray | Suisse
phone:+41(0)32 491 65 30 | fax:+41(0)32 491 65 35
email: <email address hidden> | web: www.mwprogrammation.ch

Jay (Open ERP) a écrit :
> Hello Arnaud,
>
> Are you sure you are using the latest versions of server and addons
> codes?
>
> This is neither 'producible' nor 'expected' here.
>
> Those functional fields should get updated when SO Line(s)
> change(s)(without regard to its a new SO or duplicated). And they are
> updated normally.
>
> I would like to request you to check again.
>
> Thanks.
>
>

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote : Re: When duplicating a sale order, total fields not updated

Yes, Arnaud.

Thank you for the response.

We are taking care of the same.

Thanks.

affects: openobject-server → openobject-addons
Changed in openobject-addons:
status: New → Confirmed
summary: - When duplicating a sale order, total fields not updated
+ Sale Order: Pricelist should have Onchange()
Changed in openobject-addons:
assignee: nobody → gpa(tiny) (gpa-openerp)
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
invitu (invitu) wrote : Re: Sale Order: Pricelist should have Onchange()

Dear all,

I would ask that instead of having onchange() property, the computation of new prices when pricelist changes would be associated with a separate "Compute listprice" button.

It would allow to have a better control for prices calculation.

The following scenario would work :
- We change the pricelist and do a recalculation of prices for each line in the order
- We could change partner without reassign new prices and by clicking on the "Compute" button, only the tax would be calculated without changing the prices in the order.

Moreover, the same improvement would be very useful in purchase module too.

Regards

Changed in openobject-addons:
assignee: gpa(Open ERP) (gpa-openerp) → DHS(OpenERP) (dhs-openerp)
summary: - Sale Order: Pricelist should have Onchange()
+ Sale Order: Change in pricelist should affect order lines
Revision history for this message
Dhruti Shastri(OpenERP) (dhs-openerp) wrote :

Hello Arnaud and Invitu,

I am attaching the Sale extend Module which will add wizard "Change Pricelist". This wizard will change the priecelist of sale order as you specified.

Please note that It is compatible with Trunk.

Have a check with the module and let us know.

Thanks

Revision history for this message
Kinly (kinlyfordy4u) wrote :

hi...

Is there any need for new module for price list ?
can we use that code in existing sale module ?

Revision history for this message
Dhruti Shastri(OpenERP) (dhs-openerp) wrote :

Hello Kinly,

This is a new feature and we cannot add this in the module of a stable release.

So, here is a new module made compatible with trunk. You may use the same code for your stable module if needed so.

Thanks.

Changed in openobject-addons:
importance: Undecided → Wishlist
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

This was an added development, it may land into extra-addons.
Thanks.

Changed in openobject-addons:
status: In Progress → Won't Fix
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.