Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0038976Retail ModulesWeb POSpublic2018-07-17 11:422018-09-14 11:58
Reporterasiermartirena 
Assigned Toasiermartirena 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR18Q4 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0038976: When doing a reverse payment, it shouldn't be possible to pay less than the initial amount
DescriptionActually, when doing a reverse payment of a layaway, it is possible to reverse the payment and synchronize the layaway with a total payment of zero.

It is also possible to reverse a payment and instead of paying the reversed amount, set it as credit.

When doing a reverse payment, it must be mandatory to pay again at least the initially paid amount.
Steps To ReproduceCreate a layaway, pay it partially and synchronize.
Load the layaway and reverse the payment.
The layaway can be synchronized without paying again the reversed quantity amount.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0040810 closed ranjith_qualiantech_com Cannot Sell On Credit after loading a paid ticket and reversing the payment 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-07-17 11:42asiermartirenaNew Issue
2018-07-17 11:42asiermartirenaAssigned To => asiermartirena
2018-07-17 11:42asiermartirenaTriggers an Emergency Pack => No
2018-08-23 10:22hgbotCheckin
2018-08-23 10:22hgbotNote Added: 0106458
2018-08-23 10:22hgbotStatusnew => resolved
2018-08-23 10:22hgbotResolutionopen => fixed
2018-08-23 10:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a969e720b89e65efdb7b08bb20c19c5d0d240c78 [^]
2018-08-23 10:23hgbotCheckin
2018-08-23 10:23hgbotNote Added: 0106459
2018-08-31 10:22hgbotCheckin
2018-08-31 10:22hgbotNote Added: 0106562
2018-09-14 11:58marvintmReview Assigned To => marvintm
2018-09-14 11:58marvintmStatusresolved => closed
2018-09-14 11:58marvintmFixed in Version => RR18Q4
2019-05-08 17:11ngarciaRelationship addedrelated to 0040810

Notes
(0106458)
hgbot   
2018-08-23 10:22   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: a969e720b89e65efdb7b08bb20c19c5d0d240c78
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Wed Aug 22 09:48:58 2018 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a969e720b89e65efdb7b08bb20c19c5d0d240c78 [^]

Fixed issue 38976: When doing a reverse payment, it shouldn't be possible to pay less than the initial amount

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/retail/posterminal/PaidReceiptProperties.java
M web/org.openbravo.retail.posterminal/js/components/modalmultiorders.js
M web/org.openbravo.retail.posterminal/js/model/order.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---
(0106459)
hgbot   
2018-08-23 10:23   
Repository: tools/automation/pi-mobile
Changeset: d100a43900c395a3f1d9f9f4cc5ccdd54055756e
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Wed Aug 22 11:14:00 2018 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/d100a43900c395a3f1d9f9f4cc5ccdd54055756e [^]

Related to issue 38976: Changes in the automated tests

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cancelandreplace/I36308_CancelAndRplcLayawayIII.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I31324_POSCrashesAfterReload.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/reversepayments/NoCashWhenReverse.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/reversepayments/ReversePaymentPaidOnCreditWithCredit.java
R src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I35963_VerifyLayawayDoneOnFullPayment.java
---
(0106562)
hgbot   
2018-08-31 10:22   
Repository: tools/automation/pi-mobile
Changeset: 07761674f5bd871206e563d9b55a42342d65f2e3
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Fri Aug 31 10:22:15 2018 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/07761674f5bd871206e563d9b55a42342d65f2e3 [^]

Related to issue 38976: Fixed automated test

Now is not possible to pay on credit the reversed amount.

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/reversepayments/ReversePaymentWithCredit.java
---