0029736: Wrong accounting of the cost adjusment lines under some circumstancies (Goods Movement)

Description

Wrong accounting of the cost adjusment lines under some circumstancies (Goods Movement)

Steps To Reproduce

As group Admin role,
  Go to Product window:
    create a new one
    Name: test
  Add a purchase Price list: Tarifa bebidas
  unit/list price: 5

Costing Rule assigned to the organization should be "Average".

Go to Purchase Order:
   Create a new one for this product: qty: 10
Complete it.
Go to Goods receipt window,
  Create the lines from the Purchase Order created before.
  Storage Bin: RN- 2-0-0
Complete it.

Go to Goods movement,
  create a new one with this product.
  From: RN-2-0-0
  to: RN-0-0-0
Complete it

Go to Product window:
  create a new one
  Type: service
  name: transport
Accounting tab:
Asset Account : 62400 Transportes
Product Expense : 62400 Transportes

Go to Landed Cost Type document:
select the Distribution by Amount algorithm
select the product: Transport

Create a landed cost document --> Cost 20 assigned to the Goods Receipt created before. Process it

Launch the Costing Background process and realise that the transactions have been calculted.

Go to Cost Adjustment document and Post it.
The entries posted are the ones related to the Goods Movement transaction which needs also to be adjusted.

Realise that the journal entry is:

35000 Productos terminados A Debit: 40.00
61000 Variación de existencias de mercaderías Credit: 40.00

That is wrong, as the net effect of the adjustment accounting should be 0.

Proposed Solution

Therefore correct posting should be:

(1) In the case of NOT allowing Negative Posting.

The journal entry for the Movement From transaction must be:

61000 Variación de existencias de mercaderías Debit: 20.00
35000 Productos terminados A Credit: 20.00

The journal entry for the 'Movement To' transaction must be:

35000 Productos terminados A Debit: 20.00
61000 Variación de existencias de mercaderías Credit: 20.00

(2) In the case of allowing Negative Posting.

The journal entry for the Movement From transaction must be:

35000 Productos terminados A Debit: -20.00
61000 Variación de existencias de mercaderías Credit: -20.00

The journal entry for the 'Movement To' transaction must be:

35000 Productos terminados A Debit: 20.00
61000 Variación de existencias de mercaderías Credit: 20.00

Tags

Approved

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

blocks

defect

0029720

3.0PR15Q3

closed

aferraz

Wrong accounting of the cost adjusment lines under some circumstancies (Goods Movement)

Notes
(0076911) hgbot (developer) 2015-04-30 13:11	Repository: erp/backports/3.0PR15Q1.4 Changeset: 95ec611b34c2823a11a4b98eda581657168e2216 Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Wed Apr 29 13:56:15 2015 +0200 URL: http://code.openbravo.com/erp/backports/3.0PR15Q1.4/rev/95ec611b34c2823a11a4b98eda581657168e2216 [^] Fixes issue 29736: Wrong accounting in Cost Adjusment Wrong accounting of the cost adjusment lines in case of Goods Movement From. In this case, accounting should consider negative amounts, in order to have a net effect of the goods movement adjustment accounting as 0. --- M src/org/openbravo/erpCommon/ad_forms/DocCostAdjustment.java ---

(0076912) umartirena (viewer) 2015-04-30 13:12	Code Review + Testing OK

Issue History
Date Modified	Username	Field	Change
2015-04-29 14:25	aferraz	Type	defect => backport
2015-04-29 14:25	aferraz	Target Version	3.0PR15Q3 => 3.0PR15Q1.4
2015-04-30 13:11	hgbot	Checkin
2015-04-30 13:11	hgbot	Note Added: 0076911
2015-04-30 13:11	hgbot	Status	scheduled => resolved
2015-04-30 13:11	hgbot	Resolution	open => fixed
2015-04-30 13:11	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/backports/3.0PR15Q1.4/rev/95ec611b34c2823a11a4b98eda581657168e2216 [^]
2015-04-30 13:12	umartirena	Review Assigned To	=> umartirena
2015-04-30 13:12	umartirena	Note Added: 0076912
2015-04-30 13:12	umartirena	Status	resolved => closed
2015-04-30 13:12	umartirena	Fixed in Version	=> 3.0PR15Q1.4
2015-04-30 14:08	dmitry_mezentsev	Tag Attached: Approved