diff --git a/bramble-api/src/main/java/org/briarproject/bramble/api/FeatureFlags.java b/bramble-api/src/main/java/org/briarproject/bramble/api/FeatureFlags.java index d9410b26a..e05c6fec5 100644 --- a/bramble-api/src/main/java/org/briarproject/bramble/api/FeatureFlags.java +++ b/bramble-api/src/main/java/org/briarproject/bramble/api/FeatureFlags.java @@ -7,7 +7,5 @@ public interface FeatureFlags { boolean shouldEnableImageAttachments(); - boolean shouldEnableRemoteContacts(); - boolean shouldEnablePrivateMessageDeletion(); } diff --git a/bramble-core/src/test/java/org/briarproject/bramble/test/BrambleCoreIntegrationTestModule.java b/bramble-core/src/test/java/org/briarproject/bramble/test/BrambleCoreIntegrationTestModule.java index 9e9317649..0f92fbce9 100644 --- a/bramble-core/src/test/java/org/briarproject/bramble/test/BrambleCoreIntegrationTestModule.java +++ b/bramble-core/src/test/java/org/briarproject/bramble/test/BrambleCoreIntegrationTestModule.java @@ -25,11 +25,6 @@ public class BrambleCoreIntegrationTestModule { return true; } - @Override - public boolean shouldEnableRemoteContacts() { - return true; - } - @Override public boolean shouldEnablePrivateMessageDeletion() { return true; diff --git a/briar-android/src/main/java/org/briarproject/briar/android/AppModule.java b/briar-android/src/main/java/org/briarproject/briar/android/AppModule.java index 6daebe10f..65bdcf782 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/AppModule.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/AppModule.java @@ -242,11 +242,6 @@ public class AppModule { return IS_DEBUG_BUILD; } - @Override - public boolean shouldEnableRemoteContacts() { - return IS_DEBUG_BUILD; - } - @Override public boolean shouldEnablePrivateMessageDeletion() { return IS_DEBUG_BUILD; diff --git a/briar-android/src/main/java/org/briarproject/briar/android/contact/ContactListFragment.java b/briar-android/src/main/java/org/briarproject/briar/android/contact/ContactListFragment.java index a4d5116b3..3cc1ce505 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/contact/ContactListFragment.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/contact/ContactListFragment.java @@ -14,7 +14,6 @@ import android.view.View; import android.view.ViewGroup; import android.widget.TextView; -import org.briarproject.bramble.api.FeatureFlags; import org.briarproject.bramble.api.contact.Contact; import org.briarproject.bramble.api.contact.ContactId; import org.briarproject.bramble.api.contact.ContactManager; @@ -57,7 +56,6 @@ import javax.inject.Inject; import io.github.kobakei.materialfabspeeddial.FabSpeedDial; import io.github.kobakei.materialfabspeeddial.FabSpeedDial.OnMenuItemClickListener; -import io.github.kobakei.materialfabspeeddial.FabSpeedDialMenu; import static android.os.Build.VERSION.SDK_INT; import static android.support.design.widget.Snackbar.LENGTH_INDEFINITE; @@ -85,8 +83,6 @@ public class ContactListFragment extends BaseFragment implements EventListener, EventBus eventBus; @Inject AndroidNotificationManager notificationManager; - @Inject - FeatureFlags featureFlags; private ContactListAdapter adapter; private BriarRecyclerView list; @@ -126,19 +122,7 @@ public class ContactListFragment extends BaseFragment implements EventListener, container, false); FabSpeedDial speedDial = contentView.findViewById(R.id.speedDial); - if (featureFlags.shouldEnableRemoteContacts()) { - speedDial.addOnMenuItemClickListener(this); - } else { - speedDial.setMenu(new FabSpeedDialMenu(contentView.getContext())); - speedDial.addOnStateChangeListener(open -> { - if (open) { - Intent intent = new Intent(getContext(), - ContactExchangeActivity.class); - startActivity(intent); - speedDial.closeMenu(); - } - }); - } + speedDial.addOnMenuItemClickListener(this); OnContactClickListener onContactClickListener = (view, item) -> { @@ -169,9 +153,10 @@ public class ContactListFragment extends BaseFragment implements EventListener, startActivity(i); } }; - adapter = new ContactListAdapter(getContext(), onContactClickListener); + adapter = new ContactListAdapter(requireContext(), + onContactClickListener); list = contentView.findViewById(R.id.list); - list.setLayoutManager(new LinearLayoutManager(getContext())); + list.setLayoutManager(new LinearLayoutManager(requireContext())); list.setAdapter(adapter); list.setEmptyImage(R.drawable.ic_empty_state_contact_list); list.setEmptyText(getString(R.string.no_contacts)); diff --git a/briar-headless/src/main/java/org/briarproject/briar/headless/HeadlessModule.kt b/briar-headless/src/main/java/org/briarproject/briar/headless/HeadlessModule.kt index 6719f0508..adbd63546 100644 --- a/briar-headless/src/main/java/org/briarproject/briar/headless/HeadlessModule.kt +++ b/briar-headless/src/main/java/org/briarproject/briar/headless/HeadlessModule.kt @@ -98,7 +98,6 @@ internal class HeadlessModule(private val appDir: File) { @Provides internal fun provideFeatureFlags() = object : FeatureFlags { override fun shouldEnableImageAttachments() = false - override fun shouldEnableRemoteContacts() = true override fun shouldEnablePrivateMessageDeletion() = true } } diff --git a/briar-headless/src/test/java/org/briarproject/briar/headless/HeadlessTestModule.kt b/briar-headless/src/test/java/org/briarproject/briar/headless/HeadlessTestModule.kt index f094c9ef9..8ef96b5e4 100644 --- a/briar-headless/src/test/java/org/briarproject/briar/headless/HeadlessTestModule.kt +++ b/briar-headless/src/test/java/org/briarproject/briar/headless/HeadlessTestModule.kt @@ -65,7 +65,6 @@ internal class HeadlessTestModule(private val appDir: File) { @Provides internal fun provideFeatureFlags() = object : FeatureFlags { override fun shouldEnableImageAttachments() = false - override fun shouldEnableRemoteContacts() = true override fun shouldEnablePrivateMessageDeletion() = true } }