Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0038333 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | always | 2018-04-11 15:43 | 2018-05-11 12:45 | |||
Reporter | rafaroda | View Status | public | |||||
Assigned To | ranjith_qualiantech_com | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | RR18Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 8c76bcb53704 | ||||
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 | marvintm | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0038333: Wrong Payment Details created in backend when paying Layaway with Cash first then overpaying with Digital Coupon | |||||||
Description | Wrong Payment Details created in backend when paying Layaway with Cash first then overpaying with Digital Coupon | |||||||
Steps To Reproduce | Reproduce in LIVE BUILDS https://livebuilds.openbravo.com/retail_modules_pgsql_pi/web/org.openbravo.retail.posterminal/?terminal=YS-11 [^] 1) Open terminal YS-11 which has digital coupons enabled (overpayment allowed) 2) Click on Layaway this Receipt 3) Add Avalanche transceiver product: Total Amount = 150,50€ 4) Click on Total Amount 5) Select payment method: Cash and add 100€ payment 6) Click on Layaway button 7) Reopen the Layaway 8) Click on total Amount: 50,50€ remain to pay 9) Select Digital Coupon payment method 10) Add a digital coupon of 60€ There is a red error message saying, should this message be displayed? See screenshot: Only cash payment methods are allowed to return change. 11) We select cash as Payment Method: 9,50€ are displayed to be returned, wrong? See screenshot: Only cash payment methods are allowed to return change 12) Click on Done 13) Go to backend and search for the Sales Order: * Payment Plan: Expected Amount: 150,50 OK, Received: 160 OK, Outstanding: -9,50 NOK * Payment Details: ** Cash: Expected Amount: 150,50 OK, Received Amount: 100 OK ** Wire transfer: Expected Amount: 150,50 OK, Received Amount: 60 OK See screenshot: Sales order Backend Payments-1 14) Reopen the layaway ticket: still 9,50€ remain to pay: you can keep adding cash bys electing Cash payment method and the OK V icon plus Done 15) Reopen the ticket again: now 10,50€ remain to pay, you can go back to step 14 again and again 16) In backend, in the sales order: * Payment Plan: Expected Amount: 150,50 OK, Received: 161 NOK, Outstanding: -10,50 NOK * Payment Details: ** Cash: Expected Amount: 150,50 OK, Received Amount: 100 OK ** Wire transfer: Expected Amount: 150,50 OK, Received Amount: 60 OK ** Cash: Expected Amount: 150,50 OK, Received Amount: 1 NOK See screenshot: Sales order Backend Payments-2 | |||||||
Tags | SER-QA | |||||||
Attached Files | Only cash payment methods are allowed to return change.png [^] (98,620 bytes) 2018-04-11 15:43
Sales order Backend Payments - 1.png [^] (82,840 bytes) 2018-04-11 15:43 Sales order Backend Payments - 2.png [^] (90,840 bytes) 2018-04-11 15:44 | |||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||
|
Notes | |
(0104267) hgbot (developer) 2018-05-04 08:39 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 4dc25baceae5f616c7d85492bacab988519435f0 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Fri May 04 12:09:34 2018 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4dc25baceae5f616c7d85492bacab988519435f0 [^] Fixed issue 38333 : Change should not be calculated based on Prepayments * For Partial Paid Layaway, Cash change should not given from Cash Prepayments if Card is overpaid. It should be Overpayment --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0104268) hgbot (developer) 2018-05-04 08:40 |
Repository: tools/automation/pi-mobile Changeset: af64455e7880148cb779cf437218a6ee64cc180a Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Fri May 04 12:09:51 2018 +0530 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/af64455e7880148cb779cf437218a6ee64cc180a [^] Verifies issue 38333 : Added automated test 'I38333_VerfiyCashChangeOnLayaway --- M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/storecurrencyexchangerate/VerifyPaymentBasedOnStoreConversionRate.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I38333_VerfiyCashChangeOnLayaway.java --- |
(0104284) marvintm (manager) 2018-05-04 14:39 |
Solution doesn't seem to be correct. I can reproduce a problem doing the following steps: - Create a layaway for total 150 - Add a payment in cash for 50 - Sync with backend. - Open layaway again. - Add payment in card for 200 Expected result should be: payment 50 in cash+100 in card, then overpayment of 100 in card. However, the WebPOS tries to generate change for the cash payment method, and won't allow to complete ticket. |
(0104325) hgbot (developer) 2018-05-08 14:17 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: 8c76bcb53704107730534ae07f77fc049a9a1f42 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Tue May 08 17:47:13 2018 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8c76bcb53704107730534ae07f77fc049a9a1f42 [^] Fixed issue 38333 : Change should not be calculated based on Prepayments * For Partial Paid Layaway, Cash change should not given from Cash Prepayments if Card is overpaid. Card should be Overpayment --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0104326) hgbot (developer) 2018-05-08 14:17 |
Repository: tools/automation/pi-mobile Changeset: a0296ab3e05fa6a7d9c0e38708c9942ad8d9da32 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Tue May 08 17:47:28 2018 +0530 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/a0296ab3e05fa6a7d9c0e38708c9942ad8d9da32 [^] Verifies issue 38333 : Added automated test 'I38333_VerfiyCashChangeOnLayawayII' --- A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I38333_VerfiyCashChangeOnLayawayII.java --- |
Issue History | |||
Date Modified | Username | Field | Change |
2018-04-11 15:43 | rafaroda | New Issue | |
2018-04-11 15:43 | rafaroda | Assigned To | => Retail |
2018-04-11 15:43 | rafaroda | File Added: Only cash payment methods are allowed to return change.png | |
2018-04-11 15:43 | rafaroda | Triggers an Emergency Pack | => No |
2018-04-11 15:43 | rafaroda | File Added: Sales order Backend Payments - 1.png | |
2018-04-11 15:44 | rafaroda | File Added: Sales order Backend Payments - 2.png | |
2018-04-12 10:48 | avicente | Resolution time | => 1525039200 |
2018-04-12 10:48 | avicente | Tag Attached: SER-QA | |
2018-04-12 15:13 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
2018-04-12 15:14 | ranjith_qualiantech_com | Status | new => scheduled |
2018-04-24 08:55 | ranjith_qualiantech_com | Relationship added | related to 0038038 |
2018-05-04 08:39 | hgbot | Checkin | |
2018-05-04 08:39 | hgbot | Note Added: 0104267 | |
2018-05-04 08:39 | hgbot | Status | scheduled => resolved |
2018-05-04 08:39 | hgbot | Resolution | open => fixed |
2018-05-04 08:39 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4dc25baceae5f616c7d85492bacab988519435f0 [^] |
2018-05-04 08:40 | hgbot | Checkin | |
2018-05-04 08:40 | hgbot | Note Added: 0104268 | |
2018-05-04 14:39 | marvintm | Note Added: 0104284 | |
2018-05-04 14:39 | marvintm | Status | resolved => new |
2018-05-04 14:39 | marvintm | Resolution | fixed => open |
2018-05-07 09:37 | ranjith_qualiantech_com | Status | new => scheduled |
2018-05-08 14:17 | hgbot | Checkin | |
2018-05-08 14:17 | hgbot | Note Added: 0104325 | |
2018-05-08 14:17 | hgbot | Status | scheduled => resolved |
2018-05-08 14:17 | hgbot | Resolution | open => fixed |
2018-05-08 14:17 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4dc25baceae5f616c7d85492bacab988519435f0 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8c76bcb53704107730534ae07f77fc049a9a1f42 [^] |
2018-05-08 14:17 | hgbot | Checkin | |
2018-05-08 14:17 | hgbot | Note Added: 0104326 | |
2018-05-11 12:45 | marvintm | Review Assigned To | => marvintm |
2018-05-11 12:45 | marvintm | Status | resolved => closed |
2018-05-11 12:45 | marvintm | Fixed in Version | => RR18Q3 |
2018-06-13 15:16 | ranjith_qualiantech_com | Relationship added | related to 0038750 |
Copyright © 2000 - 2009 MantisBT Group |