Bladeren bron

Merge pull request #3225 from nextcloud/notificationOutOfBounds

As async, it can happen that list already changed
Andy Scherzinger 6 jaren geleden
bovenliggende
commit
8ba7257628
1 gewijzigde bestanden met toevoegingen van 5 en 2 verwijderingen
  1. 5 2
      src/main/java/com/owncloud/android/ui/adapter/NotificationListAdapter.java

+ 5 - 2
src/main/java/com/owncloud/android/ui/adapter/NotificationListAdapter.java

@@ -234,8 +234,11 @@ public class NotificationListAdapter extends RecyclerView.Adapter<NotificationLi
         protected void onPostExecute(Boolean success) {
             if (success) {
                 int position = holder.getAdapterPosition();
-                notificationsList.remove(position);
-                notifyItemRemoved(position);
+
+                if (position >= 0 && position < notificationsList.size()) {
+                    notificationsList.remove(position);
+                    notifyItemRemoved(position);
+                }
             } else {
                 setButtonEnabled(holder, true);
                 DisplayUtils.showSnackMessage(notificationsActivity, "Failed to execute action!");