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
Timob 14060: Parity: Making the app background on Android does not send ti.end event #4345
Conversation
@@ -37,7 +37,7 @@ | |||
private static final String TAG = "TitaniumAnalyticsEvent"; | |||
|
|||
private static TimeZone utc = TimeZone.getTimeZone("UTC"); | |||
private static SimpleDateFormat isoDateFormatter = | |||
public static SimpleDateFormat isoDateFormatter = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be public. It should be private and final.
Updated. |
return; | ||
} | ||
} catch (ParseException e) { | ||
Log.e(TAG, "Incorrect timestamp. Unable to send the ti.start event.", e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't we return here instead of adding the event ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although this error should not happen, I still add this event to the DB and send it to the server. Then the server side can decide to use it or discard it.
Code reviewed. Please address comments. |
…ys were inserted into the map.
Functionally Tested and APPROVED |
Code reviewed. Request accepted |
Timob 14060: Parity: Making the app background on Android does not send ti.end event
https://jira.appcelerator.org/browse/TIMOB-14060