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

Dependency on abandoned sensio/framework-extra-bundle #2381

Open
speller opened this issue Jan 20, 2023 · 3 comments
Open

Dependency on abandoned sensio/framework-extra-bundle #2381

speller opened this issue Jan 20, 2023 · 3 comments

Comments

@speller
Copy link

speller commented Jan 20, 2023

The latest version 3.5.0 is dependent on the abandoned sensio/framework-extra-bundle. The sensio/framework-extra-bundle functionality is expected to be moved to Symfony core in v 6.2. But on 6.2, I'm getting the following error when I try using FOSRestBundle with Symfony 6.2 without FrameworkExtraBundle:

$ bin/console

In ConfigurationCheckPass.php line 30:

  You need to enable the parameter converter listeners in SensioFrameworkExtraBundle when using the FOSRestBundle RequestBodyParamConverter

The SensioFrameworkExtraBundle readme says:

SensioFrameworkExtraBundle is not maintained anymore. Please move to native PHP attribute support as added in Symfony core. For full support, use Symfony 6.2.

So I expect FOSRestBundle to work fine with Symfony 6.2 without SensioFrameworkExtraBundle.

@mbabker
Copy link
Contributor

mbabker commented Jan 20, 2023

#2354 already covers this issue, there's just been no action on it yet.

(Also keep in mind without a new major supporting only Symfony 6.2 and later, it wouldn't be possible to 100% decouple this bundle from the SensioFrameworkExtraBundle as that would break apps on the Symfony 5.4 LTS, which just makes a compat layer for a bundle like this one even more complex)

@speller
Copy link
Author

speller commented Feb 1, 2023

@mbabker It looks like a discussion thread. But I have an issue right now and expect that this issue will be fixed instead of just discussing it.

@jgauthi
Copy link

jgauthi commented Feb 9, 2023

You can continue to use SensioFrameworkExtraBundle for the moment.

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

No branches or pull requests

3 participants