Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for Spring property replacement in webflow-config schema parser [SWF-1681] #858

Open
spring-operator opened this issue Feb 23, 2016 · 3 comments

Comments

@spring-operator
Copy link
Contributor

Scott Cantor opened SWF-1681 and commented

There is some support for Spring property replacement in the custom Spring XML parser that implements the webflow-config schema. For example, the webflow:flow-registry element passes through a BeanDefinitionBuilder, so Spring handles replacement of properties such as the base-path attribute.

It would be really useful to have that capability in the <flow-location> elements since that would allow outside injection of the paths to use for flows, but those elements are directly turned into FlowLocation objects, so there's no opportunity for Spring to handle the property replacement.

I can work on a patch for this if it would be considered for inclusion, it's just a matter of indirecting things through BeanDefinitionBuilders to give Spring a chance to do its work.


Affects: 2.4.2

Attachments:

@spring-operator
Copy link
Contributor Author

Rossen Stoyanchev commented

Yes a patch would be appreciated, thanks.

@spring-operator
Copy link
Contributor Author

Scott Cantor commented

Ok, will do, thank you.

@spring-operator
Copy link
Contributor Author

Scott Cantor commented

Patch attached that indirects all the object construction in that Spring parser. I tested the two cases we use, flow-location and flow-location-pattern, and properties work for both.

@rstoyanchev rstoyanchev modified the milestones: 2.5 Backlog, General Backlog Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants