Merge pull request #179 from RocketChat/fix_okhttp_error_notification

fix issue that sendMessage error not notified properly

Merge pull request #179 from RocketChat/fix_okhttp_error_notification
fix issue that sendMessage error not notified properly
d1bcc7c5 · Yusuke Iwaki · GitHub · d13702d9 · bcf0fe2e · d1bcc7c5
Commit d1bcc7c5 authored Jan 25, 2017 by Yusuke Iwaki Committed by GitHub Jan 25, 2017
Show whitespace changes
Inline Side-by-side

Showing with 8 additions and 0 deletions

DDPClientImpl.java .../src/main/java/chat/rocket/android_ddp/DDPClientImpl.java +8 -0

No files found.
--- a/android-ddp/src/main/java/chat/rocket/android_ddp/DDPClientImpl.java
+++ b/android-ddp/src/main/java/chat/rocket/android_ddp/DDPClientImpl.java
@@ -134,6 +134,8 @@ public class DDPClientImpl {
              }));

      addErrorCallback(subscriptions, task);
+    } else {
+      task.trySetError(new DDPClientCallback.Closed(client));
    }
  }

@@ -174,6 +176,8 @@ public class DDPClientImpl {
              }));

      addErrorCallback(subscriptions, task);
+    } else {
+      task.trySetError(new DDPClientCallback.Closed(client));
    }
  }

@@ -202,6 +206,8 @@ public class DDPClientImpl {
              }));

      addErrorCallback(subscriptions, task);
+    } else {
+      task.trySetError(new DDPClientCallback.Closed(client));
    }
  }

@@ -241,6 +247,8 @@ public class DDPClientImpl {
              }));

      addErrorCallback(subscriptions, task);
+    } else {
+      task.trySetError(new DDPClientCallback.Closed(client));
    }
  }