0037362: The financial account configuration for a payment method is not taken into account to create reconciliations in the cashup

Openbravo Issue Tracking System - Retail Modules

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0037362

Retail Modules

Cash up

public

2017-07-21 10:02

2017-12-20 19:49

Reporter

aaroncalero

Assigned To

ranjith_qualiantech_com

Priority

urgent

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

RR17Q3.2

Fixed in Version

RR17Q3.2

Merge Request Status

Review Assigned To

marvintm

OBNetwork customer

Support ticket

Regression level

Pre packaging ( pi )

Regression date

Regression introduced in release

Regression introduced by commit

https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3dbcb6b316eb645730de8604dbc5f2df21ef91d8 [^]

Triggers an Emergency Pack

Summary

0037362: The financial account configuration for a payment method is not taken into account to create reconciliations in the cashup

Description

When a cashup is closed, transactions created during that cashup are reconciled adn the money is moved to the financial account defined on the Close events for each payment method.
The "automatic deposit" flag of the payment method defines if these reconciliations should be created or not.
This flag can be found on the generic Payment Method configuration, and also on the Financial Account Payment method configuration (which should override the generic configuration).

The problem is that in order to decide if reconciliations should be created or not, the generic configuration is being taken into account, instead of the configuration on the financial account.

Steps To Reproduce

Using White Valley sampledata:
Log in in backend, go to the Payment Method window, locate the "Wire Transfer" record (this is the Card payment method in webpos) and verify that the Automatic Deposit flag is checked.
Go to the Financial account window, locate the "Credit Card - Vall Blanca Store" record and on the Payment Method tab, verify that the Wire Transfer record is also configured as Automatic Deposit.
Go to the Organization window, Vall Blanca Store record, and on the Cash Management Events tab locate the "Credit Card - Close - Vall Blanca Store" record.

Log in in webpos, terminal VBS-1 and do a cashup to clear all values
Create a new ticket, add a product and complete the ticket paying with Card
Do another cashup.
Go to backend, POS Terminal window; select the VBS-1 record, go to the Payment Type tab, select Card and open the Reconciliation subtab. Verify that a new reconciliation has been created. Navigate to the reconciliation and verify that on the Cleared Items subtab there are two transactions (the payment and the movement to the Close Financial Account).

Go back to the Payment Method window, Wire Transfer record and uncheck the Automatic Deposit flag.
Go back to webpos, create a new ticket, pay it with card, and do another cashup.
Verify that this time, there is no new reconciliation created for the Card payment method, even if the configuration on the Financial Account window still flags the payment method as Automatic Deposit.

Proposed Solution

The following line on the commit that causes the issue:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/3dbcb6b316eb645730de8604dbc5f2df21ef91d8#l1.14 [^]
reads the automatic deposit flag directly from the payment method, which is wrong.

At this point we have the information of the financial account and the payment method, so it should be easy to refactor that code to get the Automatic Deposit flag from the financial account configuration.

Additional Information