From 4fe4e2475cdb0139246e1e98f34f6d688ec1ec22 Mon Sep 17 00:00:00 2001
From: Karim Abou Zeid <karim23697@gmail.com>
Date: Sat, 19 Mar 2016 14:16:24 +0100
Subject: [PATCH] Fixed a FC

---
 .../gramophone/adapter/SongFileAdapter.java   | 25 +++++++++++++------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/SongFileAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/SongFileAdapter.java
index 4d7a93da..b485b25c 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/SongFileAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/SongFileAdapter.java
@@ -179,7 +179,10 @@ public class SongFileAdapter extends AbsMultiSelectAdapter<SongFileAdapter.ViewH
                 menu.setOnClickListener(new View.OnClickListener() {
                     @Override
                     public void onClick(View v) {
-                        callbacks.onFileMenuClicked(dataSet.get(getAdapterPosition()));
+                        int position = getAdapterPosition();
+                        if (isPositionInRange(position)) {
+                            callbacks.onFileMenuClicked(dataSet.get(position));
+                        }
                     }
                 });
             }
@@ -187,18 +190,26 @@ public class SongFileAdapter extends AbsMultiSelectAdapter<SongFileAdapter.ViewH
 
         @Override
         public void onClick(View v) {
-            if (isInQuickSelectMode()) {
-                toggleChecked(getAdapterPosition());
-            } else {
-                if (callbacks != null) {
-                    callbacks.onFileSelected(dataSet.get(getAdapterPosition()));
+            int position = getAdapterPosition();
+            if (isPositionInRange(position)) {
+                if (isInQuickSelectMode()) {
+                    toggleChecked(position);
+                } else {
+                    if (callbacks != null) {
+                        callbacks.onFileSelected(dataSet.get(position));
+                    }
                 }
             }
         }
 
         @Override
         public boolean onLongClick(View view) {
-            return toggleChecked(getAdapterPosition());
+            int position = getAdapterPosition();
+            return isPositionInRange(position) && toggleChecked(position);
+        }
+
+        private boolean isPositionInRange(int position) {
+            return position >= 0 && position < dataSet.size();
         }
     }