Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0025517Openbravo ERPA. Platformpublic2014-01-20 17:582014-03-06 11:30
Reportercaristu 
Assigned Toshankarb 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0PR14Q3Fixed in Version3.0PR14Q2 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0025517: Wrong calculations on numeric field depending on the formula
DescriptionUsing "." as decimal separator and "," as grouping separator, if we use an arithmetic calculation for enter a value on a numeric field it fails if the first operand is a decimal number.
Steps To Reproduce1) Go to the [Sales Order] window and create one record
2) Go to the [Lines] tab and create a new record
3) In the "Gross Unit Price" field, type the following: =2.3+1
4) Save the record. The calculation returns an invalid value: =2.N0
5) Replace the previous value with =1+2.3
6) Save the record. It returns the correct value: 3.3
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0027698 acknowledged Triage Platform Base Wrong calculations on numeric field depending on the formula when using a comma as decimal separator 
related to defect 00284273.0PR15Q2 closed AugustoMauch Price is not rounded to calculate the line net amount in Create Lines process of Purchase Order 
related to defect 0031901 closed caristu decimal value can't be changed by trigger/observer after manual edition in form view 
causes defect 00270533.0PR14Q4 closed AugustoMauch Error when adding record in grid mode and working with comma as decimal separator 
causes defect 0028747 closed AugustoMauch If a numeric value is entered using a formula, it is saved without being rounded 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-01-20 17:58caristuNew Issue
2014-01-20 17:58caristuAssigned To => AugustoMauch
2014-01-20 17:58caristuModules => Core
2014-01-20 17:58caristuTriggers an Emergency Pack => No
2014-01-20 17:58caristuIssue Monitored: networkb
2014-01-20 18:07caristuResolution time => 1396735200
2014-01-20 18:07caristuversion => pi
2014-01-20 18:07caristuTarget Version => 3.0MP33
2014-03-04 11:30shankarbAssigned ToAugustoMauch => shankarb
2014-03-05 06:57shankarbStatusnew => scheduled
2014-03-05 06:57shankarbNote Added: 0064807
2014-03-05 06:57shankarbfix_in_branch => pi
2014-03-05 09:37shankarbReview Assigned To => AugustoMauch
2014-03-05 09:37shankarbfix_in_branchpi =>
2014-03-05 09:37shankarbIssue Monitored: AugustoMauch
2014-03-05 09:44hgbotCheckin
2014-03-05 09:44hgbotNote Added: 0064810
2014-03-05 09:44hgbotStatusscheduled => resolved
2014-03-05 09:44hgbotResolutionopen => fixed
2014-03-05 09:44hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/381047f42270513a9fbfae0c09866e77f835061b [^]
2014-03-05 12:16hgbotCheckin
2014-03-05 12:16hgbotNote Added: 0064818
2014-03-06 04:24hudsonbotCheckin
2014-03-06 04:24hudsonbotNote Added: 0064875
2014-03-06 04:24hudsonbotCheckin
2014-03-06 04:24hudsonbotNote Added: 0064877
2014-03-06 11:30AugustoMauchNote Added: 0064902
2014-03-06 11:30AugustoMauchStatusresolved => closed
2014-03-06 11:30AugustoMauchFixed in Version => PR14Q2
2014-09-26 15:50AugustoMauchRelationship addedcauses 0027053
2014-09-26 16:25AugustoMauchRelationship addedrelated to 0027698
2015-01-23 11:02AugustoMauchRelationship addedrelated to 0028427
2015-01-23 14:21AugustoMauchRelationship addedcauses 0028747
2016-01-25 17:09caristuRelationship addedrelated to 0031901

Notes
(0064807)
shankarb   
2014-03-05 06:57   
Fix Pushed to try.
(0064810)
hgbot   
2014-03-05 09:44   
Repository: erp/devel/pi
Changeset: 381047f42270513a9fbfae0c09866e77f835061b
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Wed Mar 05 11:28:14 2014 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/381047f42270513a9fbfae0c09866e77f835061b [^]

Fixes Issue 0025517: Wrong calculations on numeric field depending on the formula

Formatting of the number is done after evaluation of the math expression.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js
---
(0064818)
hgbot   
2014-03-05 12:16   
Repository: erp/devel/pi
Changeset: 4cbd0d375833e6eb8181cb20cfa51fc917b6a80b
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Wed Mar 05 16:43:33 2014 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/4cbd0d375833e6eb8181cb20cfa51fc917b6a80b [^]

Related to Issue 25517 : Code refactor

using local variable value instead of using getValue.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js
---
(0064875)
hudsonbot   
2014-03-06 04: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/94a044af1a46 [^]
Maturity status: Test
(0064877)
hudsonbot   
2014-03-06 04: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/94a044af1a46 [^]
Maturity status: Test
(0064902)
AugustoMauch   
2014-03-06 11:30   
Code reviewed and verified in pi@9484419af022