Add final modifiers

pull/996/head
Alex Baker 6 years ago
parent bba7e85c48
commit c95fa9a1df

@ -15,7 +15,7 @@ import org.tasks.time.DateTime;
@RunWith(AndroidJUnit4.class) @RunWith(AndroidJUnit4.class)
public class AppleRemindersTests { public class AppleRemindersTests {
private TimeZone defaultTimeZone = TimeZone.getDefault(); private final TimeZone defaultTimeZone = TimeZone.getDefault();
@Before @Before
public void before() { public void before() {

@ -15,7 +15,7 @@ import org.tasks.time.DateTime;
@RunWith(AndroidJUnit4.class) @RunWith(AndroidJUnit4.class)
public class NextCloudTests { public class NextCloudTests {
private TimeZone defaultTimeZone = TimeZone.getDefault(); private final TimeZone defaultTimeZone = TimeZone.getDefault();
@Before @Before
public void before() { public void before() {

@ -13,7 +13,7 @@ import org.junit.runner.RunWith;
@RunWith(AndroidJUnit4.class) @RunWith(AndroidJUnit4.class)
public class SynologyTests { public class SynologyTests {
private TimeZone defaultTimeZone = TimeZone.getDefault(); private final TimeZone defaultTimeZone = TimeZone.getDefault();
@Before @Before
public void before() { public void before() {

@ -15,7 +15,7 @@ import org.tasks.time.DateTime;
@RunWith(AndroidJUnit4.class) @RunWith(AndroidJUnit4.class)
public class ThunderbirdTests { public class ThunderbirdTests {
private TimeZone defaultTimeZone = TimeZone.getDefault(); private final TimeZone defaultTimeZone = TimeZone.getDefault();
@Before @Before
public void before() { public void before() {

@ -185,7 +185,7 @@ public final class TaskListFragment extends InjectingFragment
private TaskAdapter taskAdapter = null; private TaskAdapter taskAdapter = null;
private TaskListRecyclerAdapter recyclerAdapter; private TaskListRecyclerAdapter recyclerAdapter;
private Filter filter; private Filter filter;
private PublishSubject<String> searchSubject = PublishSubject.create(); private final PublishSubject<String> searchSubject = PublishSubject.create();
private Disposable searchDisposable; private Disposable searchDisposable;
private MenuItem search; private MenuItem search;
private String searchQuery; private String searchQuery;

@ -54,7 +54,7 @@ public class NavigationDrawerAdapter extends RecyclerView.Adapter<ViewHolder> {
private OnClick onClick; private OnClick onClick;
private Filter selected = null; private Filter selected = null;
private AsyncListDiffer<FilterListItem> differ; private final AsyncListDiffer<FilterListItem> differ;
@Inject @Inject
public NavigationDrawerAdapter( public NavigationDrawerAdapter(

@ -33,7 +33,7 @@ public class FilterSelectionActivity extends InjectingAppCompatActivity {
private CompositeDisposable disposables; private CompositeDisposable disposables;
private Filter selected; private Filter selected;
private BroadcastReceiver refreshReceiver = new BroadcastReceiver() { private final BroadcastReceiver refreshReceiver = new BroadcastReceiver() {
@Override @Override
public void onReceive(Context context, Intent intent) { public void onReceive(Context context, Intent intent) {
refresh(); refresh();

@ -43,7 +43,7 @@ public class RemoteListPicker extends InjectingDialogFragment
@Inject LocalBroadcastManager localBroadcastManager; @Inject LocalBroadcastManager localBroadcastManager;
private CompositeDisposable disposables; private CompositeDisposable disposables;
private BroadcastReceiver refreshReceiver = new BroadcastReceiver() { private final BroadcastReceiver refreshReceiver = new BroadcastReceiver() {
@Override @Override
public void onReceive(Context context, Intent intent) { public void onReceive(Context context, Intent intent) {
refresh(); refresh();

@ -66,7 +66,7 @@ public class TasksJsonImporter {
private final Preferences preferences; private final Preferences preferences;
private final LocationDao locationDao; private final LocationDao locationDao;
private ImportResult result = new ImportResult(); private final ImportResult result = new ImportResult();
@Inject @Inject
public TasksJsonImporter( public TasksJsonImporter(

@ -21,7 +21,7 @@ public class Inventory {
private final SignatureVerifier signatureVerifier; private final SignatureVerifier signatureVerifier;
private final LocalBroadcastManager localBroadcastManager; private final LocalBroadcastManager localBroadcastManager;
private Map<String, Purchase> purchases = new HashMap<>(); private final Map<String, Purchase> purchases = new HashMap<>();
private Purchase subscription = null; private Purchase subscription = null;
@Inject @Inject

@ -7,7 +7,7 @@ import org.tasks.R;
public class IconLayoutManager extends GridLayoutManager { public class IconLayoutManager extends GridLayoutManager {
private int iconSize; private final int iconSize;
public IconLayoutManager(Context context) { public IconLayoutManager(Context context) {
super(context, DEFAULT_SPAN_COUNT, RecyclerView.VERTICAL, false); super(context, DEFAULT_SPAN_COUNT, RecyclerView.VERTICAL, false);

@ -43,7 +43,7 @@ import java.util.Comparator;
*/ */
public class AlphanumComparator<T> implements Comparator<T> { public class AlphanumComparator<T> implements Comparator<T> {
public static Function<Filter, String> FILTER = f -> f.listingTitle; public static final Function<Filter, String> FILTER = f -> f.listingTitle;
private final Function<T, String> getTitle; private final Function<T, String> getTitle;

@ -37,7 +37,7 @@ public class PlaceFilter extends Filter {
} }
}; };
private Place place; private final Place place;
private static final Table G2 = Geofence.TABLE.as("G2"); private static final Table G2 = Geofence.TABLE.as("G2");
private static final Field G2_PLACE = Field.field("G2.place"); private static final Field G2_PLACE = Field.field("G2.place");
private static final Field G2_TASK = Field.field("G2.task"); private static final Field G2_TASK = Field.field("G2.task");

@ -62,7 +62,7 @@ public class WorkManager {
private final GoogleTaskListDao googleTaskListDao; private final GoogleTaskListDao googleTaskListDao;
private final CaldavDao caldavDao; private final CaldavDao caldavDao;
private final AlarmManager alarmManager; private final AlarmManager alarmManager;
private androidx.work.WorkManager workManager; private final androidx.work.WorkManager workManager;
@Inject @Inject
public WorkManager( public WorkManager(

@ -125,7 +125,7 @@ public class LocationPickerActivity extends InjectingAppCompatActivity
private List<PlaceUsage> places = Collections.emptyList(); private List<PlaceUsage> places = Collections.emptyList();
private int offset; private int offset;
private MenuItem search; private MenuItem search;
private PublishSubject<String> searchSubject = PublishSubject.create(); private final PublishSubject<String> searchSubject = PublishSubject.create();
private PlaceSearchViewModel viewModel; private PlaceSearchViewModel viewModel;
@Override @Override

@ -129,7 +129,7 @@ public class LocationSearchAdapter extends RecyclerView.Adapter<ViewHolder>
public static class FooterViewHolder extends RecyclerView.ViewHolder { public static class FooterViewHolder extends RecyclerView.ViewHolder {
View divider; final View divider;
FooterViewHolder(@NonNull View itemView, @DrawableRes int attributionRes) { FooterViewHolder(@NonNull View itemView, @DrawableRes int attributionRes) {
super(itemView); super(itemView);

@ -31,7 +31,7 @@ public class MapboxMapFragment implements MapFragment, OnMapReadyCallback, OnMar
private MapFragmentCallback callbacks; private MapFragmentCallback callbacks;
private boolean dark; private boolean dark;
private MapboxMap map; private MapboxMap map;
private Map<Marker, Place> markers = new HashMap<>(); private final Map<Marker, Place> markers = new HashMap<>();
public MapboxMapFragment(Context context) { public MapboxMapFragment(Context context) {
this.context = context; this.context = context;

@ -17,9 +17,9 @@ import org.tasks.data.Place;
public class PlaceSearchViewModel extends ViewModel { public class PlaceSearchViewModel extends ViewModel {
private PlaceSearchProvider searchProvider; private PlaceSearchProvider searchProvider;
private MutableLiveData<List<PlaceSearchResult>> searchResults = new MutableLiveData<>(); private final MutableLiveData<List<PlaceSearchResult>> searchResults = new MutableLiveData<>();
private MutableLiveData<Event<String>> error = new MutableLiveData<>(); private final MutableLiveData<Event<String>> error = new MutableLiveData<>();
private MutableLiveData<Place> selection = new MutableLiveData<>(); private final MutableLiveData<Place> selection = new MutableLiveData<>();
void setSearchProvider(PlaceSearchProvider searchProvider) { void setSearchProvider(PlaceSearchProvider searchProvider) {
this.searchProvider = searchProvider; this.searchProvider = searchProvider;

@ -37,9 +37,9 @@ public class DragAndDropRecyclerAdapter extends TaskListRecyclerAdapter {
private final TaskListFragment taskList; private final TaskListFragment taskList;
private final RecyclerView recyclerView; private final RecyclerView recyclerView;
private List<TaskContainer> list; private List<TaskContainer> list;
private PublishSubject<List<TaskContainer>> publishSubject = PublishSubject.create(); private final PublishSubject<List<TaskContainer>> publishSubject = PublishSubject.create();
private CompositeDisposable disposables = new CompositeDisposable(); private final CompositeDisposable disposables = new CompositeDisposable();
private Queue<Pair<List<TaskContainer>, DiffResult>> updates = new LinkedList<>(); private final Queue<Pair<List<TaskContainer>, DiffResult>> updates = new LinkedList<>();
private boolean dragging; private boolean dragging;
public DragAndDropRecyclerAdapter( public DragAndDropRecyclerAdapter(

@ -14,7 +14,7 @@ import org.tasks.data.TaskContainer;
public class PagedListRecyclerAdapter extends TaskListRecyclerAdapter { public class PagedListRecyclerAdapter extends TaskListRecyclerAdapter {
private final RecyclerView recyclerView; private final RecyclerView recyclerView;
private AsyncPagedListDiffer<TaskContainer> differ; private final AsyncPagedListDiffer<TaskContainer> differ;
public PagedListRecyclerAdapter( public PagedListRecyclerAdapter(
TaskAdapter adapter, TaskAdapter adapter,

@ -40,7 +40,7 @@ public class TaskListViewModel extends ViewModel implements Observer<PagedList<T
private MutableLiveData<List<TaskContainer>> tasks = new MutableLiveData<>(); private MutableLiveData<List<TaskContainer>> tasks = new MutableLiveData<>();
private Filter filter; private Filter filter;
private boolean manualSort; private boolean manualSort;
private CompositeDisposable disposable = new CompositeDisposable(); private final CompositeDisposable disposable = new CompositeDisposable();
private LiveData<PagedList<TaskContainer>> internal; private LiveData<PagedList<TaskContainer>> internal;
public void setFilter(@NonNull Filter filter) { public void setFilter(@NonNull Filter filter) {

Loading…
Cancel
Save