IDEMPIERE-4154 ServerPush Adjustments. Fix logging error. commitResponse should be ok as long as resource is available.
This commit is contained in:
parent
174ff192a4
commit
ea1f86837d
|
@ -122,7 +122,7 @@ public class AtmosphereServerPush implements ServerPush {
|
||||||
|
|
||||||
private boolean commitResponse() throws IOException {
|
private boolean commitResponse() throws IOException {
|
||||||
AtmosphereResource resource = this.resource.getAndSet(null);
|
AtmosphereResource resource = this.resource.getAndSet(null);
|
||||||
if (resource != null && resource.isSuspended()) {
|
if (resource != null) {
|
||||||
resource.resume();
|
resource.resume();
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
|
@ -53,7 +53,8 @@ public class ZkAtmosphereHandler implements AtmosphereHandler {
|
||||||
if (session.getWebApp() instanceof WebAppCtrl) {
|
if (session.getWebApp() instanceof WebAppCtrl) {
|
||||||
WebAppCtrl webAppCtrl = (WebAppCtrl) session.getWebApp();
|
WebAppCtrl webAppCtrl = (WebAppCtrl) session.getWebApp();
|
||||||
Desktop desktop = webAppCtrl.getDesktopCache(session).getDesktopIfAny(dtid);
|
Desktop desktop = webAppCtrl.getDesktopCache(session).getDesktopIfAny(dtid);
|
||||||
log.warn("Could not find desktop: " + dtid);
|
if (desktop == null)
|
||||||
|
log.warn("Could not find desktop: " + dtid);
|
||||||
return new Either<String, Desktop>("Could not find desktop", desktop);
|
return new Either<String, Desktop>("Could not find desktop", desktop);
|
||||||
}
|
}
|
||||||
return new Either<String, Desktop>("Webapp does not implement WebAppCtrl", null);
|
return new Either<String, Desktop>("Webapp does not implement WebAppCtrl", null);
|
||||||
|
|
Loading…
Reference in New Issue