Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0028029
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2014-10-30 13:152014-12-30 23:24
ReporterpsanjuanView Statuspublic 
Assigned Toumartirena 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision5c9fcf65a9af
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0028029: [Costing] Negative Stock Correction adjustments should NOT be marked as "Unit Cost".

Description[Costing] Negative Stock Correction adjustments should NOT be marked as "Unit Cost".
Steps To ReproduceIn QA Testing creates a product named "Costing V221". In Price tab enter a unit/list price of 15 €

Creates a purchase order dated on 11/10/2014 for 500 units. Check that the price shown is 15 €/unit. Book the order

Creates a purchase order dated on 15/10/2014 for 700 units. Change the price to 12 €/unit. Book the order

Creates a goods receipt dated on 16/10/2014 from the first order of 150 units. Enter a storage bin and complete the receipt.

Run costing background process so cost are calculated therefore receipt can be posted. Value posted is 2250 (150 units *15 €/unit)

Creates a goods shipment dated on 20/10/2014 for 600 units. Complete the shipment.

Run costing background process so cost are calculated therefore shipment can be posted. Value posted is 9000 (600 units *15 €/unit)

Creates a goods receipt dated on 22/10/2014 from the second order. Receipt 500 units.

Run costing background process so cost are calculated therefore receipt can be posted. Value posted is 6000 (500 units * 12 €/unit)

Realize that a negative stock correction cost adjustment has been created for an amount of 1350 (2250-9000+6000+X)/50 = 12

This negative stock correction should not be marked as "Unit Cost" but it is. See image attached.
Proposed SolutionThis negative stock correction should not be marked as "Unit Cost" but it is. See image attached.
TagsNo tags attached.
Attached Filespng file icon issue_28029.png [^] (97,687 bytes) 2014-10-30 13:38

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0028113 closedumartirena [Costing] wrong negative stock correction adjustment 

-  Notes
(0071283)
hgbot (developer)
2014-10-30 18:58

 Repository: erp/devel/pi
Changeset: 5c9fcf65a9af22480c0823b1b9cf25874472e8c6
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Thu Oct 30 18:57:32 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/5c9fcf65a9af22480c0823b1b9cf25874472e8c6 [^]

Fixes issue 28029: Negative Stock Correction adjustments not marked as Unit Cost

---
M src/org/openbravo/costing/CostingServer.java
---
(0071409)
hgbot (developer)
2014-11-04 17:56

 Repository: erp/devel/pi
Changeset: c6389ceb2f1177f9c1a927b9d2a646c8b63ca2ec
Author: Unai Martirena <unai.martirena <at> openbravo.com>
Date: Tue Nov 04 17:55:30 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c6389ceb2f1177f9c1a927b9d2a646c8b63ca2ec [^]

Related to issue 28029: Prevent extra negative stock adjustment line

AverageCostAdjustment it was creating an extra negative stock adjustment line when there was already an adjustment line of negative stock. This was happening on a receipt with previuos stock in negative, that leaves the stock still in negative after the completion of the receipt.
To prevent this and extra check has been added in AverageCostAdjustment: If baseCAL is of type negative stock, then dont add a new negative stock line.

---
M src/org/openbravo/costing/AverageCostAdjustment.java
---
(0071469)
psanjuan (manager)
2014-11-06 18:02

 Verified that negative stock correction are now do not created as Unit Cost.
(0073005)
hudsonbot (developer)
2014-12-30 23:23

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/6525fe229e06 [^]
Maturity status: Test
(0073016)
hudsonbot (developer)
2014-12-30 23:24

 A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/6525fe229e06 [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2014-10-30 13:15 psanjuan New Issue
2014-10-30 13:15 psanjuan Assigned To => dmiguelez
2014-10-30 13:15 psanjuan Modules => Core
2014-10-30 13:15 psanjuan Triggers an Emergency Pack => No
2014-10-30 13:37 psanjuan Steps to Reproduce Updated View Revisions
2014-10-30 13:37 psanjuan Proposed Solution updated
2014-10-30 13:38 psanjuan File Added: issue_28029.png
2014-10-30 13:39 psanjuan Assigned To dmiguelez => umartirena
2014-10-30 18:58 hgbot Checkin
2014-10-30 18:58 hgbot Note Added: 0071283
2014-10-30 18:58 hgbot Status new => resolved
2014-10-30 18:58 hgbot Resolution open => fixed
2014-10-30 18:58 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/5c9fcf65a9af22480c0823b1b9cf25874472e8c6 [^]
2014-11-04 17:56 hgbot Checkin
2014-11-04 17:56 hgbot Note Added: 0071409
2014-11-06 18:02 psanjuan Note Added: 0071469
2014-11-06 18:02 psanjuan Status resolved => closed
2014-11-06 18:06 psanjuan Relationship added related to 0028113
2014-12-30 23:23 hudsonbot Checkin
2014-12-30 23:23 hudsonbot Note Added: 0073005
2014-12-30 23:24 hudsonbot Checkin
2014-12-30 23:24 hudsonbot Note Added: 0073016

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker