Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Lacking wrapper for ABAddressBookCopyPeopleWithName() #9

Closed
GoogleCodeExporter opened this issue Apr 21, 2015 · 3 comments
Closed

Lacking wrapper for ABAddressBookCopyPeopleWithName() #9

GoogleCodeExporter opened this issue Apr 21, 2015 · 3 comments
Labels
auto-migrated Priority-Medium Type-Defect

Comments

@GoogleCodeExporter
Copy link 

There is no analogue for the AddressBook API's
ABAddressBookCopyPeopleWithName() in GTMABAddressBook. This would be
trivial to add.

I'm working on a patch that I'll submit shortly - just need to add unit
testing and figure out how to jump through all the submission hoops.

Original issue reported on code.google.com by bryanhe...@mac.com on 7 Dec 2008 at 7:54
@GoogleCodeExporter
Copy link
Author 

ABAddressBookCopyPeopleWithName() is affected by rdar://problem//6425729
(ABAddressBookCopyPeopleWithName() does not correctly match by composite name). 
See
http://openradar.appspot.com/6425729 for the text of the bug report.

I've added a note to the top of GTMABAddressBook.h and adjusted the unit tests 
to
test for this.

Original comment by bryanhe...@mac.com on 7 Dec 2008 at 9:11

@GoogleCodeExporter
Copy link
Author 

I've added an issue to Rietveld for code review. Let me know if anything needs 
to be
modified.

http://codereview.appspot.com/10244

Original comment by bryanhe...@mac.com on 7 Dec 2008 at 9:23

@GoogleCodeExporter
Copy link
Author

Original comment by thoma...@gmail.com on 9 Dec 2008 at 4:21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated Priority-Medium Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thomasvl @GoogleCodeExporter