Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013320Openbravo ERP03. Procurement managementpublic2010-05-13 23:032010-06-29 12:53
Reporternetworkb 
Assigned Toadrianromero 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.50MP14 
Target Version2.50MP18Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0013320: Different behaviour when changing decimal separator
DescriptionIf you change the format.xml and set ',' as decimal separator and '.' as grouping, when modify line net amount of a purchase invoice line, a callout sets unit price to 0. This doesn't happen if you use the standar format.xml.
Steps To ReproduceModify Format.xml: change all decimal sep to ',', all grouping to '.'

Run Openbravo ERP2.50MP14,
New Purchase invoice, go to Lines
Select a product
Click "Edit Line Net Amount" and change "Line Net Amount" (e.g. to "2,00") and press
return => "Net unit price" and "line net amount" changes to "0,00"
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00133252.50MP18 closed harikrishnan Openbravo ERP The application does not work properly depending Format's and PostgreSQL's configuration 
related to defect 00139692.50 closed dalsasua Modules Different behaviour when changing decimal separator 
related to defect 0014531 closed mohankumar Openbravo ERP Using Comma as decimal separator causes malfunctions on Sales Invoice Lines 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-05-13 23:03networkbNew Issue
2010-05-13 23:03networkbAssigned To => adrianromero
2010-05-17 13:11adrianromeroStatusnew => scheduled
2010-05-17 13:11adrianromerofix_in_branch => pi
2010-05-17 14:26hgbotCheckin
2010-05-17 14:26hgbotNote Added: 0027367
2010-05-17 14:26hgbotStatusscheduled => resolved
2010-05-17 14:26hgbotResolutionopen => fixed
2010-05-17 14:26hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/5f15b78035a11b94edaa03bb6a3fba987e47eb97 [^]
2010-05-17 14:29adrianromeroNote Added: 0027368
2010-05-18 12:12gorka_gilTarget Version2.50MP16 => 2.50MP18
2010-05-18 12:44arunkumarNote Added: 0027413
2010-05-18 12:44arunkumarStatusresolved => closed
2010-05-19 00:01anonymoussf_bug_id0 => 3003689
2010-05-20 11:15hudsonbotCheckin
2010-05-20 11:15hudsonbotNote Added: 0027583
2010-05-26 12:39vmromanosRelationship addedrelated to 0013325
2010-06-29 12:53rafarodaIssue Monitored: rafaroda
2010-07-15 10:15rafarodaRelationship addedrelated to 0013969
2010-09-14 11:56adrianromeroRelationship addedrelated to 0014531

Notes
(0027367)
hgbot   
2010-05-17 14:26   
Repository: erp/devel/pi
Changeset: 5f15b78035a11b94edaa03bb6a3fba987e47eb97
Author: Adrián Romero <adrianromero <at> openbravo.com>
Date: Mon May 17 14:26:04 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/5f15b78035a11b94edaa03bb6a3fba987e47eb97 [^]

Fixes issue 0013320: Different behaviour when changing decimal separator
This changeset replaces in the callout how to read the Line Net Amount value. This value is a numeric value and must be read using the function getNumericParameter because this function takes into acount the number format defined in the file Format.xml

---
M src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Amt.java
---
(0027368)
adrianromero   
2010-05-17 14:29   
* Steps to test
Just follow the steps to reproduce and notice that now the "Net unit price" and "line net amount" do not change to "0,00" and take the correct value
* Other areas affected by the changeset
All the windows that use this callout are affected. It has been tested and works
(0027413)
arunkumar   
2010-05-18 12:44   
Tested working fine
(0027583)
hudsonbot   
2010-05-20 11:15   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/5f15b78035a1 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/a7901e097e1a [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17407.obx [^]