[
https://issues.apache.org/jira/browse/CAMEL-12334?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16398515#comment-16398515
]
ASF GitHub Bot commented on CAMEL-12334:
----------------------------------------
hemangajmera commented on issue #2254: CAMEL-12334: Mapped date to LocalDate
and time to LocalTime
URL: https://github.com/apache/camel/pull/2254#issuecomment-373007729
@zregvart Thanks for your feedback..
We are using ZonedDateTime, so DateTime is not an issue.
Time can be converted to OffsetTime, so that conversion to localTime can be
handled properly.
Regarding Date, I think it should be still LocalTime, since most of the
Business UseCases which I have seen, date should be date only. Common example
of dates are BirthDate, etc, where time is not significant, or should not be
considered. Let me know your thought,
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Salesforce DTO does not use correct datatype for "date" and "time" field
> ------------------------------------------------------------------------
>
> Key: CAMEL-12334
> URL: https://issues.apache.org/jira/browse/CAMEL-12334
> Project: Camel
> Issue Type: Improvement
> Components: camel-salesforce
> Affects Versions: 2.20.2
> Reporter: Hemang Ajmera
> Priority: Major
> Labels: maven
>
> The DTO classes generated by the Maven plugin uses ZonedDateTime for all three
> * datetime
> * date
> * time
> The generation works fine, but is is not able to marshal/unmarshal correctly.
> The DTO should use different kind of field. My suggestion is to use
> * ZonedDateTime for datetime
> * LocalDate for date
> * LocalTime for time
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
