fixed omemo shown as unavailable in 1:1 chats

Daniel Gultsch created

Change summary

src/main/java/eu/siacs/conversations/crypto/axolotl/AxolotlService.java | 3 
src/main/java/eu/siacs/conversations/persistance/FileBackend.java       | 4 
src/main/java/eu/siacs/conversations/ui/ConversationActivity.java       | 5 
src/main/java/eu/siacs/conversations/ui/adapter/MessageAdapter.java     | 3 
src/main/res/values/strings.xml                                         | 2 
5 files changed, 12 insertions(+), 5 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/crypto/axolotl/AxolotlService.java 🔗

@@ -660,7 +660,8 @@ public class AxolotlService implements OnAdvancedStreamFeaturesLoaded {
 	}
 
 	public Pair<AxolotlCapability,Jid> isConversationAxolotlCapableDetailed(Conversation conversation) {
-		if (conversation.getMucOptions().membersOnly() && conversation.getMucOptions().nonanonymous()) {
+		if (conversation.getMode() == Conversation.MODE_SINGLE
+				|| (conversation.getMucOptions().membersOnly() && conversation.getMucOptions().nonanonymous())) {
 			final List<Jid> jids = getCryptoTargets(conversation);
 			for(Jid jid : jids) {
 				if (!hasAny(jid) && (!deviceIds.containsKey(jid) || deviceIds.get(jid).isEmpty())) {

src/main/java/eu/siacs/conversations/persistance/FileBackend.java 🔗

@@ -59,6 +59,8 @@ import eu.siacs.conversations.xmpp.pep.Avatar;
 public class FileBackend {
 	private static final SimpleDateFormat IMAGE_DATE_FORMAT = new SimpleDateFormat("yyyyMMdd_HHmmss", Locale.US);
 
+	public static final String CONVERSATIONS_FILE_PROVIDER = "eu.siacs.conversations.files";
+
 	private XmppConnectionService mXmppConnectionService;
 
 	public FileBackend(XmppConnectionService service) {
@@ -439,7 +441,7 @@ public class FileBackend {
 		File file = new File(getTakePhotoPath()+"IMG_" + this.IMAGE_DATE_FORMAT.format(new Date()) + ".jpg");
 		file.getParentFile().mkdirs();
 		if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
-			return FileProvider.getUriForFile(mXmppConnectionService, "eu.siacs.conversations.files", file);
+			return FileProvider.getUriForFile(mXmppConnectionService, CONVERSATIONS_FILE_PROVIDER, file);
 		} else {
 			return Uri.fromFile(file);
 		}

src/main/java/eu/siacs/conversations/ui/ConversationActivity.java 🔗

@@ -451,14 +451,15 @@ public class ConversationActivity extends XmppActivity
 
 	private boolean quickOmemoDebugger(Conversation c) {
 		if (c != null) {
+			boolean single = c.getMode() == Conversation.MODE_SINGLE;
 			AxolotlService axolotlService = c.getAccount().getAxolotlService();
 			Pair<AxolotlService.AxolotlCapability,Jid> capabilityJidPair = axolotlService.isConversationAxolotlCapableDetailed(c);
 			switch (capabilityJidPair.first) {
 				case MISSING_PRESENCE:
-					Toast.makeText(ConversationActivity.this,getString(R.string.missing_presence_subscription_with_x,capabilityJidPair.second.toBareJid().toString()),Toast.LENGTH_SHORT).show();
+					Toast.makeText(ConversationActivity.this,single ? getString(R.string.missing_presence_subscription) : getString(R.string.missing_presence_subscription_with_x,capabilityJidPair.second.toBareJid().toString()),Toast.LENGTH_SHORT).show();
 					return true;
 				case MISSING_KEYS:
-					Toast.makeText(ConversationActivity.this,getString(R.string.missing_keys_from_x,capabilityJidPair.second.toBareJid().toString()),Toast.LENGTH_SHORT).show();
+					Toast.makeText(ConversationActivity.this,single ? getString(R.string.missing_omemo_keys) : getString(R.string.missing_keys_from_x,capabilityJidPair.second.toBareJid().toString()),Toast.LENGTH_SHORT).show();
 					return true;
 				case WRONG_CONFIGURATION:
 					Toast.makeText(ConversationActivity.this,R.string.wrong_conference_configuration, Toast.LENGTH_SHORT).show();

src/main/java/eu/siacs/conversations/ui/adapter/MessageAdapter.java 🔗

@@ -51,6 +51,7 @@ import eu.siacs.conversations.entities.DownloadableFile;
 import eu.siacs.conversations.entities.Message;
 import eu.siacs.conversations.entities.Message.FileParams;
 import eu.siacs.conversations.entities.Transferable;
+import eu.siacs.conversations.persistance.FileBackend;
 import eu.siacs.conversations.ui.ConversationActivity;
 import eu.siacs.conversations.ui.widget.ClickableMovementMethod;
 import eu.siacs.conversations.utils.CryptoHelper;
@@ -698,7 +699,7 @@ public class MessageAdapter extends ArrayAdapter<Message> {
 		Uri uri;
 		if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
 			try {
-				uri = FileProvider.getUriForFile(activity, "eu.siacs.conversations.files", file);
+				uri = FileProvider.getUriForFile(activity, FileBackend.CONVERSATIONS_FILE_PROVIDER, file);
 			} catch (IllegalArgumentException e) {
 				Toast.makeText(activity,activity.getString(R.string.no_permission_to_access_x,file.getAbsolutePath()), Toast.LENGTH_SHORT).show();
 				return;

src/main/res/values/strings.xml 🔗

@@ -284,6 +284,7 @@
 	<string name="conference_requires_password">Conference requires password</string>
 	<string name="enter_password">Enter password</string>
 	<string name="missing_presence_updates">Missing presence updates from contact</string>
+	<string name="missing_presence_subscription">Missing presence subscription</string>
 	<string name="request_presence_updates">Please request presence updates from your contact first.\n\n<small>This will be used to determine what client(s) your contact is using.</small></string>
 	<string name="request_now">Request now</string>
 	<string name="delete_fingerprint">Delete Fingerprint</string>
@@ -684,6 +685,7 @@
 	<string name="unable_to_update_account">Unable to update account</string>
 	<string name="missing_presence_subscription_with_x">Missing presence subscription with %s.</string>
 	<string name="missing_keys_from_x">Missing OMEMO keys from %s.</string>
+	<string name="missing_omemo_keys">Missing OMEMO keys</string>
 	<string name="wrong_conference_configuration">This is not a private, non-anonymous conference.</string>
 	<string name="this_conference_has_no_members">There are no members in this conference.</string>
 	<string name="report_jid_as_spammer">Report this JID as sending unwanted messages.</string>