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

Java time picker: use ISO date string #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Java time picker: use ISO date string #3

wants to merge 1 commit into from

Conversation

PreludeAndFugue
Copy link

I have a Polish user who has an android phone with polish localisation. This caused a bug in the app and I tracked it down to the value of date.toLocaleString() which is sent to the javascript callback resulting in an Invalid date in the javascript code. The changes I have made should avoid any localisation problems by using the ISO standard that javascript recognises.

This change is for the TimePicker, so the actual date values being sent to the javascript callback are irrelevant.

Javascript ISO standard

ECMAScript defines a string interchange format for date-times based upon
a simplification of the ISO 8601 Extended Format. The format is as
follows: YYYY-MM-DDTHH:mm:ss.sssZ

Use this date string format to avoid javascript Date parse error when
this string is passed to the javascript callback.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant