Skip to content

WW-5524 Fixup StrutsConverterFactory - Struts 6 - #1316

Merged
lukaszlenart merged 1 commit into
release/struts-6-7-xfrom
fix/WW-5524-converter-factory-s6
Aug 20, 2025
Merged

WW-5524 Fixup StrutsConverterFactory - Struts 6#1316
lukaszlenart merged 1 commit into
release/struts-6-7-xfrom
fix/WW-5524-converter-factory-s6

Conversation

@lukaszlenart

@lukaszlenart lukaszlenart commented Aug 20, 2025

Copy link
Copy Markdown
Member

It should delegate back to ObjectFactory#buildBean instead of directly calling Container#inject, otherwise overrides of buildBean in subclasses of ObjectFactory e.g. SpringObjectFactory are skipped; meaning that TypeConverters cannot make use of Spring dependency injection

Closes WW-5524

@lukaszlenart lukaszlenart changed the title WW-5524 Fixup StrutsConverterFactory WW-5524 Fixup StrutsConverterFactory - Struts 6 Aug 20, 2025
It should delegate back to ObjectFactory#buildBean instead of directly
calling Container#inject, otherwise overrides of buildBean in subclasses
of ObjectFactory e.g. SpringObjectFactory are skipped; meaning that
TypeConverters cannot make use of Spring dependency injection

Fixes: https://issues.apache.org/jira/projects/WW/issues/WW-5524
@lukaszlenart
lukaszlenart force-pushed the fix/WW-5524-converter-factory-s6 branch from 92a2a4b to a925684 Compare August 20, 2025 04:48
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
22 Security Hotspots
42.3% Coverage on New Code (required ≥ 80%)
3.4% Duplication on New Code (required ≤ 3%)
E Security Rating on New Code (required ≥ A)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@lukaszlenart
lukaszlenart merged commit f253216 into release/struts-6-7-x Aug 20, 2025
9 of 10 checks passed
@lukaszlenart
lukaszlenart deleted the fix/WW-5524-converter-factory-s6 branch August 20, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants