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

Provide Converters for Java 8 Date and Time API (JSR 310) #1370

Closed
eclipse-faces-bot opened this issue May 1, 2015 · 12 comments
Closed

Provide Converters for Java 8 Date and Time API (JSR 310) #1370

eclipse-faces-bot opened this issue May 1, 2015 · 12 comments

Comments

@eclipse-faces-bot
Copy link

Apologies if I have missed an existing issue but I'm trying to determine how the convertDateTime converters will be aligned to support Java 8 Date and Time API in JSF 2.3

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Reported by codylerum

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Issue-Links:
is duplicated by
JAVASERVERFACES_SPEC_PUBLIC-1272
is related to
JAVASERVERFACES-4070

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
codylerum said:
I meant to do this as a SPEC issue. Can it be moved?

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@manfredriem said:
Done

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Yes, this is a must for JSF 2.3.

EG member Josh Juneau wrote a blog post about it, but to actually get it in the spec we need a lot more work: < http://www.javacodegeeks.com/2015/06/utilizing-the-java-8-date-time-api-with-jsf-and-java-ee-7.html >.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Some questions about putting this in the spec.

1. What do we need to say in section 3.3.3 Standard Converter Implementations?

2. Which of java.time.

{LocalDate,LocaleTime,LocalDateTime}

do we want to support? All of them I'd think, but how does this square with the JSF conversion model which seems to favor the combined "DateTime" concept.

3. Perhaps we should create a new converter along side DateTimeConverter, say JavaTimeDateTimeConverter and trick it out with lots of attributes as the current DateTimeConverter does?

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
javajuneau said:
Actively working on prototype of LocalDateTime conversion.

Creating a new converter, as suggested by Ed Burns, and utilizing the "type" attribute of the convertDateTime tag to indicate the conversion of "localDateTime".

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
@edburns said:
Josh! I created a topic branch for this, it is JAVASERVERFACES-4070. Please pull that branch and work from there.

I've made some good progress already.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
javajuneau said:
Thanks Ed, will do! I appreciate it.

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
Marked as fixed on Monday, October 5th 2015, 3:43:41 pm

@eclipse-faces-bot
Copy link
Author

@glassfishrobot Commented
This issue was imported from java.net JIRA JAVASERVERFACES_SPEC_PUBLIC-1370

@eclipse-faces-bot
Copy link
Author

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