Skip to content

fix(metadata): use operation output class for mapping instead of operation class - #7601

Merged
soyuka merged 1 commit into
api-platform:4.2from
Spomky:bugs/mapped-dto
Dec 11, 2025
Merged

soyuka merged 1 commit into
api-platform:4.2from
Spomky:bugs/mapped-dto

Conversation

@Spomky

@Spomky Spomky commented Dec 10, 2025

Copy link
Copy Markdown
Contributor
Q A
Branch? 4.2
Tickets Closes #7593
License MIT
Doc PR not needed

As shown in #7593, the output class is not used when the Object Mapper is enabled. It appears that the ObjectMapperProvider always relies on the resource class instead.
This PR fixes the issue in the same way it is already handled in ObjectMapperProcessor.

@Spomky

Spomky commented Dec 10, 2025

Copy link
Copy Markdown
Contributor Author

Please wait before merging this PR. There is another issue now and I cannot figure out why.
The @id is now /.well-known/genid/[hex value] instead of the correct resource IRI.
Not sure why...

@soyuka
soyuka merged commit d3bcded into api-platform:4.2 Dec 11, 2025
127 of 130 checks passed
@soyuka

soyuka commented Dec 11, 2025

Copy link
Copy Markdown
Member

thanks!

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