Skip to content

fix(metadata): extract the hydra option of properties from YAML and XML - #8740

Open
Maxcastel wants to merge 1 commit into
api-platform:5.0from
Maxcastel:fix/property-extractors-hydra
Open

Maxcastel wants to merge 1 commit into
api-platform:5.0from
Maxcastel:fix/property-extractors-hydra

Conversation

@Maxcastel

Copy link
Copy Markdown
Contributor
Q A
Branch? 5.0
License MIT

hydra is in the properties XSD (added in #6748) but neither property extractor read it, so <property name="isbn" hydra="false"/> did nothing.

The compatibility test already had 'hydra' => false, it passed because assertEquals treats false and null as equal. I changed it to true, it fails without the fix.

@Maxcastel
Maxcastel changed the base branch from main to 5.0 October 19, 2026 10:00
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.

1 participant