Skip to content

Conversation

@alexndlm
Copy link
Contributor

@alexndlm alexndlm commented Dec 1, 2025

Q A
Branch? 7.4
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

This PR removes the redundant and often incorrect docblocks from transform() and reverseTransform() methods in data transformers. These PHPDocs duplicate the interface, break generics, and add unnecessary noise.
Additionally, parameter names are updated to match those defined in DataTransformerInterface, ensuring consistency across all implementations.

@carsonbot carsonbot added this to the 7.4 milestone Dec 1, 2025
@carsonbot carsonbot changed the title Clean up wrong method docblocks in data transformers Clean up wrong method docblocks in data transformers Dec 1, 2025
@carsonbot carsonbot changed the title Clean up wrong method docblocks in data transformers [Form] Clean up wrong method docblocks in data transformers Dec 1, 2025
@nicolas-grekas
Copy link
Member

Thank you @alexndlm.

@nicolas-grekas nicolas-grekas force-pushed the feature/clean-up-wrong-method-docblocks-in-data-transformers-new branch from 4763f3f to 269dc4a Compare December 4, 2025 17:16
@nicolas-grekas nicolas-grekas merged commit 56c348c into symfony:7.4 Dec 4, 2025
This was referenced Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants