Anonymous | Login
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajorhave not tried2016-02-24 20:142016-03-21 19:07
ReportermarvintmView Statuspublic 
Assigned Tomarvintm 
PrioritynormalResolutionfixedFixed in VersionRR16Q2
StatusclosedFix in branchFixed in SCM revisionaecf07933024
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toguilleaer
Regression levelProduction - Confirmed Stable
Regression date2015-05-08
Regression introduced in releaseRR15Q3
Regression introduced by commit [^]
Triggers an Emergency PackNo

0032331: Tickets not included in cashup if cashup was done offline

DescriptionIn certain circumstances, if tickets are created while being offline, and then a cashup is done also while being offline, then the cashup may be processed before all tickets have been processed. In this case, the tickets will not be invoiced, and their payments will be missing in the cashup reconciliation.
Steps To ReproduceThis issue is very easy to reproduce in 15Q3, but it's much harder to reproduce in 15Q4 or 16Q1, although it's still theoretically possible to face it there.

The steps to reproduce are the following:
- Log in the Web POS. Go offline.
- Create five or more tickets.
- Do the cashup.
- Go back online. Log in again.
- Verify that the tickets have been synchronized, and the cashup is processed. However, all tickets except the first one will not be invoiced, and their payments will not be reconciled.
Proposed SolutionThe cause of the problem is that the synchronization tasks in the backend are not processing correctly entries which contain more than one record in the JSON. In this case, the import entry should be marked as processed only when all the records have been processed, and currently this is not the case (the entry is marked as processed as soon as the first record is processed).
TagsNo tags attached.
Attached Filesdiff file icon issue32331.pi.diff [^] (1,226 bytes) 2016-02-24 20:40 [Show Content]
diff file icon issue32331.mobilecore.diff [^] (1,730 bytes) 2016-02-24 20:40 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 0032332RR16Q1 closedmarvintm Tickets not included in cashup if cashup was done offline 
depends on backport 0032333RR15Q4.4 closedmarvintm Tickets not included in cashup if cashup was done offline 
caused by feature request 0029766RR15Q3 closedmtaal Retail Operations Buffer: store all transactions in operations table before processing 

-  Notes
hgbot (developer)
2016-03-08 11:21

Repository: erp/devel/pi
Changeset: 1e5083005e09288f96453a38ff05026794705514
Author: Antonio Moreno Perez <antonio.moreno <at>>
Date: Tue Mar 08 11:21:24 2016 +0100
URL: [^]

Fixed issue 32331. Added method to check if importentry has status 'Error'

M src/org/openbravo/service/importprocess/
hgbot (developer)
2016-03-08 11:22

Repository: erp/pmods/
Changeset: aecf07933024310744d7f762337caeb43bfcb8af
Author: Antonio Moreno Perez <antonio.moreno <at>>
Date: Tue Mar 08 11:22:39 2016 +0100
URL: [^]

Fixed issue 32331. Prevent early update to the Import Entry record:
Before this change, if a request to the DataSynchronizationProcess contained more than one record, the Import Entry record was updated after the first one instead of being updated after the last one.

M src/org/openbravo/mobile/core/process/
hudsonbot (developer)
2016-03-17 10:56

A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2016-02-24 20:14 marvintm New Issue
2016-02-24 20:14 marvintm Assigned To => Retail
2016-02-24 20:14 marvintm Triggers an Emergency Pack => No
2016-02-24 20:14 marvintm Assigned To Retail => marvintm
2016-02-24 20:14 marvintm Status new => scheduled
2016-02-24 20:19 marvintm Regression level => Production - Confirmed Stable
2016-02-24 20:19 marvintm Regression date => 2015-05-08
2016-02-24 20:19 marvintm Regression introduced in release => RR15Q3
2016-02-24 20:19 marvintm Regression introduced by commit => [^]
2016-02-24 20:40 dmitry_mezentsev Relationship added caused by 0029766
2016-02-24 20:40 marvintm File Added: issue32331.pi.diff
2016-02-24 20:40 marvintm File Added: issue32331.mobilecore.diff
2016-03-08 11:21 hgbot Checkin
2016-03-08 11:21 hgbot Note Added: 0084814
2016-03-08 11:21 hgbot Status scheduled => resolved
2016-03-08 11:21 hgbot Resolution open => fixed
2016-03-08 11:21 hgbot Fixed in SCM revision => [^]
2016-03-08 11:22 hgbot Checkin
2016-03-08 11:22 hgbot Note Added: 0084815
2016-03-08 11:22 hgbot Fixed in SCM revision [^] => [^]
2016-03-17 10:56 hudsonbot Checkin
2016-03-17 10:56 hudsonbot Note Added: 0085191
2016-03-21 19:07 guilleaer Review Assigned To => guilleaer
2016-03-21 19:07 guilleaer Status resolved => closed
2016-03-21 19:07 guilleaer Fixed in Version => RR16Q2

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker