Openbravo Issue Tracking System - Retail Modules
View Issue Details
0039026Retail ModulesWeb POSpublic2018-07-24 13:042018-07-26 15:31
guilleaer 
ranjith_qualiantech_com 
normalcriticalhave not tried
closedfixed 
5
 
RR18Q4 
marvintm
Production - Confirmed Stable
2017-04-05
RR17Q3
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/741c14452bce [^]
No
0039026: Terminal with empty cache receives an error retrieving current cashup when it have all payment methods inactive
Login cannot be done in a terminal which match with the following conditions

a) Empty cache
b) Valid cashup (not processed)
c) Terminal should have several INACTIVE payment types (defined in POS terminal window)
c) Valid cashup should have a payment method cashup created for each inactive payment type

Under these circumstances, a null pointer exception is raised by org.openbravo.retail.posterminal.master.Cashup
Method: GetPayments

Below query results is null

      OBCriteria<OBPOSAppPayment> paymentAppMethodCriteria = OBDal.getInstance().createCriteria(
          OBPOSAppPayment.class);
      paymentAppMethodCriteria.add(Restrictions.eq(OBPOSAppPayment.PROPERTY_ID,
          paymentMethodJSON.get("paymentType")));
      OBPOSAppPayment paymentAppMethod = (OBPOSAppPayment) paymentAppMethodCriteria.uniqueResult();

Later on code tries to pick the lineNo field of the object created above which is null. It results in a NullPointerException

paymentMethodJSON.put("lineNo", paymentAppMethod.getLine());



3146207 [http-bio-8098-exec-62] ERROR org.openbravo.retail.posterminal.master.Cashup - Error during exec
java.lang.NullPointerException
  at org.openbravo.retail.posterminal.master.Cashup.getPayments(Cashup.java:161)
Generate above scenario
Login
When a payment method cashup is connected with an inactive payment type, then lineNo should be 1
No tags attached.
depends on backport 0039027RR18Q3 closed ranjith_qualiantech_com Terminal with empty cache receives an error retrieving current cashup when it have all payment methods inactive 
depends on backport 0039028RR18Q2.2 closed ranjith_qualiantech_com Terminal with empty cache receives an error retrieving current cashup when it have all payment methods inactive 
diff diffIssue39026_Sharaf.diff (3,469) 2018-07-24 13:05
https://issues.openbravo.com/file_download.php?file_id=11991&type=bug
? usefulQueriesToReproduce.sql (2,376) 2018-07-24 13:17
https://issues.openbravo.com/file_download.php?file_id=11992&type=bug
Issue History
2018-07-24 13:04guilleaerNew Issue
2018-07-24 13:04guilleaerAssigned To => Retail
2018-07-24 13:04guilleaerRegression level => Production - Confirmed Stable
2018-07-24 13:04guilleaerRegression date => 2017-04-05
2018-07-24 13:04guilleaerRegression introduced in release => RR17Q3
2018-07-24 13:04guilleaerRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/741c14452bce [^]
2018-07-24 13:04guilleaerTriggers an Emergency Pack => No
2018-07-24 13:05guilleaerFile Added: diffIssue39026_Sharaf.diff
2018-07-24 13:06guilleaerNote Added: 0105913
2018-07-24 13:07guilleaerStatusnew => scheduled
2018-07-24 13:17guilleaerFile Added: usefulQueriesToReproduce.sql
2018-07-24 13:17guilleaerDescription Updatedbug_revision_view_page.php?rev_id=17450#r17450
2018-07-24 13:25egoitzResolution time => 1533074400
2018-07-26 08:05ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2018-07-26 12:13hgbotCheckin
2018-07-26 12:13hgbotNote Added: 0105958
2018-07-26 12:13hgbotStatusscheduled => resolved
2018-07-26 12:13hgbotResolutionopen => fixed
2018-07-26 12:13hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/24f1e7cec00bf9acd44c4fd29258a47b917826d4 [^]
2018-07-26 15:31marvintmReview Assigned To => marvintm
2018-07-26 15:31marvintmStatusresolved => closed
2018-07-26 15:31marvintmFixed in Version => RR18Q4

Notes
(0105913)
guilleaer   
2018-07-24 13:06   
Patch applies to Sharaf environment. Should be adapted for PI and backports
(0105958)
hgbot   
2018-07-26 12:13   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 24f1e7cec00bf9acd44c4fd29258a47b917826d4
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Jul 26 15:43:32 2018 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/24f1e7cec00bf9acd44c4fd29258a47b917826d4 [^]

Fixed issue 39026 : Validate Invalid Payment Method while retrieving Cashup

* For generating Payment Method lineNo while retrieving cashup, if Payment Method is not available then it should be replaced with one

---
M src/org/openbravo/retail/posterminal/master/Cashup.java
---