-
Notifications
You must be signed in to change notification settings - Fork 55
java.util.Date support using Spring's CustomDateEditor #19
Conversation
Description: 1) add java.util.Date support.
* Replace callback Properties EventHandler with Guava EventBus | ||
* Ability to configure usage via spring's @Configuration | ||
|
||
### Contributions ### | ||
* Thankyou [normanatashbar](https://github.com/normanatashbar) for adding composite string replacement | ||
* Thank you [normanatashbar](https://github.com/normanatashbar) for adding composite string replacement |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fee free to add you name as a contributor
static class TestObject { | ||
Period period = new Period(); | ||
LocalTime localTime = new LocalTime(); | ||
LocalDate localDate = new LocalDate(); | ||
LocalDateTime localDateTime = new LocalDateTime(); | ||
Date dateValue = new Date(); | ||
boolean booleanValue; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - can you enhance the integration test ReloadablePropertyPostProcessorIntTest
just to prove it all wires together correct and properties can be injected. Thanks for the contribution.
Hi James, committed changes you mentioned please review. I am very happy if you merge these changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - will give it a quick test tomorrow morning and then I'll merge 👍 thanks
Hi James, this change provide support for converting pre-defined date patterns to java.util.Date.
Changes included.
ConfigurableBeanFactory.getTypeConverter()
to get default converter.spring-defaultConfiguration.xml
.Please notify me if more changes has to do.