Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 463917bc authored by Tyler Gunn's avatar Tyler Gunn Committed by Divya Sharma
Browse files

Updating "Add Contact" to pre-populate full contact info.

1. For recent call entries with a name and phone number, the name will
now also be added.
2. For recent call entries which have complete address information provided
by nearby places, all information except the contact photo will be added.  The contact photo is
excluded, as the ContactsLoader would attempt to load the contact photo
while parsing the contact information.  On a slow connection this could
cause a noticeable pause while adding the contact.

Bug: 17308163

Conflicts:
	src/com/android/dialer/calllog/CallLogAdapter.java

Change-Id: I074301b316a61c85699f91cca22444b389de9b3d
parent e73daa51
Loading
Loading
Loading
Loading
+16 −2
Original line number Diff line number Diff line
@@ -1108,8 +1108,22 @@ public class DialtactsActivity extends TransactionSafeActivity implements View.O
    }

    public static Intent getAddNumberToContactIntent(CharSequence text) {
        final Intent intent = new Intent(Intent.ACTION_INSERT_OR_EDIT);
        intent.putExtra(Intents.Insert.PHONE, text);
        return getAddToContactIntent(null /* name */, text /* phoneNumber */,
                -1 /* phoneNumberType */);
    }

    public static Intent getAddToContactIntent(CharSequence name, CharSequence phoneNumber,
            int phoneNumberType) {
        Intent intent = new Intent(Intent.ACTION_INSERT_OR_EDIT);
        intent.putExtra(Intents.Insert.PHONE, phoneNumber);
        // Only include the name and phone type extras if they are specified (the method
        // getAddNumberToContactIntent does not use them).
        if (name != null) {
            intent.putExtra(Intents.Insert.NAME, name);
        }
        if (phoneNumberType != -1) {
            intent.putExtra(Intents.Insert.PHONE_TYPE, phoneNumberType);
        }
        intent.setType(Contacts.CONTENT_ITEM_TYPE);
        return intent;
    }
+67 −5
Original line number Diff line number Diff line
@@ -20,6 +20,7 @@ import android.accounts.Account;
import android.content.ContentValues;
import android.content.Context;
import android.content.Intent;
import android.content.Loader;
import android.content.res.Resources;
import android.database.Cursor;
import android.graphics.drawable.Drawable;
@@ -27,6 +28,7 @@ import android.net.Uri;
import android.os.Handler;
import android.os.Message;
import android.provider.CallLog.Calls;
import android.provider.ContactsContract;
import android.provider.ContactsContract.PhoneLookup;
import android.telecom.PhoneAccountHandle;
import android.telephony.SubscriptionManager;
@@ -46,6 +48,8 @@ import com.android.common.widget.GroupingListAdapter;
import com.android.contacts.common.CallUtil;
import com.android.contacts.common.ContactPhotoManager;
import com.android.contacts.common.ContactPhotoManager.DefaultImageRequest;
import com.android.contacts.common.model.Contact;
import com.android.contacts.common.model.ContactLoader;
import com.android.contacts.common.util.UriUtils;
import com.android.dialer.DialtactsActivity;
import com.android.dialer.PhoneCallDetails;
@@ -57,6 +61,7 @@ import com.android.dialer.util.ExpirableCache;
import com.google.common.annotations.VisibleForTesting;
import com.google.common.base.Objects;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.LinkedList;

@@ -1057,7 +1062,7 @@ public class CallLogAdapter extends GroupingListAdapter
    }

    protected void bindBadge(
            View view, ContactInfo info, final PhoneCallDetails details, int callType) {
            View view, final ContactInfo info, final PhoneCallDetails details, int callType) {
        // Do not show badge in call log.
        if (!mIsCallLog) {
            final ViewStub stub = (ViewStub) view.findViewById(R.id.link_stub);
@@ -1073,10 +1078,18 @@ public class CallLogAdapter extends GroupingListAdapter
                mBadgeContainer.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                            // If no lookup uri is provided, we need to rely on what information
                            // we have available; namely the phone number and name.
                            if (info.lookupUri == null) {
                                final Intent intent =
                                DialtactsActivity.getAddNumberToContactIntent(details.number);
                                        DialtactsActivity.getAddToContactIntent(details.name,
                                                details.number,
                                                details.numberType);
                                DialerUtils.startActivityWithErrorToast(mContext, intent,
                                        R.string.add_contact_not_available);
                            } else {
                                addContactFromLookupUri(info.lookupUri);
                            }
                    }
                });
                mBadgeImageView.setImageResource(R.drawable.ic_person_add_24dp);
@@ -1426,4 +1439,53 @@ public class CallLogAdapter extends GroupingListAdapter
            }
        }
    }

    /**
     * Invokes the "add contact" activity given the expanded contact information stored in a
     * lookup URI.  This can include, for example, address and website information.
     *
     * @param lookupUri The lookup URI.
     */
    private void addContactFromLookupUri(Uri lookupUri) {
        Contact contactToSave = ContactLoader.parseEncodedContactEntity(lookupUri);
        if (contactToSave == null) {
            return;
        }

        // Note: This code mirrors code in Contacts/QuickContactsActivity.
        final Intent intent = new Intent(Intent.ACTION_INSERT_OR_EDIT);
        intent.setType(ContactsContract.Contacts.CONTENT_ITEM_TYPE);

        ArrayList<ContentValues> values = contactToSave.getContentValues();
        // Only pre-fill the name field if the provided display name is an nickname
        // or better (e.g. structured name, nickname)
        if (contactToSave.getDisplayNameSource()
                >= ContactsContract.DisplayNameSources.NICKNAME) {
            intent.putExtra(ContactsContract.Intents.Insert.NAME,
                    contactToSave.getDisplayName());
        } else if (contactToSave.getDisplayNameSource()
                == ContactsContract.DisplayNameSources.ORGANIZATION) {
            // This is probably an organization. Instead of copying the organization
            // name into a name entry, copy it into the organization entry. This
            // way we will still consider the contact an organization.
            final ContentValues organization = new ContentValues();
            organization.put(ContactsContract.CommonDataKinds.Organization.COMPANY,
                    contactToSave.getDisplayName());
            organization.put(ContactsContract.Data.MIMETYPE,
                    ContactsContract.CommonDataKinds.Organization.CONTENT_ITEM_TYPE);
            values.add(organization);
        }

        // Last time used and times used are aggregated values from the usage stat
        // table. They need to be removed from data values so the SQL table can insert
        // properly
        for (ContentValues value : values) {
            value.remove(ContactsContract.Data.LAST_TIME_USED);
            value.remove(ContactsContract.Data.TIMES_USED);
        }
        intent.putExtra(ContactsContract.Intents.Insert.DATA, values);

        DialerUtils.startActivityWithErrorToast(mContext, intent,
                R.string.add_contact_not_available);
    }
}