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

Android getAllContacts not working #10

Closed
niftylettuce opened this issue Nov 4, 2015 · 22 comments · Fixed by #13
Closed

Android getAllContacts not working #10

niftylettuce opened this issue Nov 4, 2015 · 22 comments · Fixed by #13

Comments

@niftylettuce
Copy link

red screen

attempt to invoke interface method boolean android.database.Cursor.moveToNext() on a null object reference
@niftylettuce
Copy link
Author

Occured at getAll ... ContactsManager.java:38

@niftylettuce
Copy link
Author

"Read contacts... 2 mins ago... Allowed 9843 times"

Something must really be off!

@rt2zz
Copy link
Collaborator

rt2zz commented Nov 4, 2015

yikes x.x

I have no specific timeline for addressing this, but I will post here when and if I am jumping in

@rt2zz
Copy link
Collaborator

rt2zz commented Nov 4, 2015

fwiw, if anyone wants to look into this, I think the better approach here is to adapt code from an existing package like cordova contacts plugin: https://github.com/apache/cordova-plugin-contacts/tree/master/src/android

@niftylettuce
Copy link
Author

@rt2zz I tried patching it, but getAll doesn't work. Not sure if this is due to some change in Lollipop. I'm not sure really. I would gladly pay a $50-100+ bounty if someone fixed this.

@niftylettuce
Copy link
Author

@elliottsj can you help?

@niftylettuce
Copy link
Author

Can anyone help here? @rt2zz please re-open /cc @morenoh149

@niftylettuce
Copy link
Author

$250 bug bounty

@morenoh149
Copy link
Owner

It's definitely working. Try performing all the steps in the setup guide.

@niftylettuce
Copy link
Author

No it's not
On Dec 9, 2015 8:23 PM, "Harry Moreno" notifications@github.com wrote:

It's definitely working. Try performing all the steps in the setup guide.
[image: screen shot 2015-12-09 at 8 22 22 pm]
https://cloud.githubusercontent.com/assets/587438/11704096/abbc09c4-9eb2-11e5-9696-7539166925c3.png


Reply to this email directly or view it on GitHub
#10 (comment)
.

@niftylettuce
Copy link
Author

Try it with 300 contacts, try to break it like I did and got those cursor
errors
On Dec 9, 2015 8:24 PM, niftylettuce@gmail.com wrote:

No it's not
On Dec 9, 2015 8:23 PM, "Harry Moreno" notifications@github.com wrote:

It's definitely working. Try performing all the steps in the setup guide.
[image: screen shot 2015-12-09 at 8 22 22 pm]
https://cloud.githubusercontent.com/assets/587438/11704096/abbc09c4-9eb2-11e5-9696-7539166925c3.png


Reply to this email directly or view it on GitHub
#10 (comment)
.

@morenoh149
Copy link
Owner

how do I load >300 contacts into the simulator?

@niftylettuce
Copy link
Author

@niftylettuce
Copy link
Author

@niftylettuce
Copy link
Author

@niftylettuce
Copy link
Author

@morenoh149
Copy link
Owner

It'd be really great if you could upload a sample .vcf file with >300 contacts to gist.github.com for me @niftylettuce

@morenoh149
Copy link
Owner

Might be more productive to implement addContact to android and see what happens when we add >300 contacts

@niftylettuce
Copy link
Author

Yeah - maybe try that and then iterate it over 300 with fake names or
something?

On Thu, Dec 10, 2015 at 12:43 PM, Harry Moreno notifications@github.com
wrote:

Might be more productive to implement addContact to android and see what
happens when we add >300 contacts


Reply to this email directly or view it on GitHub
#10 (comment)
.

@niftylettuce
Copy link
Author

@morenoh149
Copy link
Owner

I found one with two http://www.phpclasses.org/browse/file/5543.html

@niftylettuce
Copy link
Author

@morenoh149 Anything I can do to help? Do you need some of the bug bounty up front?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants