iOS ABPeoplePickerNavigationController

[XcodeNewb]

I am trying to get a phone number from the contacts picker. My problem is that the identifier provided by the shouldContinueAfterSelectingPerson function is not consistent and is causing the app to crash.

Here is the code for that function:

- (BOOL)peoplePickerNavigationController:(ABPeoplePickerNavigationController *)peoplePicker shouldContinueAfterSelectingPerson:(ABRecordRef)person property:(ABPropertyID)property identifier:(ABMultiValueIdentifier)identifier{
	
        NSLog(@"Here ident[%d]", identifier);
	
	ABMultiValueRef container = ABRecordCopyValue(person, property);
	CFStringRef contactData = ABMultiValueCopyValueAtIndex(container, identifier);
	CFRelease(container);
	NSString *contactString = [NSString stringWithString:(NSString *)contactData];
	CFRelease(contactData);
	
	NSLog(@"PhoneNumber[%@]", contactString);
	
	[self dismissModalViewControllerAnimated:YES];	
	return NO;
}

The problem is that when I choose the first phone number listed for the contact person sometimes the identifier is given as a "0" and sometimes it is given as a "1" and sometimes a "2".

So when I try and run the command:

	CFStringRef contactData = ABMultiValueCopyValueAtIndex(container, identifier);

The app will crash if the identifier is given as a "2" and there is only one phone number. It is trying to access an object at index 2 when it does not exist.

Why would I be getting back a bad identifier number?

Please keep in mind this is my first dealing with the contact list so I may be completely missing something here.

Thanks

 

[XcodeNewb]

Anyone please??

 

[MacDonaldsd]

You need to use the ABMultiValueGetIndexForIdentifier() to get the index. The identifier isn't the index of that property.

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

	ABMultiValueRef multiValue = ABRecordCopyValue(person, property);
	
	NSString *number = (NSString *)ABMultiValueCopyValueAtIndex(multiValue, ABMultiValueGetIndexForIdentifier(multiValue, identifier));

        //Copy the number etc before cleaning everything up
	
        [number release];
	CFRelease(multiValue);

        return NO;
     



}

 

[XcodeNewb]

Thank you very much!

 

[snakeninny]

You need to use the ABMultiValueGetIndexForIdentifier() to get the index. The identifier isn't the index of that property.

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

	ABMultiValueRef multiValue = ABRecordCopyValue(person, property);
	
	NSString *number = (NSString *)ABMultiValueCopyValueAtIndex(multiValue, ABMultiValueGetIndexForIdentifier(multiValue, identifier));

        //Copy the number etc before cleaning everything up
	
        [number release];
	CFRelease(multiValue);

        return NO;
     



}

people are copying and pasting the same mistake everywhere, so your correct solution is very precious thank you! you've made my day