Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0028720 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | critical | have not tried | 2015-01-22 09:19 | 2015-02-04 20:34 | |||
Reporter | jonalegriaesarte | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | 3.0PR15Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 3e5c7b3e8a8f | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | AugustoMauch | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | 2015-01-09 | |||||||
Regression introduced in release | 3.0PR14Q3.5 | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/rev/e5465c5925cf54475d090258a3832ec8cc9d14ca [^] | |||||||
Triggers an Emergency Pack | Yes | |||||||
Summary | 0028720: System is not saving big numbers properly | |||||||
Description | System is not saving big numbers properly | |||||||
Steps To Reproduce | - GL Journal window - Create a new header - Create a new line - Add a credit: 10200500.45 - Save it. System stores 1020050045.00 | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||||||||||||||||||||||||||||||
|
Notes | |
(0073712) alostale (manager) 2015-01-22 10:42 |
It is reproducible in 3.0PR14Q3.5 but not in 3.0PR14Q4 |
(0073713) alostale (manager) 2015-01-22 10:44 |
The problem is caused by fix for issue 0026132 which does not handle properly scientific notation for numbers which are not integer, this problem was hidden until fix for 0028552. It is only reproducible in case a callout (re)sets numeric values with this case. |
(0073714) hgbot (developer) 2015-01-22 11:16 |
Repository: erp/devel/pi Changeset: 3e5c7b3e8a8f29ba98c312b092eadef36f03aa81 Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Thu Jan 22 11:14:24 2015 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/3e5c7b3e8a8f29ba98c312b092eadef36f03aa81 [^] fixed bug 28720: callouts incorrectly set big non integer numbers When a big non integer number was set by a callout, decimal separator was removed resulting in a different number, ie. 10200500.45 resulted in 1020050045.00. The problem was in the OB.Utilities.Number.ScientificToDecimal JavaScript function which wrongly assumed scientific exponent always added zeroes to coefficient, which is not true in this case where exponent determines where the decimal separator is inserted in the coefficient. --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/utilities/ob-utilities-number.js --- |
(0073715) alostale (manager) 2015-01-22 11:18 |
added test in testLink, pending to decide on automation: https://testlink.openbravo.com/testlink/linkto.php?tprojectPrefix=Communit&item=testcase&id=Communit-8029 [^] |
(0073721) AugustoMauch (administrator) 2015-01-22 13:09 |
Code reviewed and verified in pi@3e5c7b3e8a8f |
(0073722) AugustoMauch (administrator) 2015-01-22 13:11 |
Check that [1] is no longer reproducible and that [2] and [3] remain fixed. Tested using '.' and ',' as decimal separators. [1] https://issues.openbravo.com/view.php?id=28720 [^] [2] https://issues.openbravo.com/view.php?id=28552 [^] [3] https://issues.openbravo.com/view.php?id=26132 [^] |
(0074172) hudsonbot (developer) 2015-02-04 20:34 |
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/f36c91d0ad63 [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2015-01-22 09:19 | jonalegriaesarte | New Issue | |
2015-01-22 09:19 | jonalegriaesarte | Assigned To | => AugustoMauch |
2015-01-22 09:19 | jonalegriaesarte | Modules | => Core |
2015-01-22 09:19 | jonalegriaesarte | Resolution time | => 1421881200 |
2015-01-22 09:19 | jonalegriaesarte | Triggers an Emergency Pack | => No |
2015-01-22 09:25 | alostale | Assigned To | AugustoMauch => alostale |
2015-01-22 09:27 | alostale | Relationship added | related to 0028552 |
2015-01-22 10:39 | alostale | Relationship replaced | caused by 0028552 |
2015-01-22 10:39 | alostale | Relationship added | related to 0026132 |
2015-01-22 10:41 | alostale | Regression level | => Production - Confirmed Stable |
2015-01-22 10:41 | alostale | Regression date | => 2015-01-09 |
2015-01-22 10:41 | alostale | Regression introduced in release | => 3.0PR14Q3.5 |
2015-01-22 10:41 | alostale | Regression introduced by commit | => https://code.openbravo.com/erp/devel/pi/rev/e5465c5925cf54475d090258a3832ec8cc9d14ca [^] |
2015-01-22 10:42 | alostale | Note Added: 0073712 | |
2015-01-22 10:44 | alostale | Note Added: 0073713 | |
2015-01-22 11:16 | hgbot | Checkin | |
2015-01-22 11:16 | hgbot | Note Added: 0073714 | |
2015-01-22 11:16 | hgbot | Status | new => resolved |
2015-01-22 11:16 | hgbot | Resolution | open => fixed |
2015-01-22 11:16 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/3e5c7b3e8a8f29ba98c312b092eadef36f03aa81 [^] |
2015-01-22 11:18 | alostale | Note Added: 0073715 | |
2015-01-22 11:18 | alostale | Review Assigned To | => AugustoMauch |
2015-01-22 11:21 | alostale | Triggers an Emergency Pack | No => Yes |
2015-01-22 13:09 | AugustoMauch | Note Added: 0073721 | |
2015-01-22 13:09 | AugustoMauch | Status | resolved => closed |
2015-01-22 13:09 | AugustoMauch | Fixed in Version | => 3.0PR15Q2 |
2015-01-22 13:11 | AugustoMauch | Note Added: 0073722 | |
2015-01-26 08:36 | alostale | Relationship added | related to 0028427 |
2015-02-04 20:34 | hudsonbot | Checkin | |
2015-02-04 20:34 | hudsonbot | Note Added: 0074172 | |
2015-05-26 09:30 | NaroaIriarte | Relationship added | related to 0029709 |
2015-05-26 13:20 | NaroaIriarte | Relationship added | related to 0030014 |
Copyright © 2000 - 2009 MantisBT Group |