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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0034067
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 04. Warehouse managementminoralways2016-09-23 10:142016-09-27 21:59
ReporterngarciaView Statuspublic 
Assigned Toinigosanchez 
PriorityimmediateResolutionfixedFixed in Version3.0PR16Q4
StatusclosedFix in branchFixed in SCM revision65f08702a333
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tocaristu
Web browser
ModulesCore
Regression levelPre packaging ( pi )
Regression date2016-09-08
Regression introduced in releasepi
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/b4a8ad7a445ff946888867836f3c3c80cb988f2a [^]
Triggers an Emergency PackNo
Summary

0034067: No UOM Conversion Rate between UOMs found warning shown although the product does not have a 2nd UOM configured

DescriptionNo UOM Conversion Rate between UOMs found warning shown although the product does not have a 2nd UOM configured
Steps To ReproduceAs group admin role:
   Create a new product
   Add it to a Goods Receipt
   Check the following warning is shown and it should not:
      No UOM Conversion Rate between UOMs found
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
caused by design defect 00323663.0PR16Q4 closedinigosanchez callout invocation ineficiencies 

-  Notes
(0090211)
caristu (developer)
2016-09-23 16:14

This problem is caused by the following line: https://code.openbravo.com/erp/devel/pi/file/a570fdbe9083/src/org/openbravo/erpCommon/ad_callouts/SimpleCallout.java#l327 [^]

Previously it was necessary to send the "null" value as part of the callout execution result, this way the parseCalloutResponse() of the FIC was able to retrieve the null values properly. This now is not necessary in SimpleCallouts due to its new internal implementation.
(0090304)
hgbot (developer)
2016-09-27 16:28

Repository: erp/devel/pi
Changeset: 65f08702a33332adc9d9bff1a5a278d8d1326487
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Tue Sep 27 16:26:36 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/65f08702a33332adc9d9bff1a5a278d8d1326487 [^]

Fixes issue 34067: No UOM Conversion Rate between UOMs found warning shown.

This problem is related with Improve performance callouts project. The problem
was related with how to manage null and "null" values.

Previously it was necessary to send the "null" value as part of the callout
execution result because StringBuilder in CalloutInfo class couldn't accept
null Objects values. Then, this way the parseCalloutResponse() of the FIC
was able to retrieve the null values properly. This now is not necessary in
SimpleCallouts due to its new internal implementation.

It have been resolved this problem by sending null Object as part of the
callout execution result in both cases: When a "null" String or null
Object is recieved. Now the problem it has fixed and the null and "null"
values are managing properly.

---
M src/org/openbravo/erpCommon/ad_callouts/SimpleCallout.java
M src/org/openbravo/erpCommon/ad_callouts/SimpleCalloutInformationProvider.java
---
(0090313)
caristu (developer)
2016-09-27 19:10

Verified
(0090314)
hudsonbot (developer)
2016-09-27 21:59

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

- Issue History
Date Modified Username Field Change
2016-09-23 10:14 ngarcia New Issue
2016-09-23 10:14 ngarcia Assigned To => Triage Finance
2016-09-23 10:14 ngarcia Modules => Core
2016-09-23 10:14 ngarcia Triggers an Emergency Pack => No
2016-09-23 10:14 ngarcia Issue Monitored: networkb
2016-09-23 12:54 aferraz Assigned To Triage Finance => platform
2016-09-23 12:56 aferraz Relationship added caused by 0032366
2016-09-23 16:03 caristu Regression level => Pre packaging ( pi )
2016-09-23 16:03 caristu Regression date => 2016-09-08
2016-09-23 16:03 caristu Regression introduced in release => pi
2016-09-23 16:03 caristu Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/b4a8ad7a445ff946888867836f3c3c80cb988f2a [^]
2016-09-23 16:14 caristu Note Added: 0090211
2016-09-26 15:58 inigosanchez Status new => scheduled
2016-09-26 15:58 inigosanchez Assigned To platform => inigosanchez
2016-09-27 16:27 inigosanchez Review Assigned To => caristu
2016-09-27 16:28 hgbot Checkin
2016-09-27 16:28 hgbot Note Added: 0090304
2016-09-27 16:28 hgbot Status scheduled => resolved
2016-09-27 16:28 hgbot Resolution open => fixed
2016-09-27 16:28 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/65f08702a33332adc9d9bff1a5a278d8d1326487 [^]
2016-09-27 19:10 caristu Note Added: 0090313
2016-09-27 19:10 caristu Status resolved => closed
2016-09-27 19:10 caristu Fixed in Version => 3.0PR16Q4
2016-09-27 21:59 hudsonbot Checkin
2016-09-27 21:59 hudsonbot Note Added: 0090314


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker