Openbravo Issue Tracking System - Retail Modules
View Issue Details
0030946Retail ModulesCash uppublic2015-02-17 23:062015-10-02 15:11
malsasua 
marvintm 
normalminoralways
closedfixed 
20Community Appliance
 
RR15Q2.5RR15Q2.5 
Orekaria
No
0030946: additional checks to avoid wrong cashups
When a same terminal is accessed from two or more devices, the cashups generated by this terminal can be wrong.

When it happens, the cashup should not be processed and it should be included in "Errors while importing data" window
in device1
. clean cache
. do login in VBS-1
. new cashup with "id1" is created in local database
. create and pay a new receipt
. logout

in device2
. clean cache
. do login in VBS-1
. cashup with "id1" is loaded in local database
. create and pay a new receipt
. do a cashup
. logout

in device1
. do login in VBS-1
. cashup with "id1" is loaded in local database
. create and pay a new receipt

in backoffice, the cashup with "id1" is set as
. processed=N
. processedbo=Y

it is wrong
No tags attached.
blocks defect 0028975 closed marvintm additional checks to avoid wrong cashups 
Issue History
2015-09-28 11:04marvintmTypedefect => backport
2015-09-28 11:04marvintmTarget Version => RR15Q2.5
2015-09-28 11:05marvintmNote Added: 0080662
2015-09-28 11:06hgbotCheckin
2015-09-28 11:06hgbotNote Added: 0080664
2015-09-28 11:06hgbotStatusscheduled => resolved
2015-09-28 11:06hgbotResolutionopen => fixed
2015-09-28 11:06hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR15Q2/org.openbravo.retail.posterminal/rev/616729f0e0ecf356797e40e89e374bd7b50179f0 [^] => http://code.openbravo.com/retail/backports/3.0RR15Q2.5/org.openbravo.retail.posterminal/rev/60ae023272faad8ba85b44153e2e4c6f09d6f862 [^]
2015-10-02 15:11OrekariaReview Assigned Toadrianromero => Orekaria
2015-10-02 15:11OrekariaStatusresolved => closed
2015-10-02 15:11OrekariaFixed in Version => RR15Q2.5

Notes
(0080662)
marvintm   
2015-09-28 11:05   
This issue was originally intended to be included in 15Q2, but due to some technical issues finally it didn't happen. It will be included in 15Q2.5.
(0080664)
hgbot   
2015-09-28 11:06   
Repository: retail/backports/3.0RR15Q2.5/org.openbravo.retail.posterminal
Changeset: 60ae023272faad8ba85b44153e2e4c6f09d6f862
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Mon Sep 28 11:06:14 2015 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR15Q2.5/org.openbravo.retail.posterminal/rev/60ae023272faad8ba85b44153e2e4c6f09d6f862 [^]

Fixed issue 30946. aditionals checks to verify if the cashup was processed previously

three changes:
when a receipt is synced, in the orderloader is verified if the cashup related to the receipt has been processed previously
in the query to load the cashup in the terminal from backend, only will be loaded the cashups not processed
when a cashup is synced, it is not allowed to change the processed field from Y to N

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
M src/org/openbravo/retail/posterminal/ProcessCashClose.java
M src/org/openbravo/retail/posterminal/master/Cashup.java
M web/org.openbravo.retail.posterminal/js/utils/cashUpReportUtils.js
---