Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035213Openbravo ERPA. Platformpublic2017-02-09 16:312017-02-27 12:49
Reportermaite 
Assigned Toinigosanchez 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionno change required 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035213: Callout problem when working with comma (,) as decimal separator for euro references defined in format.xml
DescriptionValue of the field is not properly loaded in that situation, so callout calculations are wrong
Steps To Reproduce1. Edit config/Format.xml file to set comma as decimal separator for "euroXXX" references:

<Number name="euroEdition" decimal="," grouping="." formatOutput="#0.00" formatInternal="#0.00" />

2. Compile application using: ant smartbuild
3. Register new Sales Invoice record for "España" org and any BP. Add any line
4. Access to "Exchange Rate" tab, create new record setting USD, set "Foreign Amount"= 5 and realize that "Rate" field has not been properly calculated

Problem is that "SE_CalculateExchangeRate.java" callout is loading value of 500 (instead of 5) in "foreignAmt" field. Seems to be a generic problem more than a specific problem of this callout
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to design defect 00323663.0PR16Q4 closed inigosanchez callout invocation ineficiencies 
related to design defect 0035336 new Triage Platform Base Several improvements in Format.xml behaviour/definition should be done. 
related to defect 0035334 closed inigosanchez Callout problem when working with comma (,) as decimal separator 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-02-09 16:31maiteNew Issue
2017-02-09 16:31maiteAssigned To => platform
2017-02-09 16:31maiteModules => Core
2017-02-09 16:31maiteResolution time => 1488150000
2017-02-09 16:31maiteTriggers an Emergency Pack => No
2017-02-09 16:31maiteIssue Monitored: networkb
2017-02-10 09:07alostaleRelationship addedrelated to 0032366
2017-02-17 08:51inigosanchezAssigned Toplatform => inigosanchez
2017-02-17 08:51inigosanchezStatusnew => scheduled
2017-02-19 12:02jfrancesIssue Monitored: jfrances
2017-02-21 17:21maiteResolution time1488150000 =>
2017-02-22 10:55maiteRelationship addedrelated to 0035334
2017-02-22 11:03inigosanchezReview Assigned To => alostale
2017-02-22 11:03inigosanchezNote Added: 0094529
2017-02-22 11:03inigosanchezStatusscheduled => closed
2017-02-22 11:03inigosanchezResolutionopen => no change required
2017-02-22 11:16inigosanchezRelationship addedrelated to 0035336
2017-02-22 11:17inigosanchezNote Added: 0094530
2017-02-23 19:02juabaezIssue Monitored: juabaez
2017-02-27 12:49inigosanchezSummaryCallout problem when working with comma (,) as decimal separator => Callout problem when working with comma (,) as decimal separator for euro references defined in format.xml

Notes
(0094529)
inigosanchez   
2017-02-22 11:03   
As Openbravo wiki [1] says: "You can define a decimal and grouping separator per format type, but the qtyEdition format is the one that defines them. Makes no sense to define different decimal and grouping per format type."

[1].- http://wiki.openbravo.com/wiki/Format.xml#Important_Notes [^]
(0094530)
inigosanchez   
2017-02-22 11:17   
A design defect issue [1] has been created in order to improve several problems of current Format.xml and topics related with format.

[1].- https://issues.openbravo.com/view.php?id=35336 [^]