search

brajabasi / osmdroid

Automatically exported from code.google.com/p/osmdroid
0 stars 0 forks source link

OpenStreetMapActivity should enable/disable location updates in onResume/onPause, not in onCreate/onDestroy #53

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. Select the sample loader
2. Select the first sample
3. Press the 'BACK' button

What is the expected output? What do you see instead?
After step 2, location updates are enabled
After step 3, location updates are disabled

What steps will reproduce the problem?
1. Select the sample loader
2. Select the first sample
3. Press the 'HOME' button

What is the expected output? What do you see instead?
After step 2, location updates are enabled
After step 3, location updates are NOT disabled

This is inconsistent behaviour. In both cases you've exited the activity so 
they should behave the same.

Original issue reported on code.google.com by neilboyd on 25 May 2010 at 10:25

GoogleCodeExporter commented 8 years ago 
Could I ask for priority to be raised to medium on this.  It results in battery 
being killed by leaving on GPS when not running app.

Original comment by palmer.a...@gmail.com on 4 Oct 2010 at 2:29

GoogleCodeExporter commented 8 years ago

Original comment by neilboyd on 4 Oct 2010 at 3:11

GoogleCodeExporter commented 8 years ago 
As a little update, since making this change myself,  my phone has now gone for 
more than a day without charging, which hasnt happened for months!

Original comment by palmer.a...@gmail.com on 8 Oct 2010 at 3:00

GoogleCodeExporter commented 8 years ago 
I think this has been fixed. If so, then this issue can be closed.

Original comment by kurtzm...@gmail.com on 12 Jan 2011 at 5:19

GoogleCodeExporter commented 8 years ago 
No, it's still an issue. Go to the first sample and press the phone's 'home' 
button. GPS is still on.

Original comment by neilboyd on 12 Jan 2011 at 9:03

GoogleCodeExporter commented 8 years ago 
This should be fixed in revision 751. Please review and close this ticket if 
everything looks good.

Original comment by kurtzm...@gmail.com on 21 Jan 2011 at 2:23

GoogleCodeExporter commented 8 years ago 
Looks okay to me.

Original comment by neilboyd on 21 Jan 2011 at 10:27

GoogleCodeExporter commented 8 years ago 
This appears to fix the problem for me too, so I will close this ticket.

Original comment by kurtzm...@gmail.com on 24 Jan 2011 at 3:35