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.

Sign up for GitHub

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

Jump to bottom

Support for Java 8 Date/Time API in parameters #125

Closed
dtaimanov opened this issue Oct 23, 2020 · 3 comments
Closed

Support for Java 8 Date/Time API in parameters #125

dtaimanov opened this issue Oct 23, 2020 · 3 comments
Assignees
@dtaimanov @lovtsovaik
Labels
type: enhancement New feature or request
Milestone
Release 7.2

Comments

@dtaimanov
Copy link
Contributor

dtaimanov commented Oct 23, 2020 •
edited
Loading

Environment:

  • Platform version: 7.2
  • Addon version: 7.2

Description:

Detais: https://www.cuba-platform.com/discuss/t/rest-api-support-for-java-8-time-module-java-time/13874

Technical description: RestParseUtils#toObject(Type,String,String) doesn't support LocalDate and other Java 8 Date/Time API types.
Support could be implemented either on RestParseUtils or on EntitySerializationAPI level.
@dtaimanov dtaimanov added the type: enhancement New feature or request label Oct 23, 2020
@knstvk knstvk added this to the Release 7.2 milestone Oct 26, 2020
@bartekspitza bartekspitza mentioned this issue Oct 29, 2020
Closed
@knstvk
Copy link
Member

knstvk commented Feb 4, 2021

The same problem with query parameters: https://www.cuba-platform.com/discuss/t/rest-query-on-localdate-possible-bug/14672

@andreysubbotin andreysubbotin assigned plakhov and unassigned t2-cuba Feb 18, 2021
plakhov added a commit that referenced this issue Feb 24, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
0592229
plakhov added a commit that referenced this issue Feb 26, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
fcec1a2
plakhov added a commit that referenced this issue Feb 26, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
1101341
@plakhov plakhov mentioned this issue Mar 1, 2021
Closed
@lovtsovaik
Copy link

Need to support also java.sql.Time

plakhov added a commit that referenced this issue Mar 3, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
52a0dd5 
add support java.sql.Time
plakhov added a commit that referenced this issue Mar 3, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
9f41585 
add support java.sql.Time
plakhov added a commit that referenced this issue Mar 3, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
10d3cb9 
add support java.sql.Time
plakhov added a commit that referenced this issue Mar 4, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
e6ab4a6 
fix tests
plakhov added a commit that referenced this issue Mar 4, 2021
@plakhov
Support for Java 8 Date/Time API in parameters #125
81f9083 
fix tests
@lovtsovaik
Copy link

Examples of values:

  • java.time.LocalDate: "2010-10-10",
  • java.time.LocalDateTime: "2010-10-10 15:15:15.001",
  • java.time.LocalTime: "15:15:15",
  • java.time.OffsetDateTime: "2010-10-10 15:15:15.001 +0300",
  • java.time.OffsetTime: "15:15:15 +0300"

Examples of values for not java.time.*:

  • java.sql.Time: "15:15:15",
  • java.util.Date: "2010-10-10"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dtaimanov dtaimanov

@lovtsovaik lovtsovaik

Labels
type: enhancement New feature or request
Projects
None yet
Milestone
Release 7.2
Development

No branches or pull requests
7 participants
@knstvk @plakhov @natfirst @MikhailSilk @dtaimanov @t2-cuba @lovtsovaik