Openbravo Issue Tracking System - Retail Modules
View Issue Details
0033172Retail ModulesSessionspublic2016-06-07 16:592016-07-20 09:14
aaroncalero 
jorge-garcia 
highmajoralways
closedfixed 
5
 
 
marvintm
No
0033172: [SER QA 1245] Till differences generated during the initial count are not linked to a cashup
Using the Sessions module, after every cashup, it is required to open the till and perform an initial cash count.
If cash differences are found during this process, a Cash Management Movement record is processed in backend to generate the correct transactions.
These transactions are created without a link to the cashup they belong to, and they are not reconciled automatically when the cashup is processed.
Login in a terminal with the Sessions module installed (e.g. https://livebuilds.openbravo.com/retail_modules_pgsql_pi/web/org.openbravo.retail.posterminal/?terminal=VBS-1#retail.pointofsale [^]).
Perform a cashup, keeping nothing.
The terminal will automatically logout after the cashup process is completed.
Verify in backend that the last available cashup is the cashup that was just processed (POS Terminal window > Cash Up History tab).
Login again in the pos terminal.
The Open Till window will be shown, requesting a cash count.
Count some cash, adding some differences since the expected cash is 0.
Open the browser console to monitor the log messages.
Click on Approve difference.
Verify on the console that the Cash Management is processed and then the Cash Up is processed.
Go to the backend, POS Terminal window, Payment Type tab, and from there Navigate to the financial account of the Cash payment type.
Verify on the Transaction tab that there is a new record with description 'Cash - Till differences' with the amount entered before.
Check the same record in database and verify that the em_obpos_app_cashup_id field is empty.
SER-QA
related to defect 0031783 closed guilleaer [SERQA 503] [Sessions] Differences in the open till process are not taken into account in the cash up. 
related to defect 0032177 closed ranjith_qualiantech_com [SERQA 648] POS Terminal || Cash up history confusing line with all values to zero 
related to defect 0034235pi closed jorge-garcia cash up not created when the till is open 
causes defect 0035602 closed marvintm It is not possible to process one receipt in Error While Importing, if the cashup has been completed 
Issue History
2016-06-07 16:59aaroncaleroNew Issue
2016-06-07 16:59aaroncaleroAssigned To => Retail
2016-06-07 16:59aaroncaleroResolution time => 1466460000
2016-06-07 16:59aaroncaleroTriggers an Emergency Pack => No
2016-06-07 16:59aaroncaleroRelationship addedrelated to 0031783
2016-06-07 17:00aaroncaleroRelationship addedrelated to 0032177
2016-06-08 10:07VictorVillarTag Attached: SER-QA
2016-06-08 10:07VictorVillarSummaryTill differences generated during the initial count are not linked to a cashup => [SER QA 1245] Till differences generated during the initial count are not linked to a cashup
2016-06-09 10:23marvintmStatusnew => acknowledged
2016-06-09 13:32shivum039Statusacknowledged => scheduled
2016-06-09 13:32shivum039Assigned ToRetail => shivum039
2016-06-15 19:19marvintmAssigned Toshivum039 => jorge-garcia
2016-07-15 12:33hgbotCheckin
2016-07-15 12:33hgbotNote Added: 0088520
2016-07-15 12:33hgbotCheckin
2016-07-15 12:33hgbotNote Added: 0088522
2016-07-15 12:34hgbotCheckin
2016-07-15 12:34hgbotNote Added: 0088524
2016-07-15 12:34hgbotStatusscheduled => resolved
2016-07-15 12:34hgbotResolutionopen => fixed
2016-07-15 12:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ee815642094928b0fa2bc661160011f33fae1b5 [^]
2016-07-15 15:01hgbotCheckin
2016-07-15 15:01hgbotNote Added: 0088545
2016-07-15 15:05marvintmReview Assigned To => marvintm
2016-07-15 15:05marvintmStatusresolved => closed
2016-07-20 09:14hgbotCheckin
2016-07-20 09:14hgbotNote Added: 0088590
2016-10-19 10:24jorge-garciaRelationship addedrelated to 0034235
2017-04-05 17:30dmitry_mezentsevRelationship addedcauses 0035602

Notes
(0088520)
hgbot   
2016-07-15 12:33   
Repository: tools/automation/pi-mobile
Changeset: 0c447c8d6c61a072ffd8a10ffef343f9e9908bd7
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 11:42:17 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/0c447c8d6c61a072ffd8a10ffef343f9e9908bd7 [^]

Verifies issue 33172, Verifies issue 32875: CashUp changes

Change some tests to adjust to the new functionality.

---
M src-test/org/openbravo/test/mobile/common/selenium/utils/OBOfflineUtils.java
M src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/cashup/I32177_VerifyInitCashupAfterCashup_II.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/cashup/I32177_VerifyInitCashupAfterCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineCashupWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineSimplesale.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOfflineSimplesaleWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOnlineCashup.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/offline/LoginOnlineCashupWait.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/system/I30962_CheckUpdateModalBeingProcessed.java
---
(0088522)
hgbot   
2016-07-15 12:33   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 13825ee93af9b930df7da27a3f23d41d1a21c3fc
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 11:44:44 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/13825ee93af9b930df7da27a3f23d41d1a21c3fc [^]

Related to issue 33172, Related to issue 32875: CashUp changes

Change synchronization criterias of the CashUp model.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-modalmodelstosync.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0088524)
hgbot   
2016-07-15 12:34   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 8ee815642094928b0fa2bc661160011f33fae1b5
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 12:31:13 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/8ee815642094928b0fa2bc661160011f33fae1b5 [^]

