Skip to content

Conversation

@s2b
Copy link
Contributor

@s2b s2b commented Aug 15, 2025

This patch introduces new API methods on the RenderingContext to
obtain and replace Fluid's argument processor implementation. By
default, the StrictArgumentProcessor is used, but the
ArgumentProcessorInterface exists to provide an alternative
implementation. However, it might be a good idea wrap custom
implementations around the StrictArgumentProcessor by using
the decorator design pattern to ensure valid input types
to ViewHelpers and components.

Resolves: #1105

@s2b s2b marked this pull request as draft August 15, 2025 16:58
@s2b s2b force-pushed the feature/argumentProcessorApi branch 6 times, most recently from 7384cb0 to 289919a Compare August 20, 2025 16:58
This patch introduces new API methods on the RenderingContext to
obtain and replace Fluid's argument processor implementation. By
default, the StrictArgumentProcessor is used, but the
ArgumentProcessorInterface exists to provide an alternative
implementation. However, it might be a good idea wrap custom
implementations around the StrictArgumentProcessor by using
the decorator design pattern to ensure valid input types
to ViewHelpers and components.

This is a breaking feature because of the interface change.

Resolves: #1105
@s2b s2b force-pushed the feature/argumentProcessorApi branch from 289919a to 973449e Compare August 20, 2025 17:02
@s2b s2b changed the title WIP: [FEATURE] Ability to replace argument processor [!!!][FEATURE] Ability to replace argument processor Aug 20, 2025
@s2b s2b marked this pull request as ready for review August 20, 2025 17:03
@s2b s2b merged commit 21ac619 into main Aug 20, 2025
10 checks passed
@s2b s2b deleted the feature/argumentProcessorApi branch August 20, 2025 17:11
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.

Switch to StrictArgumentProcessor, provide API

3 participants