From 5fbc6d7282b2b0d8863b2e94be48929f0945efc5 Mon Sep 17 00:00:00 2001 From: "B. Petersen" Date: Thu, 20 Sep 2018 11:51:59 +0200 Subject: [PATCH] remove some 'registered'-checks --- .../securesms/ConversationActivity.java | 66 +------------------ 1 file changed, 3 insertions(+), 63 deletions(-) diff --git a/src/org/thoughtcrime/securesms/ConversationActivity.java b/src/org/thoughtcrime/securesms/ConversationActivity.java index 2c62ff414..ba5dda7e3 100644 --- a/src/org/thoughtcrime/securesms/ConversationActivity.java +++ b/src/org/thoughtcrime/securesms/ConversationActivity.java @@ -346,7 +346,6 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity quickAttachmentDrawer.onResume(); initializeEnabledCheck(); - initializeMmsEnabledCheck(); composeText.setTransport(sendButton.getSelectedTransport()); titleView.setTitle(glideRequests, dcChat); @@ -931,7 +930,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity } private void initializeEnabledCheck() { - boolean enabled = !(isPushGroupConversation() && !isActiveGroup()); + boolean enabled = true; inputPanel.setEnabled(enabled); sendButton.setEnabled(enabled); attachButton.setEnabled(enabled); @@ -1010,53 +1009,8 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity handleSecurityChange(currentSecureText || isPushGroupConversation(), currentIsDefaultSms); - new AsyncTask() { - @Override - protected boolean[] doInBackground(Recipient... params) { - Context context = ConversationActivity.this; - Recipient recipient = params[0]; - Log.w(TAG, "Resolving registered state..."); - RegisteredState registeredState; - - if (recipient.isPushGroupRecipient()) { - Log.w(TAG, "Push group recipient..."); - registeredState = RegisteredState.REGISTERED; - } else if (recipient.isResolving()) { - Log.w(TAG, "Talking to DB directly."); - registeredState = DatabaseFactory.getRecipientDatabase(ConversationActivity.this).isRegistered(recipient.getAddress()); - } else { - Log.w(TAG, "Checking through resolved recipient"); - registeredState = recipient.resolve().getRegistered(); - } - - Log.w(TAG, "Resolved registered state: " + registeredState); - boolean signalEnabled = TextSecurePreferences.isPushRegistered(context); - - if (registeredState == RegisteredState.UNKNOWN) { - try { - Log.w(TAG, "Refreshing directory for user: " + recipient.getAddress().serialize()); - registeredState = DirectoryHelper.refreshDirectoryFor(context, recipient); - } catch (IOException e) { - Log.w(TAG, e); - } - } - - Log.w(TAG, "Returning registered state..."); - return new boolean[] {registeredState == RegisteredState.REGISTERED && signalEnabled, - false}; - } - - @Override - protected void onPostExecute(boolean[] result) { - if (result[0] != currentSecureText || result[1] != currentIsDefaultSms) { - Log.w(TAG, "onPostExecute() handleSecurityChange: " + result[0] + " , " + result[1]); - handleSecurityChange(result[0], result[1]); - } - future.set(true); - onSecurityUpdated(); - } - }.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR, recipient); - + future.set(true); + onSecurityUpdated(); return future; } @@ -1084,20 +1038,6 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity sendButton.setDefaultSubscriptionId(defaultSubscriptionId); } - private void initializeMmsEnabledCheck() { - new AsyncTask() { - @Override - protected Boolean doInBackground(Void... params) { - return Util.isMmsCapable(ConversationActivity.this); - } - - @Override - protected void onPostExecute(Boolean isMmsEnabled) { - ConversationActivity.this.isMmsEnabled = isMmsEnabled; - } - }.executeOnExecutor(AsyncTask.THREAD_POOL_EXECUTOR); - } - private void initializeViews() { ActionBar supportActionBar = getSupportActionBar(); if (supportActionBar == null) throw new AssertionError();