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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0029298
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementmajoralways2015-03-16 18:102015-05-07 22:16
ReporterVictorVillarView Statuspublic 
Assigned Toaferraz 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisioncd77e2cf07ed
ProjectionnoneETAnoneTarget Version3.0PR15Q3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned Tovmromanos
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0029298: It should not be possible to change the unit of measure conversion

DescriptionIt should not be possible to change the unit of measure conversion
Steps To ReproduceIt is necessary to show the fields/tabs related with second uom. (conversion tab in UOM window, Uom tab in product window...)
As group Admin role,
  Create a new conversion for Unit. Second unit of measure: hour
 Multiply rate by: 3
 Divide Rate by: 0.33

Assing this second uom to product: Agua

Create a new physical inventory for one product
 add: 40 hours
 main unit of measure: 120
Process it.

After that change the the conversion to:
Multiply rate by:0.1
Divide Rate by: 10

Create a new physical inventory
 add: 100 hours
 main unit of measure: 10

So the total is:
 Unit: 130
 second unit of measure: 140 (hours)

So the conversion is: 140/130 = 1.07

After that change the the conversion to:
Multiply rate by:3
Divide Rate by: 0.33

Go to Sales order:
 Document type: POS order
 create one register for Agua
   second qty: 40
   main unit of measure: 120
Complete it.

Go to Goods Shipment created.
 realise that the second unit of measure qty is: 43,076923077
 and the main unit of measure qty is: 120

The reason is: m_inout_create function
 
v_ActualQty:=ROUND(v_storage.QtyOrder*(v_LocatorQty/v_storage.QtyOnHand), v_storage.stdprecision) ;

where v_storage.QtyOrder= 140 v_LocatorQty= 120 v_storage.QtyOnHand= 130



 
Proposed SolutionIt is necessary to decide if it should be possible to change the rate if there is stock of any product with these units
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0076605)
aferraz (manager)
2015-04-17 13:06
edited on: 2015-04-23 13:03

 Test plan:

Go to "Unit of measure" window. Select one record and move to "Conversion" tab.
Create one record and save it.
- Realize "Uom To", "Multiplicate Rate by" and "Divide Rate by" fields are no longer editable.

Check "Conversion" tab shows now the following errors:

A insert error when:
- Inserting a new UOM Conversion record and another one already exists using the same uomFrom - uomTo
- Updating the uom field of a UOM Conversion record and another one already exists using the same uomFrom - uomTo
- Activating a UOM Conversion record and another one already exists using the same uomFrom - uomTo

An update error when:
- Updating the multiplyrateby field or the dividerateby field of a UOM Conversion record
(0076737)
hgbot (developer)
2015-04-24 08:50

 Repository: erp/devel/pi
Changeset: cd77e2cf07ed8403aa07f8a553e69d900d01a67a
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Apr 23 12:57:48 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cd77e2cf07ed8403aa07f8a553e69d900d01a67a [^]

Fixes issue 29298: It should not be possible to change the UOM Conversion

"To UOM", "Multiple Rate By" and "Divide Rate By" fields are now not updateable. Also, an event handler has been created to show different errors following this criteria:

A insert error when:
Inserting a new UOM Conversion record and another one already exists using the same uomFrom - uomTo
Updating the uom field of a UOM Conversion record and another one already exists using the same uomFrom - uomTo
Activating a UOM Conversion record and another one already exists using the same uomFrom - uomTo

An update error when:
Updating the multiplyrateby field or the dividerateby field of a UOM Conversion record

---
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_MESSAGE.xml
A src/org/openbravo/event/UOMConversionEventHandler.java
---
(0076738)
vmromanos (manager)
2015-04-24 08:52

 Code review + testing OK
(0077151)
hudsonbot (developer)
2015-05-07 22:16

 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/a1817f84bd8b [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2015-03-16 18:10 VictorVillar New Issue
2015-03-16 18:10 VictorVillar Assigned To => Sandrahuguet
2015-03-16 18:10 VictorVillar Modules => Core
2015-03-16 18:10 VictorVillar Resolution time => 1429135200
2015-03-16 18:10 VictorVillar Triggers an Emergency Pack => No
2015-03-16 18:10 VictorVillar Issue Monitored: networkb
2015-03-18 17:02 reinaldoguerra Assigned To Sandrahuguet => reinaldoguerra
2015-03-18 17:02 reinaldoguerra Status new => scheduled
2015-03-30 13:59 jonalegriaesarte Target Version 3.0PR15Q2 => 3.0PR15Q3
2015-04-15 16:10 aferraz Assigned To reinaldoguerra => aferraz
2015-04-17 13:06 aferraz Note Added: 0076605
2015-04-23 13:03 aferraz Note Edited: 0076605 View Revisions
2015-04-24 08:50 hgbot Checkin
2015-04-24 08:50 hgbot Note Added: 0076737
2015-04-24 08:50 hgbot Status scheduled => resolved
2015-04-24 08:50 hgbot Resolution open => fixed
2015-04-24 08:50 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cd77e2cf07ed8403aa07f8a553e69d900d01a67a [^]
2015-04-24 08:52 vmromanos Review Assigned To => vmromanos
2015-04-24 08:52 vmromanos Note Added: 0076738
2015-04-24 08:52 vmromanos Status resolved => closed
2015-05-07 22:16 hudsonbot Checkin
2015-05-07 22:16 hudsonbot Note Added: 0077151

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker