Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0036491 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] StoreServer | minor | have not tried | 2017-07-13 16:56 | 2017-10-24 15:06 | |||
Reporter | AugustoMauch | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | 3b09106d0238 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | AugustoMauch | |||||||
OBNetwork customer | No | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0036491: In ServerStateBackground, if a store server is not pinged by central server, its status is not set to OFFLINE | |||||||
Description | The ServerStateBackground.pingShouldBeDoneToServer(server) determines if a store server should be pinged to check/update its status. It checks the PingServerChecker, and if any returns false, the store server is not pinged and its status is not updated in the central server. Currently the only implementation of PingServerChecker that we have is RegisteredServerPingChecker, that returns false if the store server has not been deployed (it checks that is has been registered in the central server's SymmetricDS engine). In this case it would make sense to set the server status to OFFLINE to all those servers that are not registered, instead of just leaving its status as it is. | |||||||
Steps To Reproduce | - | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0099167) hgbot (developer) 2017-09-20 09:49 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 3b09106d0238d59be5effb111f31082a856a0a06 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Wed Sep 20 09:29:31 2017 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3b09106d0238d59be5effb111f31082a856a0a06 [^] Fixes issue 36491: In ServerStateBackground, if a store server is not pinged by central server, its status is not set to OFFLINE Set state to offline if the server does not need to be pinged. --- M src/org/openbravo/mobile/core/servercontroller/ServerStateBackground.java --- |
(0099542) AugustoMauch (administrator) 2017-09-25 13:13 |
Code reviewed and verified |
(0100028) hgbot (developer) 2017-10-24 15:06 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: f5e4b085b6d0371b7187c69d2619a5a341e51b50 Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Mon Sep 25 16:35:41 2017 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/f5e4b085b6d0371b7187c69d2619a5a341e51b50 [^] Related with issue 36491: Enum value is passed instead of its name updateServerStatus expects ON, OFF, etc, not ONLINE or OFFLINE. --- M src/org/openbravo/mobile/core/servercontroller/ServerStateBackground.java --- |
![]() |
|||
Date Modified | Username | Field | Change |
2017-07-13 16:56 | AugustoMauch | New Issue | |
2017-07-13 16:56 | AugustoMauch | Assigned To | => StoreServer |
2017-07-13 16:56 | AugustoMauch | OBNetwork customer | => No |
2017-07-13 16:56 | AugustoMauch | Triggers an Emergency Pack | => No |
2017-08-30 09:25 | mtaal | Assigned To | StoreServer => mtaal |
2017-09-20 09:48 | mtaal | Review Assigned To | => AugustoMauch |
2017-09-20 09:49 | hgbot | Checkin | |
2017-09-20 09:49 | hgbot | Note Added: 0099167 | |
2017-09-20 09:49 | hgbot | Status | new => resolved |
2017-09-20 09:49 | hgbot | Resolution | open => fixed |
2017-09-20 09:49 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3b09106d0238d59be5effb111f31082a856a0a06 [^] |
2017-09-25 13:13 | AugustoMauch | Note Added: 0099542 | |
2017-09-25 13:13 | AugustoMauch | Status | resolved => closed |
2017-10-24 15:06 | hgbot | Checkin | |
2017-10-24 15:06 | hgbot | Note Added: 0100028 |
Copyright © 2000 - 2009 MantisBT Group |