← Back to team overview

openerp-india team mailing list archive

[Bug 1304804] [NEW] Product Cost Update Bug

 

Public bug reported:

Revision 9779 to openobject-addons has introduced a new significant bug.

The loop in do_change_standard_price used to re-browse the product
record based on the location context.

This has been removed for some reason.

Now, it updates the cost for ALL stock for every location in the outer
browse loop.

To recreate:

Create a product, average stock valuation and real time accounting.
Let's assume 2 internal locations - Output and Stock
Output has quantity of zero
Stock has quantity of 10.
Go to product maintenance and click "- update" to update the cost.

The journals created are double.

The reason is that the main loop on product browses the product for all locations, meaning quantity is 10.
The inner loop on location does not rebrowse the product, and assumes 10 on hand for both locations.

This is a significant problem.

** Affects: openobject-addons
     Importance: Undecided
         Status: New


** Tags: cost product update

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/1304804

Title:
  Product Cost Update Bug

Status in OpenERP Addons (modules):
  New

Bug description:
  Revision 9779 to openobject-addons has introduced a new significant
  bug.

  The loop in do_change_standard_price used to re-browse the product
  record based on the location context.

  This has been removed for some reason.

  Now, it updates the cost for ALL stock for every location in the outer
  browse loop.

  To recreate:

  Create a product, average stock valuation and real time accounting.
  Let's assume 2 internal locations - Output and Stock
  Output has quantity of zero
  Stock has quantity of 10.
  Go to product maintenance and click "- update" to update the cost.

  The journals created are double.

  The reason is that the main loop on product browses the product for all locations, meaning quantity is 10.
  The inner loop on location does not rebrowse the product, and assumes 10 on hand for both locations.

  This is a significant problem.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/1304804/+subscriptions


Follow ups

References