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

Crashes if no existing SMS from Twilio number #7

Closed
alexpja opened this issue Sep 16, 2014 · 6 comments
Closed

Crashes if no existing SMS from Twilio number #7

alexpja opened this issue Sep 16, 2014 · 6 comments

Comments

@alexpja
Copy link
Contributor

alexpja commented Sep 16, 2014

I was testing the app and it continuously crashed while looking for the text messages from twilio, yet there was a thread with messages that I manually sent. I assume "address" in the SMS database is From field, because I had setup Twilio to reply with random text, and finally the app continued the process.

It was complaining about not being able to decompress data afterwards, but that is because I didn't send any data back.

So is it supposed to immediately check the inbox?

@ColdSauce
Copy link
Owner

Can you please provide the stack trace?

@alexpja
Copy link
Contributor Author

alexpja commented Sep 16, 2014

I will in three hours.

@alexpja
Copy link
Contributor Author

alexpja commented Sep 16, 2014

I uploaded two logcats, here and here. I assume that's what you need.

@ColdSauce
Copy link
Owner

All right. The first one is a bug with an IndexOutOfBounds.

Cursor allows you to get SMS data. There is probably an off by one error somewhere that causes it.

The second one is the fact that the program is not stopping at the end of the file.

I am re writing all of this code so don't worry about this for right now.

@sylvia43
Copy link
Contributor

It's a bug in the app, which Stefan is fixing/rewriting.

@alexpja
Copy link
Contributor Author

alexpja commented Sep 17, 2014

👍👍

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

No branches or pull requests

3 participants