Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036057Retail ModulesWeb POSpublic2017-05-22 08:382017-06-27 09:30
Reporterguillermogil 
Assigned Tomigueldejuana 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q3 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-06-14
Regression introduced in releaseRR16Q3
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/a17e34ee0e87 [^]
Triggers an Emergency PackNo
Summary0036057: Timeout parameter of OB.DS.Process is not working
DescriptionTimeout parameter of OB.DS.Process is not working, it does 4 more executions and then the timeout raises.
This is generating timeout to not raise and also, for third party implementation this can rely on wrong data.
Steps To Reproduce- Login on WebPOS (on a development environment)
- Put a breakpoint in PaidReceiptsHeaders.java
- Open Receipts popup
- Do a Search

You will see that 5 calls are executed and timeout does not raise in the defined time (15000 by default)
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0035965RR17Q3 closed migueldejuana NoRetries is not working as expected && Datasource not recognices NoRetries parameter 
depends on backport 0036168RR17Q2.1 closed migueldejuana Timeout parameter of OB.DS.Process is not working 
depends on backport 0036169RR17Q1.2 closed migueldejuana Timeout parameter of OB.DS.Process is not working 
Attached Filesdiff timeout16Q4.diff (6,025) 2017-06-02 12:06
https://issues.openbravo.com/file_download.php?file_id=10822&type=bug
Issue History
Date ModifiedUsernameFieldChange
2017-05-22 08:38guillermogilNew Issue
2017-05-22 08:38guillermogilAssigned To => Retail
2017-05-22 08:38guillermogilOBNetwork customer => Yes
2017-05-22 08:38guillermogilResolution time => 1494540000
2017-05-22 08:38guillermogilTriggers an Emergency Pack => No
2017-05-22 08:41marvintmResolution time1494540000 => 1497218400
2017-06-01 11:02migueldejuanaAssigned ToRetail => migueldejuana
2017-06-01 12:22migueldejuanaRelationship addedrelated to 0035965
2017-06-02 09:35hgbotCheckin
2017-06-02 09:35hgbotNote Added: 0097093
2017-06-02 09:35hgbotStatusnew => resolved
2017-06-02 09:35hgbotResolutionopen => fixed
2017-06-02 09:35hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/854b5d8dde16593b15673c59e65e13782fb5ccd7 [^]
2017-06-02 12:06migueldejuanaFile Added: timeout16Q4.diff
2017-06-02 12:47marvintmReview Assigned To => marvintm
2017-06-02 12:47marvintmStatusresolved => closed
2017-06-02 12:47marvintmFixed in Version => RR17Q3
2017-06-02 12:51marvintmStatusclosed => new
2017-06-02 12:51marvintmResolutionfixed => open
2017-06-02 12:51marvintmFixed in VersionRR17Q3 =>
2017-06-02 12:52marvintmStatusnew => scheduled
2017-06-02 12:52marvintmStatusscheduled => resolved
2017-06-02 12:52marvintmFixed in Version => RR17Q3
2017-06-02 12:52marvintmResolutionopen => fixed
2017-06-02 12:52marvintmStatusresolved => closed
2017-06-08 19:42hgbotCheckin
2017-06-08 19:42hgbotNote Added: 0097255
2017-06-26 11:13migueldejuanaRegression level => Production - Confirmed Stable
2017-06-26 11:13migueldejuanaRegression date => 2016-06-14
2017-06-26 11:13migueldejuanaRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/a17e34ee0e87 [^]
2017-06-27 09:30marvintmRegression introduced in release => RR16Q3

Notes
(0097093)
hgbot   
2017-06-02 09:35   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 854b5d8dde16593b15673c59e65e13782fb5ccd7
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Jun 01 11:10:08 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/854b5d8dde16593b15673c59e65e13782fb5ccd7 [^]

Fixed issue 0036057: Timeout parameter of OB.DS.Process is not working

- Change noRetries:true with maxNumOfRequestRetries:0
- Default and minimum value for maxNumOfRequestRetries is 0
- if maxNumOfRequestRetries is defined in OB.DS.Process overrides preference and default value
- We can define attributes in extraParams in OB.DS.Process and they will set in the ajaxRequest

---
M web/org.openbravo.mobile.core/source/data/ob-datasource.js
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0097255)
hgbot   
2017-06-08 19:42   
Repository: erp/pmods/org.openbravo.warehouse.advancedwarehouseoperations
Changeset: 1479ef0c7bf2105c6abb5e95cf802f12bfcef4ca
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Thu Jun 08 19:38:47 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.warehouse.advancedwarehouseoperations/rev/1479ef0c7bf2105c6abb5e95cf802f12bfcef4ca [^]

No Retries is not longer needed. Changed behavior in mobile core. Issue 36057

---
M web/org.openbravo.warehouse.advancedwarehouseoperations/source/models/centralbroker.js
---