Update Contact Data By Data Id

My application uses the Contacts ContentProvider to store some of its data. When I load a contact into memory, I want to save its ID (so that I know how to save changes later), an

Solution 1:

I tried locally and it works for me, here's my code slightly adapted from yours:

publicvoidtestContacts(final@Nonnull Context context, finalint rawContactId, final@Nonnull String expectedDisplayName) {
    UrientityUri= Uri.withAppendedPath(
            ContentUris.withAppendedId(ContactsContract.RawContacts.CONTENT_URI, rawContactId),
            ContactsContract.RawContacts.Entity.CONTENT_DIRECTORY);

    CursorresultData= context.getContentResolver().query(
            entityUri,
            newString[]{
                    ContactsContract.RawContacts.SOURCE_ID,
                    ContactsContract.RawContacts.Entity.DATA_ID,
                    ContactsContract.RawContacts.Entity.MIMETYPE,
                    ContactsContract.RawContacts.Entity.DATA1
            },
            null, null, null);

    intdisplayNameId= -1;
    try {
        finalintcolumnIndexDataId= resultData.getColumnIndex(ContactsContract.RawContacts.Entity.DATA_ID);
        finalintcolumnIndexMimetype= resultData.getColumnIndex(ContactsContract.RawContacts.Entity.MIMETYPE);
        finalintcolumnIndexData= resultData.getColumnIndex(ContactsContract.RawContacts.Entity.DATA1);
        while (resultData.moveToNext()) {
            if (!resultData.isNull(columnIndexDataId)) {
                if (ContactsContract.CommonDataKinds.StructuredName.CONTENT_ITEM_TYPE.equals(resultData.getString(columnIndexMimetype)) &&
                        expectedDisplayName.equals(resultData.getString(columnIndexData))) {
                    displayNameId = resultData.getInt(1);
                    break;
                }
            }
        }
    } finally {
        resultData.close();
    }

    StringreLookedUpDisplayName=null;
    if (displayNameId != -1) {
        CursorreLookupCursor= context.getContentResolver().query(
                ContactsContract.Data.CONTENT_URI,
                newString[] {
                        ContactsContract.Data._ID,
                        ContactsContract.CommonDataKinds.StructuredName.DISPLAY_NAME
                },
                ContactsContract.Data._ID + "=?",
                newString[] {String.valueOf(displayNameId)},
                null);
        try {
            finalintcolumnIndexId= reLookupCursor.getColumnIndex(ContactsContract.Data._ID);
            finalintcolumnIndexDisplayName= reLookupCursor.getColumnIndex(ContactsContract.CommonDataKinds.StructuredName.DISPLAY_NAME);
            while (reLookupCursor.moveToNext()) {
                reLookedUpDisplayName = reLookupCursor.getString(columnIndexDisplayName);
            }
        } finally {
            reLookupCursor.close();
        }
    }

    Toast.makeText(
            context,
            reLookedUpDisplayName != null ? "Found re-looked up name: " + reLookedUpDisplayName : "Didn't find name re-looking it up",
            Toast.LENGTH_LONG)
            .show();
}

There's no big difference from your code, so compare or try to replace bits of it to see where you have a problem. Make sure you use a fresh Cursor for each query, and close it correctly afterwards (in a finally clause).

Another thing, make sure that if (resultData.getString(2).equals(Fields.DISPLAY_NAME)) is really what you're wanting to do (it compares the entry mime type with Fields.DISPLAY_NAME), but since you're saying you get the data ID correctly this shouldn't be the problem.