Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug in WriteListener.php #1

Merged
merged 1 commit into from
Apr 19, 2019
Merged

Fix bug in WriteListener.php #1

merged 1 commit into from
Apr 19, 2019

Conversation

slavo2
Copy link
Owner

@slavo2 slavo2 commented Apr 19, 2019

Alternative to api-platform#2283

When calling POST on a non ORM resource, tha call failed with 400: "Unable to generate an IRI for the item of type..."
on api-platform/core/src/Bridge/Symfony/Routing/IriConverter.php line 131

This is supposed to fix it, as it tries to get the IRI from inserted resource, not from original data (where it does not exist in a POST).

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? I don't know, I did not run them
Fixed tickets
License MIT
Doc PR api-platform/docs#1234

When calling POST on a non ORM resource, tha call failed with 400: "Unable to generate an IRI for the item of type..."
on api-platform/core/src/Bridge/Symfony/Routing/IriConverter.php line 131

This is supposed to fix it, as it tries to get the IRI from inserted resource, not from original data (where it does not exist in a POST).
@slavo2 slavo2 merged commit 1ca4fe5 into 2.4 Apr 19, 2019
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