Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0035778
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Multi order paymentmajorhave not tried2017-04-12 16:422017-04-21 09:34
ReportermigueldejuanaView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR17Q3
StatusclosedFix in branchFixed in SCM revision3f8286b5dca4
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toguilleaer
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0035778: Pay Open Tickets doesn't take into account already created payments

DescriptionIn Pay Open Tickets we can load Layaways and local orders. Loading local orders in Pay Open Tickets, payments linked to that order are removed.
We have to load those payments and take them into account as we do with Layaways and previous added payments.
Local orders can have payments attached in Order Mode, those payments can be done by a payment gateway and we cannot remove them.
We need to show in Pay Open Tickets modal just Layaways and order without payments or with payments but not fully paid.
When loading orders with payments we must take them into account and show the pending amount instead of showing the total of order gross.
Steps To Reproduce- Login in Web POS
- Add Avalanche Transceiver, order total is 150,50
- Add a payment with Cash of 100
- Press Pay Open Tickets option and load created order
- ERROR! Pending to pay is 150,50
-->Solution:
*Total:150,50
*Total remaining to pay: 50,50
*Order remaining to pay: 50,50
Proposed SolutionTips:

- First remove this added code:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28fdb6187e95#l1.16 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28fdb6187e95#l3.7 [^]

- In this part of code, we just have to remove payments added in Pay Open Tickets process:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/28fdb6187e95#l2.91 [^]

- When showing Pay Open Tickets modal, just show local orders which are not fully paid.

- Do not forget to test Synchronized Mode failing in backend. Doing Reload in different cases...
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0096093)
hgbot (developer)
2017-04-20 18:04

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 3f8286b5dca49789cdab2e044eb5e1a4bd526518
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Apr 20 21:33:44 2017 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3f8286b5dca49789cdab2e044eb5e1a4bd526518 [^]

Fixes issue 35778 : Changes in Pay open ticket Payments

* Receipt Payments should be included to Pay open tickets Payments

---
M web/org.openbravo.retail.posterminal/js/components/modalmultiorders.js
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
M web/org.openbravo.retail.posterminal/js/model/order.js
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
---
(0096094)
hgbot (developer)
2017-04-20 18:04

Repository: tools/automation/pi-mobile
Changeset: 96c72b52877dcdb6beb6f7676d814bcdb22ff8ab
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Apr 20 21:33:49 2017 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/96c72b52877dcdb6beb6f7676d814bcdb22ff8ab [^]

Verifies issue 35778 : Added automated test 'I35778_VerifyPayOpenTicketPayments'

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/TestIdPack.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I35778_VerifyPayOpenTicketPayments.java
---

- Issue History
Date Modified Username Field Change
2017-04-12 16:42 migueldejuana New Issue
2017-04-12 16:42 migueldejuana Assigned To => Retail
2017-04-12 16:42 migueldejuana Triggers an Emergency Pack => No
2017-04-12 17:32 marvintm Resolution time => 1493762400
2017-04-12 17:33 marvintm Severity minor => major
2017-04-18 13:39 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2017-04-18 13:39 ranjith_qualiantech_com Status new => scheduled
2017-04-20 18:04 hgbot Checkin
2017-04-20 18:04 hgbot Note Added: 0096093
2017-04-20 18:04 hgbot Status scheduled => resolved
2017-04-20 18:04 hgbot Resolution open => fixed
2017-04-20 18:04 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3f8286b5dca49789cdab2e044eb5e1a4bd526518 [^]
2017-04-20 18:04 hgbot Checkin
2017-04-20 18:04 hgbot Note Added: 0096094
2017-04-21 09:34 guilleaer Review Assigned To => guilleaer
2017-04-21 09:34 guilleaer Status resolved => closed
2017-04-21 09:34 guilleaer Fixed in Version => RR17Q3


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker