Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0044410Retail ModulesWeb POSpublic2020-06-18 12:172020-07-09 11:40
ReporterLeyre 
Assigned Toranjith_qualiantech_com 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR20Q4 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerOBPS
Support ticket16517
Regression levelProduction - Confirmed Stable
Regression date2019-11-06
Regression introduced in releaseRR20Q1
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4369b93ea8521399ca95e53ce68e2187ea037a03 [^]
Triggers an Emergency PackNo
Summary0044410: Incorrect cash-up values: "Total to Keep" and "Total to Deposit".
DescriptionIncorrect cash-up values: "Total to Keep" and "Total to Deposit".
Steps To ReproduceBackend:
https://livebuilds.openbravo.com/retail_pi_pgsql/ [^]

1. Configure the payment methods of a Touchpoint Type -> Cash Up section(see attached image backend.png):
   Count Payment in Cash Up = Y
   Automate Movement to Other Account = Y
   Allow Not to Move = Y
   Rest of the fields = N

Web POS:
https://livebuilds.openbravo.com/retail_pi_pgsql/web/org.openbravo.retail.posterminal/?terminal=VBS-1 [^]

1. Open a Touchpoint, which belongs to the previous Touchpoint Type.
2. Finish and pay a receipt. In our example: product price = 254.10 and payment method = cash.
3. Cash up

Actual result: See attached image webpos.png and verify that the Total to Keep = 0.00 and the Total to Deposit = 254.10

Expected result: Total to Keep = 254.10 and Total to Deposit = 0.00.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0044386 closed ranjith_qualiantech_com Error in Cash Up when "Automate Movement to Other Account" is not selected 
depends on backport 0044555RR20Q3 closed ranjith_qualiantech_com Incorrect cash-up values: "Total to Keep" and "Total to Deposit". 
depends on backport 0044556RR20Q2.1 closed ranjith_qualiantech_com Incorrect cash-up values: "Total to Keep" and "Total to Deposit". 
depends on backport 0044557RR20Q1.3 closed ranjith_qualiantech_com Incorrect cash-up values: "Total to Keep" and "Total to Deposit". 
caused by defect 0042108 closed ranjith_qualiantech_com Amount to keep can be higher than total counted for a payment method in cash up 
related to defect 0045350 closed ranjith_qualiantech_com Incorrect cash-up values: "Total to Keep" 
Attached Filespng backend.png (182,550) 2020-06-18 12:17
https://issues.openbravo.com/file_download.php?file_id=14647&type=bug
png

png webpos.png (128,961) 2020-06-18 12:17
https://issues.openbravo.com/file_download.php?file_id=14648&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2020-06-18 12:17LeyreNew Issue
2020-06-18 12:17LeyreAssigned To => Retail
2020-06-18 12:17LeyreOBNetwork customer => OBPS
2020-06-18 12:17LeyreSupport ticket => 16517
2020-06-18 12:17LeyreResolution time => 1594245600
2020-06-18 12:17LeyreTriggers an Emergency Pack => No
2020-06-18 12:17LeyreFile Added: backend.png
2020-06-18 12:17LeyreFile Added: webpos.png
2020-06-26 08:43ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2020-07-02 13:34ranjith_qualiantech_comStatusnew => scheduled
2020-07-03 10:56ranjith_qualiantech_comRegression level => Production - Confirmed Stable
2020-07-03 10:56ranjith_qualiantech_comRegression date => 2019-11-06
2020-07-03 10:56ranjith_qualiantech_comRegression introduced in release => RR20Q1
2020-07-03 10:56ranjith_qualiantech_comRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4369b93ea8521399ca95e53ce68e2187ea037a03 [^]
2020-07-03 11:00ranjith_qualiantech_comRelationship addedcaused by 0042108
2020-07-05 18:01hgbotMerge Request Status => open
2020-07-05 18:01hgbotNote Added: 0121257
2020-07-06 07:56marvintmStatusscheduled => acknowledged
2020-07-06 07:56marvintmStatusacknowledged => scheduled
2020-07-08 08:57hgbotMerge Request Statusopen => approved
2020-07-08 12:36hgbotResolutionopen => fixed
2020-07-08 12:36hgbotStatusscheduled => closed
2020-07-08 12:36hgbotNote Added: 0121327
2020-07-09 10:12LeyreRelationship addedrelated to 0044386
2020-07-09 11:40ranjith_qualiantech_comFixed in Version => RR20Q4
2020-12-09 07:23ranjith_qualiantech_comRelationship addedrelated to 0045350

Notes
(0121257)
hgbot   
2020-07-05 18:01   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/28 [^]
(0121327)
hgbot   
2020-07-08 12:36   
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: b4abf5e8e48327cdda49cfa5219c6dbdada3c45c
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2020-07-08T16:05:57+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/b4abf5e8e48327cdda49cfa5219c6dbdada3c45c [^]

Fixed issue-44410: Payment method counted should be used to update the qtyToKeep

* If CountCash is not configured for Payment method, then qtyToKeep will be updated by counted instead of foreignCounted

---
M web/org.openbravo.retail.posterminal/js/closecash/model/closecash-model.js
---