Openbravo Issue Tracking System - Retail Modules
View Issue Details
0036698Retail ModulesStoreServerpublic2017-08-23 09:222017-10-31 17:12
lorenzofidalgo 
migueldejuana 
immediatemajoralways
closedfixed 
5
main 
RR18Q1 
guilleaer
Packaging and release
main
No
0036698: [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined"
With a Store Server + High Vol. configuration, searching for Layaways is not possible, a wrong record is shown and it is not possible opening that record. In a RR17Q3 context without Store Server or High Vol. this issue is not reproducible. It seems to be related with Store Server, High Vol or configuration.
0-With a Store Server + High Vol. context, login in a POS Store as vallblanca.
1-Click dropdown menu, and select Layaways option.
2-Click search button using no filters and realise only a record is shown and that record is "Undefined".
No tags attached.
depends on backport 0036773RR17Q3.1 closed migueldejuana [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined" 
depends on backport 0036774RR17Q2.4 closed migueldejuana [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined" 
depends on backport 0037196RR17Q4 closed migueldejuana [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined" 
related to defect 0036263 closed migueldejuana The queries related with a incremental refresh should be interrupted when the timeout for the request is reached 
png UndefinedLayaway.png (113,485) 2017-08-23 09:22
https://issues.openbravo.com/file_download.php?file_id=10995&type=bug
png

png WrongLayawayFlow.png (105,751) 2017-08-23 09:26
https://issues.openbravo.com/file_download.php?file_id=10996&type=bug
png
Issue History
2017-08-23 09:22lorenzofidalgoNew Issue
2017-08-23 09:22lorenzofidalgoAssigned To => StoreServer
2017-08-23 09:22lorenzofidalgoFile Added: UndefinedLayaway.png
2017-08-23 09:22lorenzofidalgoRegression level => Packaging and release
2017-08-23 09:22lorenzofidalgoRegression introduced in release => main
2017-08-23 09:22lorenzofidalgoTriggers an Emergency Pack => No
2017-08-23 09:26lorenzofidalgoFile Added: WrongLayawayFlow.png
2017-08-23 11:06AugustoMauchRelationship addedrelated to 0036263
2017-08-23 11:10AugustoMauchAssigned ToStoreServer => Retail
2017-08-23 11:12AugustoMauchNote Added: 0098605
2017-08-29 15:25marvintmNote Added: 0098705
2017-09-04 11:06marvintmStatusnew => scheduled
2017-09-04 11:06marvintmAssigned ToRetail => marvintm
2017-10-04 16:32hgbotCheckin
2017-10-04 16:32hgbotNote Added: 0099768
2017-10-04 16:32hgbotStatusscheduled => resolved
2017-10-04 16:32hgbotResolutionopen => fixed
2017-10-04 16:32hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d00d2e25c258c06f94699ae44987fcd4370a8628 [^]
2017-10-04 16:32migueldejuanaAssigned Tomarvintm => migueldejuana
2017-10-04 16:37migueldejuanaNote Added: 0099769
2017-10-31 12:05guilleaerNote Added: 0100184
2017-10-31 12:05guilleaerStatusresolved => new
2017-10-31 12:05guilleaerResolutionfixed => open
2017-10-31 12:11guilleaerStatusnew => scheduled
2017-10-31 16:13hgbotCheckin
2017-10-31 16:13hgbotNote Added: 0100195
2017-10-31 16:13hgbotStatusscheduled => resolved
2017-10-31 16:13hgbotResolutionopen => fixed
2017-10-31 16:13hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d00d2e25c258c06f94699ae44987fcd4370a8628 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/07f9cb6f7e710725ff5906cccea9aeeb8c03b8e7 [^]
2017-10-31 16:19hgbotCheckin
2017-10-31 16:19hgbotNote Added: 0100196
2017-10-31 16:19hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/07f9cb6f7e710725ff5906cccea9aeeb8c03b8e7 [^] => http://code.openbravo.com/retail/backports/3.0RR17Q3.1/org.openbravo.retail.posterminal/rev/f8ceac62310a8f1dc9e23cbba0457986123d67cf [^]
2017-10-31 16:21migueldejuanaNote Deleted: 0100196
2017-10-31 17:12guilleaerReview Assigned To => guilleaer
2017-10-31 17:12guilleaerStatusresolved => closed
2017-10-31 17:12guilleaerFixed in Version => RR18Q1

Notes
(0098605)
AugustoMauch   
2017-08-23 11:12   
The problem seems to be that the query is cancelled in the backend because it times out, see https://issues.openbravo.com/view.php?id=36263 [^]

I executed the query without a timeout and it took 0000176:0000040 seconds. The query timeout in Openbravo is set to the timeout of the request, which in this case is 15 seconds.
(0098705)
marvintm   
2017-08-29 15:25   
The cause of the problem is a combination of the retry mechanism, the change in 36263, and the fact that the enyo ajax request component can trigger the failure callback and then the success callback if the request returns a response after reaching the timeout.

The solution, as we did in other places, is to ensure that after the request has failed, it remains blocked and the success callback cannot be triggered.
(0099768)
hgbot   
2017-10-04 16:32   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: d00d2e25c258c06f94699ae44987fcd4370a8628
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Wed Oct 04 13:24:02 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/d00d2e25c258c06f94699ae44987fcd4370a8628 [^]

Fixed issue 0036698: [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined"

- Handle exceptions when loading receipts from backend

---
M web/org.openbravo.retail.posterminal/js/components/modalmultiorders.js
M web/org.openbravo.retail.posterminal/js/components/modalpaidreceipts.js
---
(0099769)
migueldejuana   
2017-10-04 16:37   
It is happening in pi with high volumes. When PaidReceiptsHeader servlet returns an error or gives timeout we do not handle the error and it creates a row.

- Login in backend and enable remote preferences
- Add an Exception or a breakpoint in PaidReceiptsHeader.java
- Login in Web POS and select in the menu "Layaways" option
- Search without any filter and you get the error
(0100184)
guilleaer   
2017-10-31 12:05   
In case of error without exception (Should not happen but its better to be ready) web POS should show a generic error.
(0100195)
hgbot   
2017-10-31 16:13   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 07f9cb6f7e710725ff5906cccea9aeeb8c03b8e7
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Oct 31 16:12:46 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/07f9cb6f7e710725ff5906cccea9aeeb8c03b8e7 [^]

Fixed issue 0036698: [RR17Q3][StoreServer][HighVol] Searching for Layaways only one record is shown and it is "Undefined"

- If it fails we have to show the message always, not just when data.exception exists

---
M web/org.openbravo.retail.posterminal/js/components/modalmultiorders.js
M web/org.openbravo.retail.posterminal/js/components/modalpaidreceipts.js
---