show escaped jids when it comes to account selection

Daniel Gultsch created 5 years ago

Change summary

src/main/java/eu/siacs/conversations/ui/BlocklistActivity.java         | 2 
src/main/java/eu/siacs/conversations/ui/ChannelDiscoveryActivity.java  | 2 
src/main/java/eu/siacs/conversations/ui/ChooseContactActivity.java     | 4 
src/main/java/eu/siacs/conversations/ui/EnterJidDialog.java            | 4 
src/main/java/eu/siacs/conversations/ui/StartConversationActivity.java | 4 
src/main/java/eu/siacs/conversations/utils/AccountUtils.java           | 4 
6 files changed, 10 insertions(+), 10 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/ui/BlocklistActivity.java

@@ -79,7 +79,7 @@ public class BlocklistActivity extends AbstractSearchableListItemActivity implem
 				getString(R.string.block_jabber_id),
 				getString(R.string.block),
 				null,
-				account.getJid().asBareJid().toString(),
+				account.getJid().asBareJid().toEscapedString(),
 				true,
 				false
 		);

src/main/java/eu/siacs/conversations/ui/ChannelDiscoveryActivity.java

@@ -263,7 +263,7 @@ public class ChannelDiscoveryActivity extends XmppActivity implements MenuItem.O
     }
 
     public void joinChannelSearchResult(String selectedAccount, Room result) {
-        final Jid jid = Config.DOMAIN_LOCK == null ? Jid.of(selectedAccount) : Jid.of(selectedAccount, Config.DOMAIN_LOCK, null);
+        final Jid jid = Config.DOMAIN_LOCK == null ? Jid.ofEscaped(selectedAccount) : Jid.ofEscaped(selectedAccount, Config.DOMAIN_LOCK, null);
         final boolean syncAutoJoin = getBooleanPreference("autojoin", R.bool.autojoin);
         final Account account = xmppConnectionService.findAccountByJid(jid);
         final Conversation conversation = xmppConnectionService.findOrCreateConversation(account, result.getRoom(), true, true, true);

src/main/java/eu/siacs/conversations/ui/ChooseContactActivity.java

@@ -361,9 +361,9 @@ public class ChooseContactActivity extends AbstractSearchableListItemActivity im
         for (Account account : xmppConnectionService.getAccounts()) {
             if (account.getStatus() != Account.State.DISABLED) {
                 if (Config.DOMAIN_LOCK != null) {
-                    this.mActivatedAccounts.add(account.getJid().getLocal());
+                    this.mActivatedAccounts.add(account.getJid().getEscapedLocal());
                 } else {
-                    this.mActivatedAccounts.add(account.getJid().asBareJid().toString());
+                    this.mActivatedAccounts.add(account.getJid().asBareJid().toEscapedString());
                 }
             }
         }

src/main/java/eu/siacs/conversations/ui/EnterJidDialog.java

@@ -146,9 +146,9 @@ public class EnterJidDialog extends DialogFragment implements OnBackendConnected
 		}
 		try {
 			if (Config.DOMAIN_LOCK != null) {
-				accountJid = Jid.of((String) binding.account.getSelectedItem(), Config.DOMAIN_LOCK, null);
+				accountJid = Jid.ofEscaped((String) binding.account.getSelectedItem(), Config.DOMAIN_LOCK, null);
 			} else {
-				accountJid = Jid.of((String) binding.account.getSelectedItem());
+				accountJid = Jid.ofEscaped((String) binding.account.getSelectedItem());
 			}
 		} catch (final IllegalArgumentException e) {
 			return;

src/main/java/eu/siacs/conversations/ui/StartConversationActivity.java

@@ -577,9 +577,9 @@ public class StartConversationActivity extends XmppActivity implements XmppConne
 			Jid jid;
 			try {
 				if (Config.DOMAIN_LOCK != null) {
-					jid = Jid.of((String) spinner.getSelectedItem(), Config.DOMAIN_LOCK, null);
+					jid = Jid.ofEscaped((String) spinner.getSelectedItem(), Config.DOMAIN_LOCK, null);
 				} else {
-					jid = Jid.of((String) spinner.getSelectedItem());
+					jid = Jid.ofEscaped((String) spinner.getSelectedItem());
 				}
 			} catch (final IllegalArgumentException e) {
 				return null;

src/main/java/eu/siacs/conversations/utils/AccountUtils.java

@@ -39,9 +39,9 @@ public class AccountUtils {
         for (Account account : service.getAccounts()) {
             if (account.getStatus() != Account.State.DISABLED) {
                 if (Config.DOMAIN_LOCK != null) {
-                    accounts.add(account.getJid().getLocal());
+                    accounts.add(account.getJid().toEscapedString());
                 } else {
-                    accounts.add(account.getJid().asBareJid().toString());
+                    accounts.add(account.getJid().asBareJid().toEscapedString());
                 }
             }
         }