PropertyPlaceholder does not work for @ImportResource in JavaConfig [SPR-6645]

[spring-projects-issues]

Liu, Yinwei David opened SPR-6645 and commented

PropertyPlaceholder fails to replace the placeholder in the XML Config files when they are in @ImportResource annotation of a JavaConfig.

I have one test which is a JavaConfig class and it uses @ImportResource to import one XML Config file importContext.xml, however, PropertyPlaceholder does not replace any placeholder (e.g. ${TEST}) in importContext.xml. Next, I add <import resource="importContext.xml"> in the root XML Config file, but PropertyPlaceholder still does not work. Finally, I remove @ImportResource from my JavaConfig class, and PropertyPlaceholder works.

I think it is a bug in JavaConfig. Please see my test case in the attachment.

---

Affects: 3.0 GA

Attachments:

• prop.zip (1.55 kB)

Issue Links:

• AnnotationConfigApplicationContext does not detect BeanFactoryPostProcessor [SPR-6455] #11121 AnnotationConfigApplicationContext does not detect BeanFactoryPostProcessor

1 votes, 2 watchers

[spring-projects-issues]

Liu, Yinwei David commented

Attach my test case.

[spring-projects-issues]

Juergen Hoeller commented

This should have been fixed along with #11121.

Juergen

[spring-projects-issues]

Liu, Yinwei David commented

Hi Juergen,

I download the latest snapshot spring-framework-3.0.1.CI-565, however, the bug still there. If you run the test case that I attached in the jira, you can see the bean importBean still has ${TEST} as its value, and ${TEST}was not replaced by its property.

Could you please check this issue? Thanks.

David

[spring-projects-issues]

Juergen Hoeller commented

David, your test works fine for me with the current HEAD version. Would be great if you could give it another try with tomorrow's 3.0.1 snapshot...

Juergen

[spring-projects-issues]

Liu, Yinwei David commented

Thanks Juergen. I tried the latest snapshot, and it works fine.

David