[TIMOB-2327] Android: Froyo ID is Non-Unique
GitHub Issue | n/a |
---|---|
Type | New Feature |
Priority | Medium |
Status | Closed |
Resolution | Fixed |
Resolution Date | 2011-04-17T01:58:46.000+0000 |
Affected Version/s | n/a |
Fix Version/s | Release 1.6.0 M05 |
Components | Android |
Labels | android, feature, release-1.6.0, rplist |
Reporter | Don Thorp |
Assignee | Don Thorp |
Created | 2011-04-15T03:16:48.000+0000 |
Updated | 2011-10-14T08:25:21.000+0000 |
Description
There is a bug in Froyo devices and we should probably just generate an ID for each app on device and give up on relying on the platform ID.
If device returns platform id
9774d56d682e549c
create a new UUID and put it in the analytics db. http://code.google.com/p/android/issues/detail?id=10603">Google code bug 10603This is a little more involved than originally anticipated. Basically we need to detect any change in ID and report both the previous and new platform ID. We have to take into account that a vendor changes the platform id for whatever reason. For example, if motorola pushes a patch to this problem to generate a "correct" ID. If a platform doesn't have an ID and we generate a UUID, we need to detect an ID change if the manufacturer begins shipping an updated OS where the ID now exists.
Moving to M05 need to test with Nika
We'll talk about this one in MV. We need to work with Nika to confirm.
(from [d25452b03140567d7855ce5d196c2ecffbb9f81d]) [#2327 state:fixed-in-qa] replace bad machine ids on android
Changed initialize method in TiPlatformHelper to check for bad IDs and replace them with a generated ID. Made change to App.java template in order to invoke the initialize method from the onCreate once the AppInfo has been loaded. Made change to DatabaseHelper to support update calls. Modified analytics enroll message to include the previous machine ID so the change from old to new ids can be tracked.
https://github.com/appcelerator/titanium_mobile/commit/d25452b03140567d7855ce5d196c2ecffbb9f81d"> https://github.com/appcelerator/titanium_mobile/commit/d25452b03140...
Verified with Nika. Difficult to automate so including the enroll events received by Nika for the 4 test cases.
"
"closing based on Opies comments