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

Invalid IriConverterInterface injected in JsonLdIdTransformer #245

Closed
TheoD02 opened this issue Mar 11, 2025 · 3 comments · Fixed by #246
Closed

Invalid IriConverterInterface injected in JsonLdIdTransformer #245

TheoD02 opened this issue Mar 11, 2025 · 3 comments · Fixed by #246

Comments

@TheoD02
Copy link
Contributor

TheoD02 commented Mar 11, 2025

Description:

I encountered an error when setting the "api_platform" parameter to "true" in the bundle configuration. The error message refers to an "Invalid IriConverterInterface injected in JsonLdIdTransformer."

  • Steps to reproduce:

    1. Set the api_platform parameter to true in the bundle configuration.
    2. Use version 4.0 or 4.1 of the bundle.
    3. The same issue occurs when using versions 9.3.1 and 9.2.1 of automapper.
  • Expected behavior:
    The system should work correctly without any errors when the parameter is set to true.

  • Actual behavior:
    An error occurs due to an invalid injection of the IriConverterInterface into JsonLdIdTransformer.

  • Versions:

    • Bundle version: 4.0 / 4.1 (same issue with both versions)
    • Automapper versions: 9.3.1 / 9.2.1 (same issue with both versions)

I haven't had a chance to debug yet, but this issue persists across the aforementioned versions.

Error Screenshot

@lyrixx
Copy link
Member

lyrixx commented Mar 11, 2025

I started a branch for the support of apip 4. I already pushed it but I'm not sure it's ready yet. But you can try it if you want and give us some feedback

@TheoD02
Copy link
Contributor Author

TheoD02 commented Mar 11, 2025

Hello, thanks for the answer !

Just tested your branch, and it's work fine, didn't have the problem reported in this issue ! Thanks 😄

If you have need to tests other things tell me 😄

Let you close now, or wait the merge ?

@lyrixx
Copy link
Member

lyrixx commented Mar 12, 2025

see #246

Korbeil added a commit that referenced this issue Mar 12, 2025
- **Add support for API Platform 4** ; fix
#245
- **Add support for PHP 8.4**
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 a pull request may close this issue.

2 participants