makroz/android-protips-location

Eclair-specific code is broken

GoogleCodeExporter opened this issue · 0 comments

It looks like there are a couple of issues with the Eclair-specific code:

1) The booleans to enable/disable the services in the manifest file are 
reversed:

<service android:enabled="@bool/pre_froyo" 
android:name=".services.PlacesUpdateService"> and <service 
android:enabled="@bool/froyo_or_later" 
android:name=".services.EclairPlacesUpdateService">

should be

<service android:enabled="@bool/froyo_or_later" 
android:name=".services.PlacesUpdateService"> and <service 
android:enabled="@bool/pre_froyo" 
android:name=".services.EclairPlacesUpdateService">

2) Several classes do

PlacesConstants.SUPPORTS_ECLAIR ? EclairPlacesUpdateService.class : 
PlacesUpdateService.class

but should do 

PlacesConstants.SUPPORTS_FROYO ? PlacesUpdateService.class : 
EclairPlacesUpdateService.class

3) LegacyPlacesUpdateRequester is never instantiated in 
PlatformSpecificImplementationFactory.getLocationUpdateRequester()

Original issue reported on code.google.com by lit...@gmail.com on 5 Oct 2011 at 11:23