JMSI18nRoutingBundle icon indicating copy to clipboard operation
JMSI18nRoutingBundle copied to clipboard

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

Open eotin opened this issue 3 years ago • 4 comments

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.

eotin avatar Aug 28 '21 11:08 eotin

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

gjuric avatar Sep 01 '21 07:09 gjuric

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

eotin avatar Sep 01 '21 07:09 eotin

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.

acasademont avatar Sep 01 '21 07:09 acasademont

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

ddonuts avatar Oct 05 '22 11:10 ddonuts