Openbravo Issue Tracking System - Retail Modules
View Issue Details
0038177Retail ModulesWeb POSpublic2018-03-20 13:062018-03-22 09:43
asiermartirena 
ranjith_qualiantech_com 
highmajoralways
closedfixed 
5
 
RR18Q1.1RR18Q1.1 
marvintm
Coding ( Testing )
2017-10-11
RR18Q1
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6c903c081a0a [^]
No
0038177: JS error when trying to pay a ticket with a payment method not in that terminal
There's a javascript error when trying to pay a ticket in a terminal that doesn't contain the payment method of the payment used to pay (partially or totally) the ticket.

In the 'checkValidOverpayment' function there's a line that takes the payment method of the payment, and tries to take that payment method from the terminal. As the payment method is not in the terminal, there's a null pointer. This is the line:

overPaymentMethod = overPaymentUsed ? OB.MobileApp.model.paymentnames[overPaymentUsed.get('kind')].paymentMethod : undefined

This only occurs in a terminal that contains at least one payment method, otherwise this function is not called.
Load in a terminal with the payment method 'Card'.

Create a ticket and pay partially or totally by 'Card'.

Load the Web POS in a terminal that doesn't contain the 'Card' payment method.

Load the previously created ticket.

Click on the total amount button -----> A javascript error appears
No tags attached.
blocks defect 0038176 closed ranjith_qualiantech_com JS error when trying to pay a ticket with a payment method not in that terminal 
Issue History
2018-03-20 14:01marvintmTypedefect => backport
2018-03-20 14:01marvintmTarget Version => RR18Q1.1
2018-03-21 07:28ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2018-03-22 08:40hgbotCheckin
2018-03-22 08:40hgbotNote Added: 0103409
2018-03-22 08:40hgbotStatusscheduled => resolved
2018-03-22 08:40hgbotResolutionopen => fixed
2018-03-22 08:40hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR18Q1.1/org.openbravo.retail.posterminal/rev/d1e8ac9d03b845eb7f0ec2b200381c52a0e72d34 [^]
2018-03-22 09:43marvintmReview Assigned To => marvintm
2018-03-22 09:43marvintmStatusresolved => closed
2018-03-22 09:43marvintmFixed in Version => RR18Q1.1

Notes
(0103409)
hgbot   
2018-03-22 08:40   
Repository: retail/backports/3.0RR18Q1.1/org.openbravo.retail.posterminal
Changeset: d1e8ac9d03b845eb7f0ec2b200381c52a0e72d34
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Mar 22 11:35:56 2018 +0530
URL: http://code.openbravo.com/retail/backports/3.0RR18Q1.1/org.openbravo.retail.posterminal/rev/d1e8ac9d03b845eb7f0ec2b200381c52a0e72d34 [^]

Fixed issue 38177 : Validate receipt payment with terminal payment

* If receipt payment is not exists in terminal payments, it should not validate overpayment

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/payment.js
---