Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0056967Retail ModulesWeb POSpublic2024-10-29 16:592024-11-16 10:00
Reportermalsasua 
Assigned ToRajesh_18 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR24Q4Fixed in VersionRR24Q4 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0056967: Overpayment is not checked before to add the payment
Descriptionif you configure a payment method with overpayment limit, the payment is added although the overpayment limit is reached
Steps To ReproduceBO:
. Touchpoint Type: Credit Card:
  - Payment provider: Legacy Payment Provider
  - Overpayment / Change Limit: 0

POS
. add Adhesive body warmers: total ticket 3.60
. click in total button -> payment tab opened
. Card: 5 -> enter button -> Payment using card was added with 5 -> Overpayment not available message is displayed, but the payment was added
Proposed Solution. the payment should not be added informing the overpayment is not available
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0056932 closed Rajesh_18 Overpayment is not checked before to add the payment 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2024-11-04 16:05migueldejuanaTypedefect => backport
2024-11-04 16:05migueldejuanaTarget Version => RR24Q4
2024-11-07 14:49migueldejuanaAssigned ToRetail => Rajesh_18
2024-11-13 06:19hgbotNote Added: 0171834
2024-11-13 06:19hgbotNote Added: 0171835
2024-11-16 10:00hgbotResolutionopen => fixed
2024-11-16 10:00hgbotStatusscheduled => closed
2024-11-16 10:00hgbotNote Added: 0172111
2024-11-16 10:00hgbotFixed in Version => RR24Q4
2024-11-16 10:00hgbotNote Added: 0172112
2024-11-16 10:00hgbotNote Added: 0172113
2024-11-16 10:00hgbotNote Added: 0172114

Notes
(0171834)
hgbot   
2024-11-13 06:19   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1774 [^]
(0171835)
hgbot   
2024-11-13 06:19   
Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/438 [^]
(0172111)
hgbot   
2024-11-16 10:00   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1774 [^]
(0172112)
hgbot   
2024-11-16 10:00   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 68311fba3d23b2d0eb3e53d8edae89b54b01c11a
Author: Rajesh Senthilkumar <r.senthilkumar@external.orisha.com>
Date: 16-11-2024 09:00:06
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/68311fba3d23b2d0eb3e53d8edae89b54b01c11a [^]

Fixed ISSUE-56967: Overpayment should be checked before adding the payment
Added a function to check overpayment limit

---
M web/org.openbravo.retail.posterminal/js/components/modalprovidergroup.js
M web/org.openbravo.retail.posterminal/js/components/paymentvalidation.js
---
(0172113)
hgbot   
2024-11-16 10:00   
Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 19edb4f2b7baf16c3be26673ba252a55052cd9f2
Author: Rajesh Senthilkumar <rajesh@qualiantech.com>
Date: 13-11-2024 10:47:23
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/19edb4f2b7baf16c3be26673ba252a55052cd9f2 [^]

Related to ISSUE-56967: Refactored overpayment tests

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/payment/I39726_VerifyNoErrorWithAndWithoutOverpayments.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/payment/I39974_VerifyOverpaymentWith2DifferentPaymentMethods.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I31791_CheckingErrorMsgInOverpayment.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I36977_VerifyOverpaymentEvaluation.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I36994_OverpaymentLimitWithDecimalSeparatorDoesNotWork.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I31842_VerifyOverpaymentByLimit.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I32826_VerifyOverpaymentByCashCard.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I32826_VerifyOverpaymentByCashCardII.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I33324_VerifyOverpaymentByCash.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I36974_VerifyOverpaymentByLimitAfterRefresh.java
---
(0172114)
hgbot   
2024-11-16 10:00   
Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/438 [^]