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

Eclair-specific code is broken #9

Open
GoogleCodeExporter opened this issue Jun 29, 2015 · 0 comments
Open

Eclair-specific code is broken #9

GoogleCodeExporter opened this issue Jun 29, 2015 · 0 comments

Comments

@GoogleCodeExporter
Copy link

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

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

No branches or pull requests

1 participant