Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0028608Retail ModulesManual user actionpublic2015-01-14 11:312016-01-25 20:18
Reporterlorenzofidalgo 
Assigned Tomarvintm 
PriorityurgentSeverityminorReproducibilityalways
StatusnewResolutionopen 
PlatformOS5OS Version
Product VersionRR14Q4 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0028608: After completing a receipt, POS shows a successful synchronization message, but Sales Orders are lost
DescriptionTested in livebuilds RR14Q4, RR14Q3.4 & RR14Q2.5.

Once completed a receipt, POS shows the following message: "Data synchronization was successful". Checking "Sales Order" backend, the receipt never appears.
After checking log, it appears: "ERROR org.openbravo.mobile.core.process.DataSynchronizationProcess - Error while loading order
org.openbravo.base.exception.OBException: The Period does not exist or it is not opened"

POS should not allow users to "complete" receipts in POS or showing successul messages if it is not possible to treat them succeessfully.
All those completed receipts in POS are lost.
Steps To ReproduceLogin POS, choose any product and complete a receipt.
Go to backend and login as Openbravo/openbravo and choose "The White Valley Group Admin" role.
Go to Sales Order window. Check there is no new sales orders.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0028562 closed marvintm Cash up does not finish and an error is shown at the console 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-01-14 11:31lorenzofidalgoNew Issue
2015-01-14 11:31lorenzofidalgoAssigned To => marvintm
2015-01-14 11:31lorenzofidalgoTriggers an Emergency Pack => No
2015-01-14 11:34lorenzofidalgoRelationship addedrelated to 0028562
2015-01-30 11:33malsasuaNote Added: 0073940
2015-01-30 11:33malsasuaStatusnew => closed
2015-01-30 11:33malsasuaResolutionopen => no change required
2016-01-13 15:20lorenzofidalgoSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=10607#r10607
2016-01-13 15:22lorenzofidalgoNote Added: 0083241
2016-01-13 15:22lorenzofidalgoStatusclosed => new
2016-01-13 15:22lorenzofidalgoResolutionno change required => open
2016-01-13 15:34lorenzofidalgoNote Added: 0083243
2016-01-13 15:38lorenzofidalgoNote Added: 0083244
2016-01-25 20:18dmitry_mezentsevNote Added: 0083600
2016-01-25 20:18dmitry_mezentsevSeveritycritical => minor

Notes
(0073940)
malsasua   
2015-01-30 11:33   
it is the behavior expected: the sales orders have been inserted in "Errors while importing data" window. From this window, the sales order can be imported when the problem of period closed has been fixed.
(0083241)
lorenzofidalgo   
2016-01-13 15:22   
No error is shown to warn the POS user that those receipt are not correctly processed. At least an error message should be shown to the user. Currently, success message is shown.
(0083243)
lorenzofidalgo   
2016-01-13 15:34   
With this Steps to Reproduce it is obvious that some changes are required:
0-Login POS as vallblanca.
1-Create a new receipt and complete it.
2-In dropdown menu, go to Receipts option.
3-Look for the just created receipt. Defect: That receipt does not appear. It has not correctly created and it has not been warned to the user. Instead of that, success message is shown.
(0083244)
lorenzofidalgo   
2016-01-13 15:38   
This has been reproduced in RR16Q1
(0083600)
dmitry_mezentsev   
2016-01-25 20:18   
Not Critical, http://wiki.openbravo.com/wiki/Bug_Reporting_Guidelines#How_to_Choose_the_Right_Severity [^]

I understand the use case but we need to think on the approach here because it applies not only to this specific case but to all scenarios when the order is saved in the errors. Now we do not notify POS user about them because it cannot influence / change them and so why to bother. But it is true that there are scenarios (like this search for the receipt) where it could help to know.

Still developing this notification is a topic because now with 2 steps order saving (at first in the raw "queue" table and then post-processing) the order for POS is saved when it is in the queue and it does not know about its processing result that happens later on.

So success message makes sense. It means the order was moved to the backoffice. But we could think on putting some on-demand request mechanism from POS to check error table that can be triggered by the user when the receipt is not found in the order table.