Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0053814Retail ModulesCopy Retail Storepublic2023-10-31 11:032023-12-05 12:15
Reportermartinsdan 
Assigned Tosreehari 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS30OS VersionUbuntu 18.04.1 LTS
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0053814: There is a problem spreading payment methods if the Payment Method is set as Leave as Credit
DescriptionWhen spreading a Payment Type, it fails trying to copy the Financial Account because there is no financial account configured in this scenario.
Steps To Reproduce- Create a new Payment Method in a Touchpoint Type and set as Leave as Credit
- Create a new Payment Type in a Touchpoint
- Click Spread Payemnt to Selected Tills and select the Touchpoint Type being used

An error is shown (it is a Null Pointer Exception).
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to feature request 0052826 closed ranjith_qualiantech_com Move copy payment method to copy retail store module 
Attached Filespng error.png (139,759) 2023-10-31 11:03
https://issues.openbravo.com/file_download.php?file_id=19110&type=bug
png

png PaymentType.png (161,539) 2023-10-31 11:04
https://issues.openbravo.com/file_download.php?file_id=19111&type=bug
png

png PaymentMethod.png (185,296) 2023-10-31 11:04
https://issues.openbravo.com/file_download.php?file_id=19112&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2023-10-31 11:03martinsdanNew Issue
2023-10-31 11:03martinsdanAssigned To => Retail
2023-10-31 11:03martinsdanFile Added: error.png
2023-10-31 11:03martinsdanTriggers an Emergency Pack => No
2023-10-31 11:04martinsdanRelationship addedrelated to 0052826
2023-10-31 11:04martinsdanFile Added: PaymentType.png
2023-10-31 11:04martinsdanFile Added: PaymentMethod.png
2023-11-03 06:17sreehariAssigned ToRetail => sreehari
2023-11-03 11:31sreehariStatusnew => scheduled
2023-11-09 11:16sreehariNote Added: 0156948
2023-11-09 11:16sreehariAssigned Tosreehari => martinsdan
2023-11-16 18:23martinsdanSummaryThere is a problem spreading payment methods if the Payment Method doesn't have the same search key as the Payment Type => There is a problem spreading payment methods if the Payment Method is set as Leave as Credit
2023-11-16 18:23martinsdanDescription Updatedbug_revision_view_page.php?rev_id=27140#r27140
2023-11-16 18:23martinsdanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=27142#r27142
2023-11-16 18:23martinsdanProposed Solution updated
2023-11-16 18:24martinsdanNote Added: 0157248
2023-11-17 08:27aaroncaleroAssigned Tomartinsdan => Retail
2023-11-20 08:32sreehariAssigned ToRetail => sreehari
2023-11-22 06:13hgbotNote Added: 0157438
2023-12-05 12:15hgbotResolutionopen => fixed
2023-12-05 12:15hgbotStatusscheduled => closed
2023-12-05 12:15hgbotNote Added: 0157966
2023-12-05 12:15hgbotNote Added: 0157967

Notes
(0156948)
sreehari   
2023-11-09 11:16   
As Discussed with Antonio & Daniel, the current reported issue is no longer reproducible. But, when we dont have the financial account config and try to spread the terminal payment we get the error reported in this issue.
(0157248)
martinsdan   
2023-11-16 18:24   
I have updated the description of the issue according to the problem found and discussed
(0157438)
hgbot   
2023-11-22 06:13   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.copystore/-/merge_requests/55 [^]
(0157966)
hgbot   
2023-12-05 12:15   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.copystore [^]
Changeset: 2dba4a263bf3382fbde61e7e07ac5b5a94589bd7
Author: Sreehari Venkataraman <sreehari.venkataraman.ext@openbravo.com>
Date: 05-12-2023 12:13:24
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.copystore/-/commit/2dba4a263bf3382fbde61e7e07ac5b5a94589bd7 [^]

Fixes ISSUE-53814: Added condition to Skip Financial Account creation

*When a payment method with no financial account is set to spread terminal then skips the financial account creation

---
M src/org/openbravo/retail/copystore/process/CopyPaymentMethodProcess.java
---
(0157967)
hgbot   
2023-12-05 12:15   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.copystore/-/merge_requests/55 [^]