0028768: Cashup date is wrong, it is set to the cashup creation date and time and not the date/time when the cashup is finally done

Openbravo Issue Tracking System - Retail Modules
View Issue Details

ID	Project	Category	View Status	Date Submitted	Last Update
0028768	Retail Modules	Web POS	public	2015-01-26 14:49	2015-01-29 17:56

Reporter	mtaal
Assigned To	migueldejuana
Priority	urgent	Severity	major	Reproducibility	have not tried
Status	closed	Resolution	fixed
Platform		OS	5	OS Version
Product Version
Target Version	RR15Q1	Fixed in Version	RR15Q1
Merge Request Status
Review Assigned To	marvintm
OBNetwork customer	No
Support ticket
Regression level	Pre packaging ( pi )
Regression date	2014-11-07
Regression introduced in release
Regression introduced by commit	https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/195fc620eeb2fcb3b44ba1bb1540ea43ecabe330 [^]
Triggers an Emergency Pack	No

Summary	0028768: Cashup date is wrong, it is set to the cashup creation date and time and not the date/time when the cashup is finally done
Description	See the attached screenshot. The cashup date/time is equal to the creation time of the cashup, the time the till/session started. This is however not equal to the date/time the cashup was finally done/completed. See the screenshot, the first ticket of this cashup was done on the 21st of Jan, the cashup was finally done on te 26th, the cashup date/time is wrong.
Steps To Reproduce	- start a fresh cashup in the client - create some tickets - wait a few minutes - do a cashup - check in the backend that the cashup has a cashup date/time equal to the cashup creation time and not equal to the time the cashup was done
Proposed Solution	Set the cashup date/time equal to the time the cashup was finalized on the client. (note should be the client side time, not the server side creation/update time) Check the causing commit: https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/195fc620eeb2fcb3b44ba1bb1540ea43ecabe330#l14.35 [^] this commit sets the cashup date at the time the cashup record is created. While the previous version set the cashup date/time at the time the cashup report was shown: https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/195fc620eeb2fcb3b44ba1bb1540ea43ecabe330#l19.200 [^] The latter approach is correct.
Additional Information
Tags	No tags attached.
Relationships
Attached Files	Selection_096.png (84,337) 2015-01-26 14:49 https://issues.openbravo.com/file_download.php?file_id=7740&type=bug

Issue History
Date Modified	Username	Field	Change
2015-01-26 14:49	mtaal	New Issue
2015-01-26 14:49	mtaal	Assigned To	=> migueldejuana
2015-01-26 14:49	mtaal	File Added: Selection_096.png
2015-01-26 14:49	mtaal	OBNetwork customer	=> No
2015-01-26 14:49	mtaal	Regression level	=> Pre packaging ( pi )
2015-01-26 14:49	mtaal	Regression date	=> 2014-11-07
2015-01-26 14:49	mtaal	Regression introduced by commit	=> https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/195fc620eeb2fcb3b44ba1bb1540ea43ecabe330 [^]
2015-01-26 14:49	mtaal	Triggers an Emergency Pack	=> No
2015-01-26 15:16	mtaal	Note Added: 0073781
2015-01-28 10:27	hgbot	Checkin
2015-01-28 10:27	hgbot	Note Added: 0073851
2015-01-28 10:27	hgbot	Status	new => resolved
2015-01-28 10:27	hgbot	Resolution	open => fixed
2015-01-28 10:27	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/025fcecf94460e0dd9781cb3f1d6b1a1c8d821ae [^]
2015-01-29 17:56	marvintm	Review Assigned To	=> marvintm
2015-01-29 17:56	marvintm	Status	resolved => closed
2015-01-29 17:56	marvintm	Fixed in Version	=> RR15Q1

Notes