commit 4ccae5617bbf9295364015e2ef7e716f65d2a18f
author Daniel Lehmann <lehmannd@google.com> Mon May 02 16:39:01 2011 -0700
committer Daniel Lehmann <lehmannd@google.com> Mon May 02 16:39:01 2011 -0700
tree b6cc1bd1c70fdd438028eb48553ad4142837c590
parent 1b22073590c1801c3e6c7bde2f4636632a049f6c

Don't do the LIMIT as part of SORT_BY, but pass it into its own parameter

This is a prerequisite for

Bug:4368912
Change-Id: Ic855ccd768cbce19a3947aff14857907ec2caf93

src/com/android/contacts/ContactPhotoManager.java

diff --git a/src/com/android/contacts/ContactPhotoManager.java b/src/com/android/contacts/ContactPhotoManager.java
index 914b697..150856a 100644
--- a/src/com/android/contacts/ContactPhotoManager.java
+++ b/src/com/android/contacts/ContactPhotoManager.java
@@ -625,12 +625,13 @@
             try {
                 Uri uri = Contacts.CONTENT_URI.buildUpon().appendQueryParameter(
                         ContactsContract.DIRECTORY_PARAM_KEY, String.valueOf(Directory.DEFAULT))
+                        .appendQueryParameter(ContactsContract.LIMIT_PARAM_KEY,
+                                String.valueOf(MAX_PHOTOS_TO_PRELOAD))
                         .build();
                 cursor = mResolver.query(uri, new String[] { Contacts.PHOTO_ID },
                         Contacts.PHOTO_ID + " NOT NULL AND " + Contacts.PHOTO_ID + "!=0",
                         null,
-                        Contacts.STARRED + " DESC, " + Contacts.LAST_TIME_CONTACTED + " DESC"
-                                + " LIMIT " + MAX_PHOTOS_TO_PRELOAD);
+                        Contacts.STARRED + " DESC, " + Contacts.LAST_TIME_CONTACTED + " DESC");
 
                 if (cursor != null) {
                     while (cursor.moveToNext()) {