Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0045766Retail ModulesWeb POSpublic2021-01-20 16:262021-01-27 15:36
Reporterlmonreal 
Assigned Toranjith_qualiantech_com 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product VersionRR19Q3.3 
Target VersionFixed in VersionRR21Q2 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0045766: Able to do a cash management event with more than three decimals storing wrong data in BO
DescriptionIn WEBPOS user is able to do a cash management event with more than three decimals.
This is generating that the data is wrongly stored in database.

This should not be allowed from WEBPOS.

It should be needed to check too the Open till and cashUp

Steps To Reproduce1. Open WEBPOS (19Q3)
2. Do a Cash Management event (deposit or withdrawal) of 40.526 (or any value containing three decimals)
3. Complete transaction
4. Check database, the value stored is 40.526 (if you check the backoffice you will see the 2 digits only)

https://drive.google.com/file/d/1SvDlMwax3NYS7AXI3I50bLB78kKlvt3f/view [^]

It is leaving the database (fin_finacc_transaction) with wrong values

Proposed SolutionWEBPOS should not allow cash management events of more than the currency precision, it should stop/warn user.

It should be needed to check too the Open till and cashUp
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff 45766_19Q3.3_posterminal.diff (2,995) 2021-01-21 15:35
https://issues.openbravo.com/file_download.php?file_id=15274&type=bug
Issue History
Date ModifiedUsernameFieldChange
2021-01-20 16:26lmonrealNew Issue
2021-01-20 16:26lmonrealAssigned To => Retail
2021-01-20 16:26lmonrealOBNetwork customer => Gold
2021-01-20 16:26lmonrealTriggers an Emergency Pack => No
2021-01-20 16:27lmonrealSummaryAble to do a cash management event with three decimals storing wrong data in BO => Able to do a cash management event with more than three decimals storing wrong data in BO
2021-01-20 16:27lmonrealSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=22053#r22053
2021-01-20 16:27lmonrealDescription Updatedbug_revision_view_page.php?rev_id=22055#r22055
2021-01-20 16:27guillermogilResolution time => 1612911600
2021-01-20 16:29guillermogilversion => RR19Q3.3
2021-01-20 16:29guillermogilSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=22056#r22056
2021-01-20 16:29guillermogilProposed Solution updated
2021-01-20 16:30lmonrealSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=22057#r22057
2021-01-20 16:31guillermogilDescription Updatedbug_revision_view_page.php?rev_id=22058#r22058
2021-01-20 16:31guillermogilProposed Solution updated
2021-01-21 05:50ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2021-01-21 07:11guillermogilSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=22059#r22059
2021-01-21 10:11hgbotMerge Request Status => open
2021-01-21 10:11hgbotNote Added: 0125706
2021-01-21 10:11ranjith_qualiantech_comFile Added: 45766_19Q3.3_posterminal.diff
2021-01-21 13:36hgbotMerge Request Statusopen => approved
2021-01-21 13:49guillermogilNote Added: 0125711
2021-01-21 15:35ranjith_qualiantech_comFile Deleted: 45766_19Q3.3_posterminal.diff
2021-01-21 15:35ranjith_qualiantech_comFile Added: 45766_19Q3.3_posterminal.diff
2021-01-21 16:40hgbotMerge Request Statusapproved => open
2021-01-21 16:40guillermogilNote Added: 0125715
2021-01-22 06:43guillermogilNote Deleted: 0125715
2021-01-22 09:40hgbotMerge Request Statusopen => approved
2021-01-22 09:40hgbotResolutionopen => fixed
2021-01-22 09:40hgbotStatusnew => closed
2021-01-22 09:40hgbotNote Added: 0125727
2021-01-22 09:40hgbotFixed in Version => RR21Q2
2021-01-22 09:40hgbotNote Added: 0125728
2021-01-27 15:36hgbotNote Added: 0125804

Notes
(0125706)
hgbot   
2021-01-21 10:11   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/272 [^]
(0125711)
guillermogil   
2021-01-21 13:49   
Attached patch needs to be updated
(0125727)
hgbot   
2021-01-22 09:40   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/272 [^]
(0125728)
hgbot   
2021-01-22 09:40   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 6c31d7b34367e0f4a445e87915ccf8f89a8527d7
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2021-01-22T08:40:17+00:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/6c31d7b34367e0f4a445e87915ccf8f89a8527d7 [^]

Fixed ISSUE-45766: CashMgmt deposit/drop amount should be updated with price precision

---
M src-db/database/sourcedata/AD_MESSAGE.xml
M web/org.openbravo.retail.posterminal/js/cashmgmt/view/cashmgmtkeyboard.js
---
(0125804)
hgbot   
2021-01-27 15:36   
Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 0f5b6cf9ef1d07e97dbdd6193328e950ef0acb4a
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2021-01-27T20:06:17+05:30
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/0f5b6cf9ef1d07e97dbdd6193328e950ef0acb4a [^]

Related to ISSUE-45766: Updated Test I36785_CashManagementDepositRoundedAmount

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashmanagement/I36785_CashManagementDepositRoundedAmountToWithdrawnOrDeposit.java
---