From 4bd12f5abe75f17afa2da5d5671d8fe373a1c63a Mon Sep 17 00:00:00 2001 From: Alexander Bakker Date: Tue, 6 Dec 2022 18:41:29 +0100 Subject: [PATCH] Don't try to move entries that are filtered out This fixes a crash that could occur when changing an entry in such a way that it is filtered out from the entry list after the change. --- .../java/com/beemdevelopment/aegis/ui/views/EntryAdapter.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/com/beemdevelopment/aegis/ui/views/EntryAdapter.java b/app/src/main/java/com/beemdevelopment/aegis/ui/views/EntryAdapter.java index ba0d8bd5..d482f67a 100644 --- a/app/src/main/java/com/beemdevelopment/aegis/ui/views/EntryAdapter.java +++ b/app/src/main/java/com/beemdevelopment/aegis/ui/views/EntryAdapter.java @@ -1,5 +1,7 @@ package com.beemdevelopment.aegis.ui.views; +import static androidx.recyclerview.widget.RecyclerView.NO_POSITION; + import android.graphics.Typeface; import android.os.Handler; import android.text.SpannableString; @@ -211,7 +213,7 @@ public class EntryAdapter extends RecyclerView.Adapter sortShownEntries(); int newPosition = _shownEntries.indexOf(newEntry); - if (position != newPosition) { + if (newPosition != NO_POSITION && position != newPosition) { notifyItemMoved(position, newPosition); } } else if (!isEntryFiltered(newEntry)) {