Landed Cost issue / cost issue

[arehman]

(Please no need to say Hi or address anyone in first query. No need also to sign and repeat "Please Help" words. See how i edited this post below. I will try to check your case in the Fitnesse testing engine and paste u the results. Thanks)
I think, in ADempiere there is a problem in costing in case of landed cost. Cost of goods sold is not booking correctly in second time.

lets say,
I have purchased 10 laptops @ 25,000 = 250,000
i have paid 50,000 freight on it.

so my cost of laptops become 250,000 + 50,000 = 300,000
now cost of one laptop is 300,000/10 = 30,000

when i ship one laptop to customer first time system generates Cost of good sold entry (COGS) by 30,000.

when i ship one laptop to customer second time system generates Cost of good sold entry (COGS) by 30,555.55. which is wrong
(basically system is recalculate freight with balance qty 50,000/ 9 = 5,555.55 and add this freight to a cost of laptop 25,000.
25,000 + 5,555.55 = 30,555.55)

I am using average method for costing.

[SETSOFTWARE]

I Have the same problem, but when a Production Header is completed, dou you have any solution or idea of solution?

[arehman]

I have changed its code. You have to change the code to cater this bug as per your requirement.

[SETSOFTWARE]

thanks for your answer.

Could you describe what you did?
I am trying to figure out what code to change. As the problem is in Production, I am looking up M_production.java, Doc_Production.java and MCostDetal.java.

so any advice?

[SETSOFTWARE]

Hello,

well i think i know where i have to change the code.
So what i want is that when Production is completed, the landed cost for raw material, won't be recalculated.
i think if I add an "If" to MCostDetail.java before the line 1185

actual code:

1185 lCost.setCurrentCostPrice(costs);

change for something like:

1185 if( getM_ProductionLine_ID() == 0)
1186 {
1187 lCost.setCurrentCostPrice(costs);
1188 }

I believe that will be enough, but I'm just not sure, because I dont know, if actually after completed production header it must be necesary recalculated landed cost.

Ok I'll be waitng any comment.

[SETSOFTWARE]

Hello,

I've changed the clase MCostDetail.java as I show in the following lines

Code: Select all

1185//don't change current Landed cost  when production is completed
1186                  
1187    if(getM_ProductionLine_ID() == 0)
1188        {
1189      lCost.setCurrentCostPrice(costs);      
1190           }

after test behaviour of Landed Cost in 5 different scenarios i think it work fine.

[red1]

You can test or compare this with the present iDempiere version 1.0.a which has a new Landed Cost enhancement done to it by Heng Sin.

[SETSOFTWARE]

Hello Red1,

I try to complete Production in Idempiere(in order to test landed cost in Production) but i get the same error that i've reported before:

Database Error.: ERROR: null value in column "m_product_id" violates not-null constraint
In Production Window

And

Could not retrieve the cost in single product production

[red1]

You try to give full details what you did to your code in our googlegroup and see what the advice if you can raise a jira tracker on this.

[globalqss]

http://jira.idempiere.com/browse/IDEMPIERE-509

[SETSOFTWARE]

excelent news.
I will try it again.