Disable ping manager on connection closed

Signed-off-by: Daniele Ricci <[email protected]>
kontalk · Mar 4, 2015 · 526db72 · 526db72
1 parent e4e63fb
commit 526db72
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/app/src/main/java/org/kontalk/service/XMPPConnectionHelper.java b/app/src/main/java/org/kontalk/service/XMPPConnectionHelper.java
@@ -357,8 +357,10 @@ public void shutdown() throws NotConnectedException {
 
 
     public interface ConnectionHelperListener extends ConnectionListener {
+        /** Connection has been created. */
         public void created(XMPPConnection connection);
 
+        /** Connection was aborted and will never be tried again. */
         public void aborted(Exception e);
 
         public void authenticationFailed();

diff --git a/app/src/main/java/org/kontalk/service/msgcenter/MessageCenterService.java b/app/src/main/java/org/kontalk/service/msgcenter/MessageCenterService.java
@@ -572,6 +572,8 @@ private synchronized void quit(boolean restarting) {
 
         // disconnect from server (if any)
         if (mConnection != null) {
+            // disable ping manager
+            ServerPingWithAlarmManager.getInstanceFor(mConnection).setEnabled(false);
             // this is because of NetworkOnMainThreadException
             new DisconnectThread(mConnection).start();
             mConnection = null;