← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 628702] Re: [mrp] action_po_assign() sometimes gets the price from the wrong supplier (5.0)

 

Hello JMA,

You are right, more steps are required to reproduce the bug. I've
updated the bug description with the new steps.

Note: Since v6 the code has changed a bit so I'm attaching a patch for
v6, too.

** Description changed:

  Hi!
  5.0 latest bzr.
  When generating POs, the procurement selects a supplier but sometimes finds the price of another supplier for the product.
  
  Example:
  Supplier A with a supplier price list defined as 'Partner section of the product form'
  Supplier B with a supplier price list defined as 'Partner section of the product form'
  Open a product and delete all the supplier information.
  Add Supplier B first and set qty 1 and price 200. Save. Then add Supplier A and set qty 1 and price 100. Save.
  Now set the sequence (priority) for Supplier B to 2.
  Now Supplier A is the first on the list with the sequence of 1.
- Set some orderpoint and generate POs. 
+ Set some orderpoint and generate POs with compute minimum stock rules.
+ Expected result: The PO should be generated for Supplier A with the price of 100.
  
- Problem: The PO will be generated for Supplier A but the product will have the price of 200 (from Supplier B).
+ Cancel the PO.
+ On the product form set Supplier A sequence to 2.
+ On the product form set Supplier B sequence to 1.
+ Now Supplier B is the first on the list with the sequence of 1.
+ Generate POs with compute minimum stock rules.
+ Expected result: The PO should be generated for Supplier B with the price of 200.
+ 
+ Check both POs, one should be for Supplier A with the price of 100 and one should be for Supplier B with the price of 200.
+ Problem: In the reality, both POs will have the same price, even they are for different suppliers.
  Because partner_id is not passed to price_get(). See the patch attached.

** Patch added: "price_get_fix_v6.patch"
   https://bugs.launchpad.net/openobject-addons/+bug/628702/+attachment/1853340/+files/price_get_fix_v6.patch

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/628702

Title:
  [mrp] action_po_assign() sometimes gets the price from the wrong
  supplier (5.0)

Status in OpenERP Modules (addons):
  Triaged

Bug description:
  Hi!
  5.0 latest bzr.
  When generating POs, the procurement selects a supplier but sometimes finds the price of another supplier for the product.

  Example:
  Supplier A with a supplier price list defined as 'Partner section of the product form'
  Supplier B with a supplier price list defined as 'Partner section of the product form'
  Open a product and delete all the supplier information.
  Add Supplier B first and set qty 1 and price 200. Save. Then add Supplier A and set qty 1 and price 100. Save.
  Now set the sequence (priority) for Supplier B to 2.
  Now Supplier A is the first on the list with the sequence of 1.
  Set some orderpoint and generate POs with compute minimum stock rules.
  Expected result: The PO should be generated for Supplier A with the price of 100.

  Cancel the PO.
  On the product form set Supplier A sequence to 2.
  On the product form set Supplier B sequence to 1.
  Now Supplier B is the first on the list with the sequence of 1.
  Generate POs with compute minimum stock rules.
  Expected result: The PO should be generated for Supplier B with the price of 200.

  Check both POs, one should be for Supplier A with the price of 100 and one should be for Supplier B with the price of 200.
  Problem: In the reality, both POs will have the same price, even they are for different suppliers.
  Because partner_id is not passed to price_get(). See the patch attached.