From c321bc790fc071b0301def4d83179e1d9ef27a40 Mon Sep 17 00:00:00 2001 From: Benjamin Orsini Date: Thu, 6 Jun 2019 20:59:11 +0200 Subject: [PATCH] refactor: use instance adapter when possible --- .../exodusprivacy/fragments/AppListFragment.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/eu/exodus_privacy/exodusprivacy/fragments/AppListFragment.java b/app/src/main/java/org/eu/exodus_privacy/exodusprivacy/fragments/AppListFragment.java index 9bae039..cceee85 100644 --- a/app/src/main/java/org/eu/exodus_privacy/exodusprivacy/fragments/AppListFragment.java +++ b/app/src/main/java/org/eu/exodus_privacy/exodusprivacy/fragments/AppListFragment.java @@ -49,7 +49,7 @@ public class AppListFragment extends Fragment { private ApplicationListAdapter.OnAppClickListener onAppClickListener; private boolean startupRefresh; private ApplistBinding applistBinding; - private ApplicationListAdapter adapter; + private @Nullable ApplicationListAdapter adapter; public static AppListFragment newInstance(NetworkListener networkListener, ApplicationListAdapter.OnAppClickListener appClickListener) { AppListFragment fragment = new AppListFragment(); @@ -116,8 +116,8 @@ public class AppListFragment extends Fragment { if(applistBinding != null) { applistBinding.layoutProgress.setVisibility(View.GONE); applistBinding.swipeRefresh.setRefreshing(false); - if(packageManager != null && applistBinding.appList.getAdapter() != null) { - ((ApplicationListAdapter) applistBinding.appList.getAdapter()).displayApplicationList(packageManager); + if(packageManager != null && adapter != null) { + adapter.displayApplicationList(packageManager); } } } @@ -175,6 +175,8 @@ public class AppListFragment extends Fragment { } public void filter(String filter){ - adapter.filter(filter); + if(adapter != null) { + adapter.filter(filter); + } } }