Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

java.util.Date support using Spring's CustomDateEditor #19

Merged
merged 7 commits into from
Apr 18, 2017
Merged

java.util.Date support using Spring's CustomDateEditor #19

merged 7 commits into from
Apr 18, 2017

Conversation

rahogata
Copy link
Contributor

Hi James, this change provide support for converting pre-defined date patterns to java.util.Date.

Changes included.

  • I used Spring's ConfigurableBeanFactory.getTypeConverter() to get default converter.
  • Inject Spring's date property editor (CustomDateEditor) in spring-defaultConfiguration.xml.
  • Modification of test class for autowiring.
    Please notify me if more changes has to do.

* 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
Copy link
Owner

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;
}
Copy link
Owner

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.

@rahogata
Copy link
Contributor Author

Hi James, committed changes you mentioned please review. I am very happy if you merge these changes.
Thank You.

Copy link
Owner

@jamesmorgan jamesmorgan left a 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

@jamesmorgan jamesmorgan merged commit abd54e8 into jamesmorgan:master Apr 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants