aboutsummaryrefslogtreecommitdiffstats
path: root/src/main/java/eu/siacs
diff options
context:
space:
mode:
authorDaniel Gultsch <daniel@gultsch.de>2016-06-19 11:08:17 +0200
committerDaniel Gultsch <daniel@gultsch.de>2016-06-19 11:08:17 +0200
commit28dc888159301d3d07a8770091fad81c77f4c9b8 (patch)
treead03c26cc0653294de83de243aa71932b22bb2b2 /src/main/java/eu/siacs
parentea1e4c773d99539da43e5c736df16f3947c23f72 (diff)
display toast on pgp error
Diffstat (limited to 'src/main/java/eu/siacs')
-rw-r--r--src/main/java/eu/siacs/conversations/crypto/PgpEngine.java2
-rw-r--r--src/main/java/eu/siacs/conversations/services/XmppConnectionService.java14
-rw-r--r--src/main/java/eu/siacs/conversations/ui/ConversationActivity.java8
3 files changed, 16 insertions, 8 deletions
diff --git a/src/main/java/eu/siacs/conversations/crypto/PgpEngine.java b/src/main/java/eu/siacs/conversations/crypto/PgpEngine.java
index fe3ac04b..bddc86ca 100644
--- a/src/main/java/eu/siacs/conversations/crypto/PgpEngine.java
+++ b/src/main/java/eu/siacs/conversations/crypto/PgpEngine.java
@@ -256,7 +256,7 @@ public class PgpEngine {
account);
return;
case OpenPgpApi.RESULT_CODE_ERROR:
- callback.error(R.string.openpgp_error, account);
+ callback.error(R.string.unable_to_connect_to_keychain, account);
}
}
});
diff --git a/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java b/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java
index 26dcb461..b85e6f42 100644
--- a/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java
+++ b/src/main/java/eu/siacs/conversations/services/XmppConnectionService.java
@@ -452,7 +452,12 @@ public class XmppConnectionService extends Service {
getFileBackend().copyFileToPrivateStorage(message, uri);
getFileBackend().updateFileParams(message);
if (message.getEncryption() == Message.ENCRYPTION_DECRYPTED) {
- getPgpEngine().encrypt(message, callback);
+ final PgpEngine pgpEngine = getPgpEngine();
+ if (pgpEngine != null) {
+ pgpEngine.encrypt(message, callback);
+ } else if (callback != null){
+ callback.error(R.string.unable_to_connect_to_keychain, null);
+ }
} else {
callback.success(message);
}
@@ -492,7 +497,12 @@ public class XmppConnectionService extends Service {
try {
getFileBackend().copyImageToPrivateStorage(message, uri);
if (conversation.getNextEncryption() == Message.ENCRYPTION_PGP) {
- getPgpEngine().encrypt(message, callback);
+ final PgpEngine pgpEngine = getPgpEngine();
+ if (pgpEngine != null) {
+ pgpEngine.encrypt(message, callback);
+ } else if (callback != null){
+ callback.error(R.string.unable_to_connect_to_keychain, null);
+ }
} else {
callback.success(message);
}
diff --git a/src/main/java/eu/siacs/conversations/ui/ConversationActivity.java b/src/main/java/eu/siacs/conversations/ui/ConversationActivity.java
index a9f4888f..e328084f 100644
--- a/src/main/java/eu/siacs/conversations/ui/ConversationActivity.java
+++ b/src/main/java/eu/siacs/conversations/ui/ConversationActivity.java
@@ -541,7 +541,7 @@ public class ConversationActivity extends XmppActivity
@Override
public void error(int error, Contact contact) {
- displayErrorDialog(error);
+ replaceToast(getString(error));
}
});
} else if (mode == Conversation.MODE_MULTI && conversation.getMucOptions().pgpKeysInUse()) {
@@ -1465,8 +1465,7 @@ public class ConversationActivity extends XmppActivity
@Override
public void error(int errorCode, Message message) {
- hidePrepareFileToast(prepareFileToast);
- displayErrorDialog(errorCode);
+ replaceToast(getString(errorCode));
}
@Override
@@ -1498,8 +1497,7 @@ public class ConversationActivity extends XmppActivity
@Override
public void error(int error, Message message) {
- hidePrepareFileToast(prepareFileToast);
- displayErrorDialog(error);
+ replaceToast(getString(error));
}
});
}