From 2adbeeeba7e715ce1a26c1e56e4528cae54e8bf2 Mon Sep 17 00:00:00 2001
From: Matthew Wong <wongma@protonmail.ch>
Date: Sun, 9 Aug 2015 13:53:31 -0400
Subject: Use getAdapterPosition instead of position variable set in
 onBindViewHolder

---
 .../pwdstore/utils/PasswordRecyclerAdapter.java    | 27 +++++++++-------------
 1 file changed, 11 insertions(+), 16 deletions(-)

diff --git a/app/src/main/java/com/zeapo/pwdstore/utils/PasswordRecyclerAdapter.java b/app/src/main/java/com/zeapo/pwdstore/utils/PasswordRecyclerAdapter.java
index fb6bc8fa..65139074 100644
--- a/app/src/main/java/com/zeapo/pwdstore/utils/PasswordRecyclerAdapter.java
+++ b/app/src/main/java/com/zeapo/pwdstore/utils/PasswordRecyclerAdapter.java
@@ -1,9 +1,7 @@
 package com.zeapo.pwdstore.utils;
 
-import android.graphics.Color;
 import android.support.v7.view.ActionMode;
 import android.support.v7.widget.RecyclerView;
-import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.Menu;
 import android.view.MenuItem;
@@ -34,7 +32,6 @@ public class PasswordRecyclerAdapter extends RecyclerView.Adapter<PasswordRecycl
         public View view;
         public TextView name;
         public TextView type;
-        public int position;
 
         public ViewHolder(View v) {
             super(v);
@@ -89,13 +86,11 @@ public class PasswordRecyclerAdapter extends RecyclerView.Adapter<PasswordRecycl
             holder.type.setTextColor(activity.getResources().getColor(R.color.blue_grey_50));
         }
 
-        holder.position = position;
-
         holder.view.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
                 if (mActionMode != null) {
-                    toggleSelection(holder.position);
+                    toggleSelection(holder.getAdapterPosition(), holder.view);
                     if (selectedItems.isEmpty()) {
                         mActionMode.finish();
                     }
@@ -111,14 +106,13 @@ public class PasswordRecyclerAdapter extends RecyclerView.Adapter<PasswordRecycl
                 if (mActionMode != null) {
                     return false;
                 }
-                toggleSelection(holder.position);
+                toggleSelection(holder.getAdapterPosition(), holder.view);
                 // Start the CAB using the ActionMode.Callback
                 mActionMode = activity.startSupportActionMode(mActionModeCallback);
                 return true;
             }
         });
 
-        holder.view.setSelected(selectedItems.contains(position));
 
     }
 
@@ -159,7 +153,6 @@ public class PasswordRecyclerAdapter extends RecyclerView.Adapter<PasswordRecycl
         public void onDestroyActionMode(ActionMode mode) {
             selectedItems.clear();
             mActionMode = null;
-            notifyDataSetChanged();
         }
     };
 
@@ -191,19 +184,21 @@ public class PasswordRecyclerAdapter extends RecyclerView.Adapter<PasswordRecycl
     public void remove(int position) {
         this.values.remove(position);
         this.notifyItemRemoved(position);
-    }
-
-    public void toggleSelection(int position) {
-        if (!selectedItems.remove(position)) {
-            selectedItems.add(position);
-        }
         for (int selected : selectedItems) {
             if (selected > position) {
                 selectedItems.remove(selected);
                 selectedItems.add(selected - 1);
             }
         }
-        notifyItemChanged(position);
+    }
+
+    public void toggleSelection(int position, View view) {
+        if (!selectedItems.remove(position)) {
+            selectedItems.add(position);
+            view.setSelected(true);
+        } else {
+            view.setSelected(false);
+        }
     }
 
 }
\ No newline at end of file
-- 
cgit v1.2.3