codeconsole commented on PR #15165: URL: https://github.com/apache/grails-core/pull/15165#issuecomment-3438032666
> So, if I understand this correctly, before this change, the `date.format` and `default.date.format` message properties were used to decide the formatting. Now, it will always use `FormatStyle.SHORT` for both date and time + timezone. > > Also, should all these types be formatted with timezone? @matrei this ticket will be for coming up with a final solution https://github.com/apache/grails-core/issues/15164 I described the current behavior there: https://github.com/apache/grails-core/blob/c27742f9ad64b361ad39dabe9e090e938dc256ca/grails-fields/grails-app/taglib/grails/plugin/formfields/FormFieldsTagLib.groovy#L949-L973 currently `g.formatDate(date: model.value)` is called which result in no type so that results in a default format: https://github.com/apache/grails-core/blob/c27742f9ad64b361ad39dabe9e090e938dc256ca/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormatTagLib.groovy#L187 all this ticket does is set the type to datetime which results in the following behavior; https://github.com/apache/grails-core/blob/c27742f9ad64b361ad39dabe9e090e938dc256ca/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormatTagLib.groovy#L193-L201 -- 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]