Fixed issue 33172, Fixed issue 32875: CashUp changes

The CashUp must be the last model to be synchronized in the backend. Sometimes,
this is not the case, and some synchronization models (such as Order and
CashMgmt) are not related with any cashup.

The solution, is to send the cashUp report within the synchronization models
(Order, CashMgmt, VoidLayaway and the CashUp itself).

---
M src-db/database/model/tables/OBPOS_APP_CASHUP.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/retail/posterminal/OrderLoader.java
M src/org/openbravo/retail/posterminal/ProcessCashClose.java
M src/org/openbravo/retail/posterminal/ProcessCashMgmt.java
M src/org/openbravo/retail/posterminal/ProcessVoidLayaway.java
M web/org.openbravo.retail.posterminal/js/cashmgmt/model/cashmgmt-model.js
M web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
M web/org.openbravo.retail.posterminal/js/model/cashmanagement.js
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
M web/org.openbravo.retail.posterminal/js/utils/cashManagementUtils.js
M web/org.openbravo.retail.posterminal/js/utils/cashUpReportUtils.js
A src/org/openbravo/retail/posterminal/UpdateCashup.java
---
(0088545)
hgbot   
2016-07-15 15:01   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: b99523e5ae3d7998ba6760979fd3933fe228da79
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jul 15 15:00:55 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b99523e5ae3d7998ba6760979fd3933fe228da79 [^]

Related to issue 33172, Related to issue 32875: CashUp changes

Remove some code that is commented and set a proper year license.

---
M src/org/openbravo/retail/posterminal/UpdateCashup.java
M web/org.openbravo.retail.posterminal/js/data/dataordersave.js
---
(0088590)
hgbot   
2016-07-20 09:14   
Repository: tools/automation/pi-mobile
Changeset: ce0bd92a0f3fab2bbfccab7761da9416e31060a6
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Jul 19 12:47:43 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/ce0bd92a0f3fab2bbfccab7761da9416e31060a6 [^]

Verifies issue 33172: Till differences generated during the initial count are
not linked to a cashup

Added two new tests:
* I33172_VerifyTillDifferencesReconciliationDeposits
* I33172_VerifyTillDifferencesReconciliationWithdrawal

---
A src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/sessions/openstoretill/I33172_VerifyTillDifferencesReconciliationDeposits.java
A src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/sessions/openstoretill/I33172_VerifyTillDifferencesReconciliationWithdrawal.java
---