Openbravo Issue Tracking System - Retail Modules |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0030425 | Retail Modules | Multi order payment | public | 2015-07-25 13:02 | 2015-09-18 13:21 |
|
| Reporter | lorenzofidalgo | |
| Assigned To | jorge-garcia | |
| Priority | immediate | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | main | |
| Target Version | | Fixed in Version | RR15Q4 | |
| Merge Request Status | |
| Review Assigned To | guilleaer |
| OBNetwork customer | |
| Support ticket | |
| Regression level | Packaging and release |
| Regression date | |
| Regression introduced in release | main |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0030425: Partial payments in a layaway through "Pay Open Tickets" option does not save Received & Outstanding fields correctly |
| Description | To pay a layaway partially through "Pay Open Tickets" option records wrongly data in backend.
In RR15Q2.2 this is working correcly. |
| Steps To Reproduce | 1-Login POS as vallblanca//openbravo.
2-Add 1x "Avalanche transceiver".
3-In dropdown menu, click "Layaway this receipt", click Total Amount button and click in "Layaway" button to layaway that receipt.
4-Click "Pay Open Tickets" option in dropdown menu, select the just created layaway and click "Done".
5-Click in the line of the product and when the "Amount to layaway" popup appears, enter 20 and apply. Then click Ok (tick) button and click Done button.
6-Go to POS backend and login with Openbravo//openbravo.
7-Using "The White Vally Group Admin" role go to "Sales Order" window and select the just created record.
8-Defect: In "Payment Plan" tab, check "Received" & "Outstanding" fields are wrong. |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | | depends on | backport | 0030536 | RR15Q3 | closed | Retail | [RR15Q3] Partial payments in a layaway through "Pay Open Tickets" option does not save Received & Outstanding fields correctly | | depends on | backport | 0030696 | RR15Q3.2 | closed | jorge-garcia | [RR15Q3] Partial payments in a layaway through "Pay Open Tickets" option does not save Received & Outstanding fields correctly |
|
| Attached Files |  WrongReceived+OutstandingAmounts.png (34,575) 2015-07-25 13:02
https://issues.openbravo.com/file_download.php?file_id=8305&type=bug
|
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2015-07-25 13:02 | lorenzofidalgo | New Issue | |
| 2015-07-25 13:02 | lorenzofidalgo | Assigned To | => Retail |
| 2015-07-25 13:02 | lorenzofidalgo | File Added: WrongReceived+OutstandingAmounts.png | |
| 2015-07-25 13:02 | lorenzofidalgo | Regression level | => Packaging and release |
| 2015-07-25 13:02 | lorenzofidalgo | Regression introduced in release | => RR15Q2.2 |
| 2015-07-25 13:02 | lorenzofidalgo | Triggers an Emergency Pack | => No |
| 2015-07-28 10:30 | lorenzofidalgo | Regression introduced in release | RR15Q2.2 => main |
| 2015-08-10 13:45 | jorge-garcia | Status | new => scheduled |
| 2015-08-10 14:00 | hgbot | Checkin | |
| 2015-08-10 14:00 | hgbot | Note Added: 0079281 | |
| 2015-08-11 08:45 | hgbot | Checkin | |
| 2015-08-11 08:45 | hgbot | Note Added: 0079293 | |
| 2015-08-14 17:25 | hgbot | Checkin | |
| 2015-08-14 17:25 | hgbot | Note Added: 0079365 | |
| 2015-08-14 17:25 | hgbot | Status | scheduled => resolved |
| 2015-08-14 17:25 | hgbot | Resolution | open => fixed |
| 2015-08-14 17:25 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f4354d4ba3340b58d1534c8859e9faf317a370a4 [^] |
| 2015-08-28 12:28 | marvintm | Assigned To | Retail => jorge-garcia |
| 2015-08-28 14:11 | marvintm | Note Added: 0079796 | |
| 2015-08-28 14:11 | marvintm | Status | resolved => new |
| 2015-08-28 14:11 | marvintm | Resolution | fixed => open |
| 2015-08-28 14:11 | marvintm | Status | new => scheduled |
| 2015-09-04 13:29 | hgbot | Checkin | |
| 2015-09-04 13:29 | hgbot | Note Added: 0080117 | |
| 2015-09-04 13:29 | hgbot | Checkin | |
| 2015-09-04 13:29 | hgbot | Note Added: 0080118 | |
| 2015-09-04 13:29 | hgbot | Status | scheduled => resolved |
| 2015-09-04 13:29 | hgbot | Resolution | open => fixed |
| 2015-09-04 13:29 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/f4354d4ba3340b58d1534c8859e9faf317a370a4 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a5224d35df572aeb927a33ebe99bac035d3d5c61 [^] |
| 2015-09-05 12:57 | hgbot | Checkin | |
| 2015-09-05 12:57 | hgbot | Note Added: 0080149 | |
| 2015-09-05 12:58 | Orekaria | Note Added: 0080150 | |
| 2015-09-05 12:58 | Orekaria | Status | resolved => new |
| 2015-09-05 12:58 | Orekaria | Resolution | fixed => open |
| 2015-09-05 19:54 | hgbot | Checkin | |
| 2015-09-05 19:54 | hgbot | Note Added: 0080152 | |
| 2015-09-07 11:26 | jorge-garcia | Summary | [RR15Q3] Partial payments in a layaway through "Pay Open Tickets" option does not save Received & Outstanding fields correctly => Partial payments in a layaway through "Pay Open Tickets" option does not save Received & Outstanding fields correctly |
| 2015-09-08 20:33 | hgbot | Checkin | |
| 2015-09-08 20:33 | hgbot | Note Added: 0080250 | |
| 2015-09-08 20:33 | hgbot | Status | new => resolved |
| 2015-09-08 20:33 | hgbot | Resolution | open => fixed |
| 2015-09-08 20:33 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a5224d35df572aeb927a33ebe99bac035d3d5c61 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c2f4e4836b52fe4d033e69c67e1eabd4508c4a79 [^] |
| 2015-09-18 12:23 | guilleaer | Review Assigned To | => guilleaer |
| 2015-09-18 13:21 | guilleaer | Note Added: 0080493 | |
| 2015-09-18 13:21 | guilleaer | Status | resolved => closed |
| 2015-09-18 13:21 | guilleaer | Fixed in Version | => RR15Q4 |
|
Notes |
|
|
(0079281)
|
|
hgbot
|
|
2015-08-10 14:00
|
|
Repository: tools/automation/pi-mobile
Changeset: 8c0e3ae408596ef3da3633ec762664a0e6509c63
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Aug 07 12:43:50 2015 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/8c0e3ae408596ef3da3633ec762664a0e6509c63 [^]
Verifies issue 30425: Added Test I30425_PartialPaymentLayaway
---
M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I30425_PartialPaymentLayaway.java
---
|
|
|
|
(0079293)
|
|
hgbot
|
|
2015-08-11 08:45
|
|
|
|
|
(0079365)
|
|
hgbot
|
|
2015-08-14 17:25
|
|
|
|
|
|
|
The fix for this issue is not correct. Instead of modifying the main execution path, it must be checked why the layaway specific flags are not set correctly. |
|
|
|
(0080117)
|
|
hgbot
|
|
2015-09-04 13:29
|
|
|
|
|
(0080118)
|
|
hgbot
|
|
2015-09-04 13:29
|
|
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: a5224d35df572aeb927a33ebe99bac035d3d5c61
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Sep 03 10:44:42 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a5224d35df572aeb927a33ebe99bac035d3d5c61 [^]
Fixes issue 30425: Partial payments in layaway through "Pay Open Tickets" option
The problem was that the partial payment of a layaway always creates a new
order, not taken into account that there is already a order created with the
same id and attributes.
The solution is to modify the newLayaway parameter to represent what it means in
the flow.
---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
|
|
|
|
(0080149)
|
|
hgbot
|
|
2015-09-05 12:57
|
|
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: bdb5e63db53c648db73a20ddc8ebbf5203fd44e9
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Sat Sep 05 12:06:47 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/bdb5e63db53c648db73a20ddc8ebbf5203fd44e9 [^]
Backed out issue 30425, changeset: a5224d35df57. Integration failing.
affected processes can be reproduced with these tests:
- I25623_VerifiedReturnsAvoidMoreThanOrdered
- I29152_NoCashAvailableWhenVoidingALayaway
- I30075_NoPositiveLinesInReturnTickets
- Layaway_Cre1Par1Fin2
- LayawayVerifyPaymentsInformation
---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
|
|
|
|
|
|
See the note of the back out |
|
|
|
(0080152)
|
|
hgbot
|
|
2015-09-05 19:54
|
|
|
|
|
(0080250)
|
|
hgbot
|
|
2015-09-08 20:33
|
|
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c2f4e4836b52fe4d033e69c67e1eabd4508c4a79
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Mon Sep 07 11:23:39 2015 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c2f4e4836b52fe4d033e69c67e1eabd4508c4a79 [^]
Fixes issue 30425: Partial payments in layaway through "Pay Open Tickets" option
The problem was that the partial payment of a layaway always creates a new
order, not taken into account that there is already a order created with the
same id and attributes.
The solution is to create a new check to see if the layaway is partially paid.
---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
|
|
|
|
|
|
Tested and code review done |
|