How do you get a persons phone number from the address book?

Question

All I want to do is let the user select a number from the address book. I found the code in this question:

http://stackoverflow.com/questions/286207/how-to-get-a-phone-number-from-an-address-book-contact-iphone-sdk

ABMultiValueRef container = ABRecordCopyValue(person, property);
CFStringRef contactData = ABMultiValueCopyValueAtIndex(container, identifier);
CFRelease(container);
NSString *contactString = [NSString stringWithString:(NSString *)contactData];
CFRelease(contactData);

The problem is that on the second line (when running on a 3.0 device) I get the following error:

Account Manager could not find account with identifier MobileMe:rustyshelf

followed by:

Program received signal: "EXC_BAD_ACCESS".

This is all inside the picker delegate method:

- (BOOL)peoplePickerNavigationController:(ABPeoplePickerNavigationController *)peoplePicker shouldContinueAfterSelectingPerson:(ABRecordRef)person property:(ABPropertyID)property identifier:(ABMultiValueIdentifier)identifier{

This is just one of the contacts in my address book, which is synched with Mobile Me

Edit: I think this might be a bug with the SDK, it happens for some of my contacts but not for others...

Answer 1

You should be able to just do the following:

ABMultiValueRef phoneNumbers = (ABMultiValueRef)ABRecordCopyValue(person, kABPersonPhoneProperty);
CFRelease(phoneNumbers);
NSString* phoneNumber = (NSString*)ABMultiValueCopyValueAtIndex(phoneNumbers, 0);

Of course, that's only going to get you the first number of (potentially) many associated with the person who's been selected.

Answer 2

I use this to pull the mobile number from an ABRecordRef/ The "record" variable is the ABRecordRef you want the phone number for. Where I have "" you can use another phone tag string to find other types of phone numbers.

//Get mobile phone number
ABMultiValueRef phones =(NSString*)ABRecordCopyValue(record, kABPersonPhoneProperty);
NSString* mobile=@"";
NSString* mobileLabel;
for(CFIndex i = 0; i < ABMultiValueGetCount(phones); i++) {
    mobileLabel = (NSString*)ABMultiValueCopyLabelAtIndex(phones, i);
    if([mobileLabel isEqualToString:@"_$!<Mobile>!$_"]) {
          mobile = (NSString*)ABMultiValueCopyValueAtIndex(phones, i);
    }
}

Answer 3

The "identifier" argument does not hold the CFIndex of the record touched. The method I use to tell which phone number a user selected is this:

- (BOOL)peoplePickerNavigationController:(ABPeoplePickerNavigationController *)peoplePicker shouldContinueAfterSelectingPerson:(ABRecordRef)person property:(ABPropertyID)property identifier:(ABMultiValueIdentifier)identifier
{
    if (property == kABPersonPhoneProperty) {
     ABMultiValueRef multiPhones = ABRecordCopyValue(person, kABPersonPhoneProperty);
     for(CFIndex i = 0; i < ABMultiValueGetCount(multiPhones); i++) {
      if(identifier == ABMultiValueGetIdentifierAtIndex (multiPhones, i)) {
       CFStringRef phoneNumberRef = ABMultiValueCopyValueAtIndex(multiPhones, i);
       CFRelease(multiPhones);
       NSString *phoneNumber = (NSString *) phoneNumberRef;
       CFRelease(phoneNumberRef);
       txtPhoneNumber.text = [NSString stringWithFormat:@"%@", phoneNumber];
       [phoneNumber release];
      }
     }
    }

    [self dismissModalViewControllerAnimated:YES];
    return NO;
}

Answer 4

Going off of JWDs answer, here is a safer version that uses the builtin constants and chooses the iphone number over another mobile number...

ABMultiValueRef phones =(NSString*)ABRecordCopyValue(person, kABPersonPhoneProperty);
NSString* mobile=@"";
NSString* mobileLabel;
for(CFIndex i = 0; i < ABMultiValueGetCount(phones); i++) {
 mobileLabel = (NSString*)ABMultiValueCopyLabelAtIndex(phones, i);
 if([mobileLabel isEqualToString:(NSString *)kABPersonPhoneMobileLabel])
 {
  [mobile release] ;
  mobile = (NSString*)ABMultiValueCopyValueAtIndex(phones, i);
 }
 else if ([mobileLabel isEqualToString:(NSString*)kABPersonPhoneIPhoneLabel])
 {
  [mobile release] ;
  mobile = (NSString*)ABMultiValueCopyValueAtIndex(phones, i);
  break ;
 }
}