PropertyPlaceHolderConfigurer valueSeparator is not a configurable property and breaks compatibility with older configuration [SPR-7429]

[spring-projects-issues]

David Ward opened SPR-7429 and commented

Despite what it says in #9462 valueSeparator is NOT a configurable property and therefore we can't disable the new behaviour which attempts to process colon characters within unrecognized placeholders.

In Alfresco we have a bean definition like this, containing a value that looks like a placeholder and contains a colon character

<bean id="uk.org.itf.dm.runCheckReviewPeriodScriptAction"
class="org.alfresco.repo.action.scheduled.SimpleTemplateActionDefinition">
<property name="actionName">
<value>script</value>
</property>
<property name="parameterTemplates">
<map>
<entry>
<key>
<value>script-ref</value>
</key>
<value>${selectSingleNode('workspace://SpacesStore', 'lucene', 'PATH:"/app:company_home/app:dictionary/app:scripts/cm:checkReviewPeriodAndSendEmail.js"')}</value>
</entry>
</map>
</property>

In order to support this, I would like to have configured this on my PropertyPlaceHolderConfigurer

<property name="ignoreUnresolvablePlaceholders">
    <value>true</value>
</property>
<property name="valueSeparator">
    <null/>
</property>

But unfortunately I can't, because there's no setter method for valueSeparator.

---

Affects: 3.0 GA

Referenced from: commits 9ef47b8

[spring-projects-issues]

Juergen Hoeller commented

Oops, that was an oversight: We simply forgot to add the actual setter method.

Fixed for 3.0.4 - will be available in tonight's 3.0.4 snapshot.

Juergen