lukaszlenart opened a new pull request, #1918:
URL: https://github.com/apache/struts/pull/1918

   Relates to [WW-3226](https://issues.apache.org/jira/browse/WW-3226).
   
   WW-3226 asked for an `overwrite` flag on `AliasInterceptor`. Triage showed 
the behaviour is already decided by interceptor ordering: `alias` and `params` 
both set the target property when they run, so whichever runs last wins. 
[struts-site#329](https://github.com/apache/struts-site/pull/329) documents 
that; this PR pins it with two tests so the documented promise is enforced:
   
   - `alias` before `params` (the `defaultStack` order) — the directly 
submitted parameter wins.
   - `params` before `alias` — the alias overrides the submitted parameter.
   
   Both tests also assert the source property bound, so a run where `params` 
binds nothing fails instead of passing vacuously. Test-only; no production 
change. The ticket itself is being closed as Won't Fix.
   
   Security review: clean — test-only diff, no shipped code touched.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to