sonata-project / SonataAdminBundle

The missing Symfony Admin Generator
https://docs.sonata-project.org/projects/SonataAdminBundle
MIT License
2.11k stars 1.26k forks source link

Check `service_container` usages #6963

Closed franmomu closed 3 years ago

franmomu commented 3 years ago

Feature Request

We are using service_container in:

VincentLanglet commented 3 years ago

Speaking on SonataAdminExtension, the only remaining method is https://github.com/sonata-project/SonataAdminBundle/blob/b737b64feea99be925af9b970bc42dca26d7587f/src/Twig/Extension/SonataAdminExtension.php#L64

But I find no usage, do you think we can deprecate the full extension ? @franmomu

franmomu commented 3 years ago

The method is not directly used, but it's used as a filter: https://github.com/search?q=org%3Asonata-project+sonata_urlsafeid&type=code.

In some places there is no admin, so I guess we should keep it for now.

jordisala1991 commented 3 years ago

But we have another filter called this same way: https://github.com/sonata-project/twig-extensions/blob/1.x/src/Extension/TemplateExtension.php

Which one are we using?

franmomu commented 3 years ago

Interesting, I guess it depends on the project, on which bundle it is registered last.

VincentLanglet commented 3 years ago

This issue can be closed. But should we open one about the sonata_urlsafeid duplicate ?