Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040146Retail ModulesWeb POSpublic2019-02-05 13:302019-03-04 15:28
Reportermarvintm 
Assigned TojavierRodriguez 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR19Q2 
Merge Request Status
Review Assigned Tojorge-garcia
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0040146: The prepayment limit check is not correctly done in Pay Open Tickets for layaways
DescriptionIn Pay Open Tickets there is the possibility to specify an amount to layaway for existing layaways, by clicking the corresponding layaway row and specifying an amount.

However, in this case the layaway limit should apply for the prepayment, and the normal receipt limit is applied instead.
Steps To Reproduce- Create a layaway for total 150 and sync it to the backoffice.
- In Pay Open Tickets, select the layaway.
- Click on the layaway, and specify 20.
- Do a payment for 20 and try to complete the layaway. Notice that the approval for the normal receipt payment limit appears. This is not correct
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 0040222RR19Q1 closed javierRodriguez The prepayment limit check is not correctly done in Pay Open Tickets for layaways 
Attached Files? issue_40146.export (3,285) 2019-02-05 14:04
https://issues.openbravo.com/file_download.php?file_id=12623&type=bug
Issue History
Date ModifiedUsernameFieldChange
2019-02-05 13:30marvintmNew Issue
2019-02-05 13:30marvintmAssigned To => Retail
2019-02-05 13:30marvintmOBNetwork customer => No
2019-02-05 13:30marvintmTriggers an Emergency Pack => No
2019-02-05 13:30marvintmAssigned ToRetail => javierRodriguez
2019-02-05 14:04asiermartirenaFile Added: issue_40146.export
2019-02-13 14:41hgbotCheckin
2019-02-13 14:41hgbotNote Added: 0109804
2019-02-13 14:41hgbotStatusnew => resolved
2019-02-13 14:41hgbotResolutionopen => fixed
2019-02-13 14:41hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4ce8ab366da80194279cb6987ade651b3334fd7a [^]
2019-02-15 12:47marvintmStatusresolved => new
2019-02-15 12:47marvintmResolutionfixed => open
2019-02-15 12:47marvintmStatusnew => scheduled
2019-02-15 12:47marvintmStatusscheduled => resolved
2019-02-15 12:47marvintmFixed in Version => RR19Q2
2019-02-15 12:47marvintmResolutionopen => fixed
2019-02-28 16:05hgbotCheckin
2019-02-28 16:05hgbotNote Added: 0110125
2019-02-28 16:28jorge-garciaReview Assigned To => jorge-garcia
2019-02-28 16:37jorge-garciaNote Added: 0110127
2019-02-28 16:37jorge-garciaStatusresolved => closed
2019-03-04 15:28hgbotCheckin
2019-03-04 15:28hgbotNote Added: 0110197

Notes
(0109804)
hgbot   
2019-02-13 14:41   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 4ce8ab366da80194279cb6987ade651b3334fd7a
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Tue Feb 05 14:03:22 2019 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4ce8ab366da80194279cb6987ade651b3334fd7a [^]

Fixed issue 40146: The prepayment limit check is not correctly done in Pay Open Tickets for layaways

Layaway orders must count their prepayment limit for layaways when checking the prepayment limit in Pay Open Tickets.

---
M web/org.openbravo.retail.posterminal/js/components/order.js
M web/org.openbravo.retail.posterminal/js/model/order.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---
(0110125)
hgbot   
2019-02-28 16:05   
Repository: tools/automation/pi-mobile
Changeset: 3bfa27418da8b31aa79402b4668e60ab88530c89
Author: Alejandro <alekosmp86 <at> gmail.com>
Date: Thu Feb 21 11:20:33 2019 -0500
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/3bfa27418da8b31aa79402b4668e60ab88530c89 [^]

Related to issue 40146: added automated test

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I40146_VerifyPrepaymentLimitCheckInLayaways.java
---
(0110127)
jorge-garcia   
2019-02-28 16:37   
Code reviewed and tested.
(0110197)
hgbot   
2019-03-04 15:28   
Repository: tools/automation/pi-mobile
Changeset: c27f3876f5534916fce58d92dbf282949b9a562a
Author: Alejandro <alekosmp86 <at> gmail.com>
Date: Thu Feb 28 16:41:34 2019 -0500
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/c27f3876f5534916fce58d92dbf282949b9a562a [^]

Related to issue 40146: fixed test in order to run in a properly configured terminal

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/layaway/I40146_VerifyPrepaymentLimitCheckInLayaways.java
---