FriendsOfSymfony / FOSRestBundle

This Bundle provides various tools to rapidly develop RESTful API's with Symfony
http://symfony.com/doc/master/bundles/FOSRestBundle/index.html
MIT License
2.79k stars 708 forks source link

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

Open speller opened 1 year ago

speller commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

@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 commented 1 year ago

You can continue to use SensioFrameworkExtraBundle for the moment.

rendler-denis commented 3 weeks ago

Hey colleagues! It is one and a half years later, is this still in discussions?