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

Fails when using it on android #3

Open
GoogleCodeExporter opened this issue Mar 17, 2015 · 3 comments
Open

Fails when using it on android #3

GoogleCodeExporter opened this issue Mar 17, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

I am making games that run on the desktop as well as on android.

This library fails to work on android because when loading the 
com.dmurph.tracking.AnalyticsConfigData class and running validation on it, it 
finds the calls to awt that are not present on android, so it fails to load the 
class.

I think splitting this class so the populateFromSystem method is in another 
class that wouldn't be loaded when using it from android would allow this 
library to be used on android.

I am not using the official google analytics for android library because I want 
my code to be common between desktop and android, and the android library won't 
work on the desktop.



Original issue reported on code.google.com by ruben01 on 24 May 2011 at 3:39

@GoogleCodeExporter
Copy link
Author

I created a fork on github, to try and fix this.

https://github.com/gemserk/jgoogleanalyticstracker/commit/c712b9c29aa5192926c2c7
89f13d884080885695

this is the change I made, it loads fine on android if you avoid calling the 
autoconfiguration for the desktop now.

will know for sure if it is working fine when the data is updated on the 
analytics site

Original comment by ruben01 on 24 May 2011 at 4:14

@GoogleCodeExporter
Copy link
Author

can confirm now that the library with this change works fine on android
at least the trackPageView feature

Original comment by ruben01 on 24 May 2011 at 4:49

@GoogleCodeExporter
Copy link
Author

hm, I'll take a look, I might take a different approach

Original comment by toucansa...@gmail.com on 28 May 2011 at 6:43

  • Changed state: Started

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