Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0036590 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] A. Platform | major | always | 2017-08-03 13:28 | 2017-09-21 16:49 | |||
Reporter | JONHM | View Status | public | |||||
Assigned To | markmm82 | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR17Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | pi | SCM revision | ||||||
Review Assigned To | aferraz | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0036590: Callouts retrieve wrong quantity when Format.xml file is set with too many decimals | |||||||
Description | When adding a product line in physical inventory (which is associated with SL_Inventory_Product callout), the quantity obtained from the product is not real, there are too many zeroes added to the real quantity. It seems to be related to the configuration of the Format.xml file and its 'qtyRelation' and 'qtyEdition' fields when set with 5 decimals | |||||||
Steps To Reproduce | - First of all, replace the Format.xml file by the attached one. Then, recompile the application. 1) Open "Physical Inventory" window and create a new one with default values. 2) Switch to lines tab and create a new one for product 'Vino Blanco 0,75L'. --> Notice that the stock for that product is 31300 and the quantity retrieved from the callout is 3130000 (The quantity in database for that product is set as 31300.00) | |||||||
Tags | No tags attached. | |||||||
Attached Files | Format.xml [^] (3,271 bytes) 2017-08-03 13:28 callout.mp4 [^] (1,375,339 bytes) 2017-08-03 13:28 36590.diff [^] (5,090 bytes) 2017-08-10 12:45 [Show Content] backport-17Q3.diff [^] (2,783 bytes) 2017-08-18 12:55 [Show Content] | |||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||||||||||||||||||||||||||||||||||||||||||||
|
Notes | |
(0098453) inigosanchez (developer) 2017-08-09 14:13 |
As mentioned in related issue 0036392, it is needed to replace the castings to Object in the mentioned callout by sending the corresponding numeric value when it applies. |
(0098475) aferraz (manager) 2017-08-10 12:45 edited on: 2017-08-11 14:25 |
Issue is reproducible in 17Q1.1, 17Q2.1 and 17Q3 but not in PI. Applied attached fix in 17Q1.1 (send quantities as BigDecimal instead of cast quantity String to Object) and it still fails. |
(0098542) inigosanchez (developer) 2017-08-18 12:54 |
The usage of "EXECUTE" callout command is no longer supported (since the refactor of the SimpleCallouts 3.0PR16Q4) and it is not needed because the display logics are recalculated on the FIC response. The refactor 0036067 of several SimpleCallouts fixes the problem in 3.0PR17Q4 but it isn't backported to 17Q1.4, 17Q2.2 and 17Q3. The attached diff fixes the problem in 17Q3 and should be reviewed it (backport-17Q3.diff). As you can see in the diff, other callouts should be reviewed apart of the reported one (Simple callouts with execute command): -SL_Inventory_Locator -SL_Inventory_Product <- Reported -SL_Movement_Product -SL_InOutLine_Product |
(0098548) hgbot (developer) 2017-08-18 14:54 |
Repository: erp/devel/pi Changeset: b29fcb712528e7603e952d036bf826c4031a9a3a Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com> Date: Fri Aug 18 14:53:29 2017 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/b29fcb712528e7603e952d036bf826c4031a9a3a [^] Related to issue 36590: Send quantities as BigDecimal --- M src/org/openbravo/erpCommon/ad_callouts/SL_InOutLine_Product.java M src/org/openbravo/erpCommon/ad_callouts/SL_Inventory_Product.java M src/org/openbravo/erpCommon/ad_callouts/SL_Movement_Product.java --- |
(0098549) aferraz (manager) 2017-08-18 15:03 |
Reverted 0036067 fix for SL_Inventory_Product.java in PI and issue is still not reproducible. Applied 0036067 fix for SL_Inventory_Product.java in 17Q3 and issue is still reproducible. |
(0098581) aferraz (manager) 2017-08-22 14:43 |
Fixed in issue 0036345. |
(0099315) hudsonbot (developer) 2017-09-21 16:49 |
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/9750b78d3e5c [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2017-08-03 13:28 | JONHM | New Issue | |
2017-08-03 13:28 | JONHM | Assigned To | => platform |
2017-08-03 13:28 | JONHM | File Added: Format.xml | |
2017-08-03 13:28 | JONHM | Modules | => Core |
2017-08-03 13:28 | JONHM | Resolution time | => 1503525600 |
2017-08-03 13:28 | JONHM | Triggers an Emergency Pack | => No |
2017-08-03 13:28 | JONHM | File Added: callout.mp4 | |
2017-08-09 14:09 | inigosanchez | Relationship added | related to 0036392 |
2017-08-09 14:13 | inigosanchez | Note Added: 0098453 | |
2017-08-09 14:37 | inigosanchez | Assigned To | platform => Triage Finance |
2017-08-09 14:57 | aferraz | Assigned To | Triage Finance => AtulOpenbravo |
2017-08-09 17:59 | AtulOpenbravo | Status | new => scheduled |
2017-08-10 12:45 | aferraz | Note Added: 0098475 | |
2017-08-10 12:45 | aferraz | File Added: 36590.diff | |
2017-08-10 12:46 | aferraz | Assigned To | AtulOpenbravo => platform |
2017-08-10 12:47 | aferraz | Note Edited: 0098475 | View Revisions |
2017-08-11 14:25 | inigosanchez | Note Edited: 0098475 | View Revisions |
2017-08-18 12:54 | inigosanchez | Note Added: 0098542 | |
2017-08-18 12:55 | inigosanchez | File Added: backport-17Q3.diff | |
2017-08-18 12:57 | inigosanchez | Relationship added | related to 0036302 |
2017-08-18 12:57 | inigosanchez | Relationship added | related to 0036067 |
2017-08-18 12:58 | inigosanchez | Assigned To | platform => Triage Finance |
2017-08-18 12:59 | aferraz | Assigned To | Triage Finance => inigosanchez |
2017-08-18 12:59 | aferraz | Status | scheduled => feedback |
2017-08-18 13:00 | aferraz | Status | feedback => scheduled |
2017-08-18 14:54 | hgbot | Checkin | |
2017-08-18 14:54 | hgbot | Note Added: 0098548 | |
2017-08-18 15:03 | aferraz | Note Added: 0098549 | |
2017-08-22 14:43 | aferraz | Review Assigned To | => aferraz |
2017-08-22 14:43 | aferraz | Note Added: 0098581 | |
2017-08-22 14:43 | aferraz | Status | scheduled => closed |
2017-08-22 14:43 | aferraz | Resolution | open => fixed |
2017-08-22 14:43 | aferraz | Fixed in Version | => 3.0PR17Q4 |
2017-08-22 14:48 | aferraz | Relationship added | duplicate of 0036345 |
2017-08-23 08:38 | caristu | Assigned To | inigosanchez => Triage Finance |
2017-08-23 10:53 | aferraz | Assigned To | Triage Finance => aferraz |
2017-08-23 10:53 | aferraz | Assigned To | aferraz => markmm82 |
2017-09-21 16:49 | hudsonbot | Checkin | |
2017-09-21 16:49 | hudsonbot | Note Added: 0099315 |
Copyright © 2000 - 2009 MantisBT Group |