stock real time accounting - wrong values for stock moves with purchase unit != UoM

Bug #889093 reported by Ferdinand
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Medium
OpenERP R&D Addons Team 2

Bug Description

I set purchase unit to hektoliter and UOM to Liter.
hektoliter = 100 liter

a) continental
-- generates wrong values for stock moves (does not take into account the
hektoliter price)

b) anglo -
-- generates wrong values for stock moves (does not take into account the
hektoliter price)
-- do not know where the price diff is coming from

Related branches

Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Dr Ferdinand,

I have tested your scenario with all possibilities, and I totally agree with your last screen-shot's problem. This issue is confirmed

This problem is occurs when our purchase order line's UoM is differ then the purchase UoM .

Now for your second issue, which is related to price different account, If product have a "Average Price" and Invoice method based on reception, and at the delivery time you have changed the cost of product then remaining cost will be goes into "price different account".

So may be in your 1st and 2nd screen-shots you have changed the cost of goods at delivery time, that's why you can see the remaining cost into price difference account.

So would you please check your 2nd issue again and notify us, If you still face same problem without above explained scenario then please elaborate more or provide precious example on this.

Thanks and waiting for your reply!

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Ferdinand (office-chricar) wrote :

Hello Amit!

I didn't alter the purchase price while validating the receipt

it looks like that the difference is ( product unit-price - PO price ) * PO quantity

I agree this is "a" price difference, but this one has NOT to be posted.

the price difference is the difference between invoice and receipt postings.

please see the 2 attachments

Revision history for this message
Ferdinand (office-chricar) wrote :

account move lines to be generated

Revision history for this message
Ferdinand (office-chricar) wrote :
Changed in openobject-addons:
status: Incomplete → New
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello DR Ferdinand,

I totally agree with you, the 'price difference account' comes when the difference occurs between invoice and receipt, and it is working fine.

Currently I am confirming this issue only for the issue#1 which is related to wrong entires on stock valuation account, If purchase order line's UoM is differ then the purchase UoM which is described on comment#3 's screen-shot.

For your 2nd issue related to price difference account, you have to post another bug report.

Thank you!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
importance: Undecided → Medium
status: New → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Kirti Savalia(OpenERP) (ksa-openerp) wrote :

Hello,

Thanks For Reporting.
It has been Fixed in http://bazaar.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-889093-ksa
revision-id: <email address hidden>
revno: 5732
It will be available in trunk soon.

Thanks.

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

This is a very sensitive area, and I'd like to have a YAML test that covers it before considering it fixed.
Technically the fix has been merged in trunk/6.1 at revision 5805 revid: <email address hidden> so the bug is fixed, but I'd like to put it back to state "Confirmed" to indicate that an action is still needed.

A small YAML test should be added for purchase module, using a product with real-time stock valuation and a UoM that is not the default product UoM. The test should fail at revision 5804, and pass after revision 5805.

Thanks!

Changed in openobject-addons:
milestone: none → 6.1
status: Fix Committed → Confirmed
Revision history for this message
Ferdinand (office-chricar) wrote :

please can you check this also
https://bugs.launchpad.net/bugs/890193

to make complete yaml checks, please see this
https://bugs.launchpad.net/bugs/892262

Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Kirti Savalia(OpenERP) (ksa-openerp) wrote :

Hello,

YML has been Fixed in https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-889093-yml-ksa
revision-id: <email address hidden>
revno: 5821

Thanks.

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote : Re: [Bug 889093] Re: stock real time accounting - wrong values for stock moves with purchase unit != UoM

On 11/29/2011 09:20 PM, Ferdinand @ Camptocamp wrote:
> please can you check this also
> https://bugs.launchpad.net/bugs/890193
>
> to make complete yaml checks, please see this
> https://bugs.launchpad.net/bugs/892262

@Ferdinand: I do not see any connection between these bugs. Please do
not hijack other bugs to request treating other unrelated bugs with
higher priority.
Bugs are qualified separately according to their specific priorities.
The other bugs your mentioned are very low priority, and you're making
the developers spend time on them rather than fixing more important
ones. Thank you for your cooperation.

@Kirti: please ignore such unrelated requests in the future

Revision history for this message
Ferdinand (office-chricar) wrote :

@Olivier
Quote
"I do not see any connection between these bugs"
Unfortunately this is exactly the problem - anglo saxon accounting can only imlemented in production if ALL components work correctly.

https://bugs.launchpad.net/bugs/890193 mentions that it takes a wrong account.

Revision history for this message
Raphael Collet (OpenERP) (rco-openerp) wrote :

landed in trunk
Revision:5805
revid:<email address hidden>

Changed in openobject-addons:
status: Fix Committed → Fix Released
Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

For the record, this bug and patch are superseded by bug 923191, that seems to better address the root cause

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.