AthiraHari77 commented on code in PR #6614:
URL:
https://github.com/apache/incubator-kie-drools/pull/6614#discussion_r2890559446
##########
kie-dmn/kie-dmn-feel/src/main/java/org/kie/dmn/feel/util/CodegenUtils.java:
##########
@@ -151,22 +153,10 @@ public static VariableDeclarationExpr
getObjectExpression(Object object, String
} else if (object instanceof String string) {
return
getVariableDeclaratorWithInitializerExpression(variableName, STRING_CT,
new
StringLiteralExpr(escapeJava(string)));
+ } else if (object instanceof CustomZonedDateTime customZonedDateTime) {
+ return getVariableDeclaratorWithZonedDateTime(variableName,
customZonedDateTime.getZonedDateTime());
Review Comment:
@gitgabrio **CustomZonedDateTime** falls through to the generic
TemporalAccessor handler which uses **toParsableString()** to re-parse it. But
the toParsableString() method is stripping out the date part. The issue is that
toParsableString() is optimal only for time values, not date-time values. Due
to this few scenarios like below fails.
**"date and time(\"2018-12-08T00:00:00\") = date and
time(\"2018-12-08T00:00:00@Europe/Rome\")"**
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
