schmittjoh / JMSI18nRoutingBundle

Allows you to internationalize your routing
http://jmsyst.com/bundles/JMSI18nRoutingBundle
358 stars 159 forks source link

The "Psr\Container\ContainerInterface" autowiring alias is deprecated #249

Open eotin opened 2 years ago

eotin commented 2 years ago

From Symfony 5.1, I have this deprecation message but can't figure out if i can fix it without a new version of this bundle ?

Thanks for your help

Since symfony/dependency-injection 5.1: The "Psr\Container\ContainerInterface" autowiring alias is deprecated. Define it explicitly in your app if you want to keep using it. It is being referenced by the "jms_i18n_routing.router" service.

gjuric commented 2 years ago

Hi @eotin , #250 fixes this for me, can you test it out as well?

eotin commented 2 years ago

Hi @eotin , #250 fixes this for me, can you test it out as well?

Hi @gjuric , yes i confirm that #250 fixes the deprecation message

acasademont commented 2 years ago

Hi all!

Thank you for the patch! I'll take a look at it asap ;)

On Wed, Sep 1, 2021 at 9:37 AM eotin @.***> wrote:

Hi @eotin https://github.com/eotin , #250 https://github.com/schmittjoh/JMSI18nRoutingBundle/pull/250 fixes this for me, can you test it out as well?

Hi @gjuric https://github.com/gjuric , yes i confirm that #250 https://github.com/schmittjoh/JMSI18nRoutingBundle/pull/250 fixes the deprecation message

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/schmittjoh/JMSI18nRoutingBundle/issues/249#issuecomment-910021585, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGJNPKEBAPPN5J5S5RBZPTT7XJZ5ANCNFSM5C7BOCYQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

ddonuts commented 1 year ago

Hi all, I'm also very interested in the fix. Is there a plan to merge it soon? Thanks