Android: fix opening the wrong document from the file picker

Sort the file list rather than the list adapter, so that
the onclick position can validly be used to index the file list

1 files changed, 9 insertions, 3 deletions

diff --git a/android/src/com/artifex/mupdf/ChoosePDFActivity.java b/android/src/com/artifex/mupdf/ChoosePDFActivity.java
index 26f4d873..441fab4b 100644
--- a/android/src/com/artifex/mupdf/ChoosePDFActivity.java
+++ b/android/src/com/artifex/mupdf/ChoosePDFActivity.java
@@ -2,8 +2,8 @@ package com.artifex.mupdf;
 
 import java.io.File;
 import java.io.FilenameFilter;
-import java.util.ArrayList;
-import java.util.List;
+import java.util.Arrays;
+import java.util.Comparator;
 
 import android.app.AlertDialog;
 import android.app.ListActivity;
@@ -83,11 +83,17 @@ public class ChoosePDFActivity extends ListActivity {
 						return false;
 					}
 				});
+
+				Arrays.sort(mFiles, new Comparator<File>() {
+					public int compare(File arg0, File arg1) {
+						return arg0.getName().compareToIgnoreCase(arg1.getName());
+					}
+				});
+
 				adapter.clear();
 				if (mFiles != null)
 					for (File f : mFiles)
 						adapter.add(f.getName());
-				adapter.sort(String.CASE_INSENSITIVE_ORDER);
 			}
 		};