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 <[email protected]>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 _______________________________________________ Mailing list: https://launchpad.net/~c2c-oerpscenario Post to : [email protected] Unsubscribe : https://launchpad.net/~c2c-oerpscenario More help : https://help.launchpad.net/ListHelp
