← Back to team overview

c2c-oerpscenario team mailing list archive

Re: [Bug 805697] [NEW] Price difference account - incorrect calculation

 

I agree, price difference account can give weird errors, but the logic here
is incorrect.  We should not use the PO Price for any calculation of price
differences.  Please do not implement this as suggested.

This is because we can specify cost at time of goods receipt and price
difference is to be the difference between receipted cost and supplier
invoice, not the difference between PO and invoice.  Priced packing slips
and/or adding freight in to costs is the primary use case where the stated
logic does not work.  It would also make interfacing with extended modules
such as those going on for landed cost calculation that much tougher,
basically requiring an anglo_saxon overwrite as well.

While I have not tested, it will be interesting if recent bugfix to store
price_unit on all inwards stock_moves fixes this issue anyway.  At the very
least it probably provides for an easy refactor of the price difference
logic in account_anglo_saxon.  Just use stock.move price_unit.

On Tue, Jul 5, 2011 at 9:22 AM, Kevin McMenamin
<805697@xxxxxxxxxxxxxxxxxx>wrote:

> Public bug reported:
>
> The accounting entries generated for the price difference account are not
> consistently correct.
> Environment:
> 6.0.2 release 30/06/2011
> anglo-saxon accounting
> PO invoicing control = from picking
> product costing method = average price
> inventory valuation = real time (automated)
>
> When a PO is raised for a product with a purchase price <> average cost AND
> the invoice when processed has a price difference to the PO, THEN
> the price difference value is calculated by (Quantity * (Old Average Price
> - New Average Price)) (+ve value = DR)
> but should be calculated by (Quantity * (Invoice Price - PO Price)) (+ve
> value = DR)
>
> I can provide a worked example if required
>
> ** Affects: openobject-addons
>     Importance: Undecided
>         Status: New
>
> --
> You received this bug notification because you are subscribed to OpenERP
> Addons.
> https://bugs.launchpad.net/bugs/805697
>
> Title:
>  Price difference account - incorrect calculation
>
> Status in OpenERP Modules (addons):
>  New
>
> Bug description:
>  The accounting entries generated for the price difference account are not
> consistently correct.
>  Environment:
>  6.0.2 release 30/06/2011
>  anglo-saxon accounting
>  PO invoicing control = from picking
>  product costing method = average price
>  inventory valuation = real time (automated)
>
>  When a PO is raised for a product with a purchase price <> average cost
> AND
>  the invoice when processed has a price difference to the PO, THEN
>  the price difference value is calculated by (Quantity * (Old Average Price
> - New Average Price)) (+ve value = DR)
>  but should be calculated by (Quantity * (Invoice Price - PO Price)) (+ve
> value = DR)
>
>  I can provide a worked example if required
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/openobject-addons/+bug/805697/+subscriptions
>

-- 
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/805697

Title:
  Price difference account - incorrect calculation

Status in OpenERP Modules (addons):
  New

Bug description:
  The accounting entries generated for the price difference account are not consistently correct.
  Environment: 
  6.0.2 release 30/06/2011
  anglo-saxon accounting
  PO invoicing control = from picking
  product costing method = average price
  inventory valuation = real time (automated)

  When a PO is raised for a product with a purchase price <> average cost AND
  the invoice when processed has a price difference to the PO, THEN
  the price difference value is calculated by (Quantity * (Old Average Price - New Average Price)) (+ve value = DR)
  but should be calculated by (Quantity * (Invoice Price - PO Price)) (+ve value = DR)

  I can provide a worked example if required

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


Follow ups

References