Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0049472 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Web POS | major | always | 2022-06-02 23:07 | 2022-06-22 17:44 | |||
| Reporter | axelmercado | View Status | public | |||||
| Assigned To | rqueralta | |||||||
| Priority | high | Resolution | no change required | Fixed in Version | ||||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | < 1 month | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | RR21Q4.2 | SCM revision | ||||||
| Review Assigned To | marvintm | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0049472: Error with OrderLoader Web Services | |||||||
| Description | When sending a json for a return with the isReturn set to "true", the return order is created with the gross line amount in positive. | |||||||
| Steps To Reproduce | 1-Create a JSON http://wiki.openbravo.com/wiki/Retail:Order_Loading_WebService#URL.2FRequest_.26_Authentication [^] 2-Send the JSON to the API with the following endpoint: http://localhost:8080/openbravo/org.openbravo.mobile.core.service.jsonrest/org.openbravo.retail.posterminal.ExternalOrderLoader?synchronizedProcessing=true [^] 3-Check that the return has been created correctly in the customer return window and the values of the "Gross Line Amount" and "Line Net Amount" lines. We performed the 2 possibilities indicated in the Wiki to enter the return but both fail in the same way. Quote:A return can be send in, in 2 ways: set a property "isReturn" on the order header to true. The complete ticket is considered to be a return. The quantities in the orderlines are positive, but still to be considered to be returns. send in a standard ticket but with negative quantity for the return. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0138240) marvintm (manager) 2022-06-10 09:11 |
The OrderLoader (and by extension the ExternalOrderLoader), are designed to receive the information of the Order in the JSON exactly in the way it should be created. This means that returns should be sent with the negative qty and total amounts. The wiki statement mentioned in the issue was confusing, and we have clarified it to better explain this point: http://wiki.openbravo.com/wiki/Retail:Order_Loading_WebService#Return_Order.2FLines [^] |
|
(0138242) ivancaceres (reporter) 2022-06-10 09:21 |
Hi Marvin, Can you check TicketPropertyMapping.java method getGrossUnitAmount? In this method, i think gross should be taken instead of lineGrossAmount. Sl2 |
|
(0138264) axelmercado (developer) 2022-06-10 14:56 |
Is it correct that "Gross Line Amount" and "Line Net Amount" both end in positive? |
|
(0138642) marvintm (manager) 2022-06-22 17:44 |
No, this part is not correct. A separate issue was reported this week about that: https://issues.openbravo.com/view.php?id=49628 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-06-02 23:07 | axelmercado | New Issue | |
| 2022-06-02 23:07 | axelmercado | Assigned To | => Retail |
| 2022-06-02 23:07 | axelmercado | Triggers an Emergency Pack | => No |
| 2022-06-02 23:08 | ivazquez | ETA | none => < 1 month |
| 2022-06-03 07:21 | ivancaceres | Issue Monitored: ivancaceres | |
| 2022-06-07 15:02 | rqueralta | Assigned To | Retail => rqueralta |
| 2022-06-07 15:03 | rqueralta | Status | new => scheduled |
| 2022-06-10 09:11 | marvintm | Review Assigned To | => marvintm |
| 2022-06-10 09:11 | marvintm | Note Added: 0138240 | |
| 2022-06-10 09:11 | marvintm | Status | scheduled => closed |
| 2022-06-10 09:11 | marvintm | Resolution | open => no change required |
| 2022-06-10 09:21 | ivancaceres | Note Added: 0138242 | |
| 2022-06-10 14:56 | axelmercado | Note Added: 0138264 | |
| 2022-06-22 17:44 | marvintm | Note Added: 0138642 | |
| 2022-06-22 17:44 | marvintm | Relationship added | related to 0049628 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |