check if account is online before doing smp

1 files changed, 24 insertions, 11 deletions

diff --git a/src/main/java/eu/siacs/conversations/ui/VerifyOTRActivity.java b/src/main/java/eu/siacs/conversations/ui/VerifyOTRActivity.java
index 2a425d127..31884bd27 100644
--- a/src/main/java/eu/siacs/conversations/ui/VerifyOTRActivity.java
+++ b/src/main/java/eu/siacs/conversations/ui/VerifyOTRActivity.java
@@ -50,29 +50,33 @@ public class VerifyOTRActivity extends XmppActivity implements XmppConnectionSer
 	private View.OnClickListener mCreateSharedSecretListener = new View.OnClickListener() {
 		@Override
 		public void onClick(final View view) {
-			final String question = mSharedSecretHint.getText().toString();
-			final String secret = mSharedSecretSecret.getText().toString();
-			if (!initSmp(question,secret)) {
-				Toast.makeText(getApplicationContext(),"smp failed",Toast.LENGTH_SHORT).show();
+			if (isAccountOnline()) {
+				final String question = mSharedSecretHint.getText().toString();
+				final String secret = mSharedSecretSecret.getText().toString();
+				initSmp(question, secret);
+				updateView();
 			}
-			updateView();
 		}
 	};
 	private View.OnClickListener mCancelSharedSecretListener = new View.OnClickListener() {
 		@Override
 		public void onClick(View view) {
-			abortSmp();
-			updateView();
+			if (isAccountOnline()) {
+				abortSmp();
+				updateView();
+			}
 		}
 	};
 	private View.OnClickListener mRespondSharedSecretListener = new View.OnClickListener() {
 
 		@Override
 		public void onClick(View view) {
-			final String question = mSharedSecretHint.getText().toString();
-			final String secret = mSharedSecretSecret.getText().toString();
-			respondSmp(question,secret);
-			updateView();
+			if (isAccountOnline()) {
+				final String question = mSharedSecretHint.getText().toString();
+				final String secret = mSharedSecretSecret.getText().toString();
+				respondSmp(question, secret);
+				updateView();
+			}
 		}
 	};
 	private View.OnClickListener mRetrySharedSecretListener = new View.OnClickListener() {
@@ -138,6 +142,15 @@ public class VerifyOTRActivity extends XmppActivity implements XmppConnectionSer
 		}
 	}
 
+	protected boolean isAccountOnline() {
+		if (this.mAccount.getStatus() != Account.STATUS_ONLINE) {
+			Toast.makeText(this,R.string.not_connected_try_again,Toast.LENGTH_SHORT).show();
+			return false;
+		} else {
+			return true;
+		}
+	}
+
 	protected boolean handleIntent(Intent intent) {
 		if (intent.getAction().equals(ACTION_VERIFY_CONTACT)) {
 			try {