[b] Fix onSuccessCallback.onSuccess() not being called when `.withP…

…rocessVisibility(false)` yesidlazaro#28
luongvo · Mar 10, 2017 · b7bf9d9 · b7bf9d9
1 parent 87205de
commit b7bf9d9
Showing 1 changed file with 16 additions and 15 deletions.
diff --git a/...groundlibrary/src/main/java/com/creativityapps/gmailbackgroundlibrary/BackgroundMail.java b/...groundlibrary/src/main/java/com/creativityapps/gmailbackgroundlibrary/BackgroundMail.java
@@ -298,22 +298,23 @@ protected Boolean doInBackground(String... arg0) {
         @Override
         protected void onPostExecute(Boolean result) {
             super.onPostExecute(result);
-            if (processVisibility) {
+            if (progressDialog != null && progressDialog.isShowing()) {
                 progressDialog.dismiss();
-                if (result) {
-                    if (!TextUtils.isEmpty(sendingMessageSuccess)) {
-                        Toast.makeText(mContext, sendingMessageSuccess, Toast.LENGTH_SHORT).show();
-                    }
-                    if (onSuccessCallback != null) {
-                        onSuccessCallback.onSuccess();
-                    }
-                }else {
-                    if (!TextUtils.isEmpty(sendingMessageError)) {
-                        Toast.makeText(mContext, sendingMessageError, Toast.LENGTH_SHORT).show();
-                    }
-                    if (onFailCallback != null) {
-                        onFailCallback.onFail();
-                    }
+            }
+
+            if (result) {
+                if (processVisibility && !TextUtils.isEmpty(sendingMessageSuccess)) {
+                    Toast.makeText(mContext, sendingMessageSuccess, Toast.LENGTH_SHORT).show();
+                }
+                if (onSuccessCallback != null) {
+                    onSuccessCallback.onSuccess();
+                }
+            } else {
+                if (processVisibility && !TextUtils.isEmpty(sendingMessageError)) {
+                    Toast.makeText(mContext, sendingMessageError, Toast.LENGTH_SHORT).show();
+                }
+                if (onFailCallback != null) {
+                    onFailCallback.onFail();
                 }
             }
         }