If you use the following example, you will get an exception:
ServiceNotFoundException: The service "twig" in the container provided to "App\Service\Dependency" has a dependency on a non-existent service "Twig\Environment".

Thenullable argument of theSubscribedService attribute is ignored.

// src/Service/TwigAware.phpnamespaceApp\Service;useTwig\Environment;useSymfony\Contracts\Service\Attribute\SubscribedService;trait TwigAware{   #[SubscribedService('twig', nullable:true)]privatefunctiontwig():Environment    {if (!$this->container->has('twig') ) {thrownew \LogicException(\sprintf('Twig is required to use "%s" method. Try to run "composer require symfony/twig-bundle".',__METHOD__));        }return$environment;    }}// src/Service/MyService.phpnamespaceApp\Service;useSymfony\Contracts\Service\ServiceSubscriberInterface;useSymfony\Contracts\Service\ServiceSubscriberTrait;class MyServiceimplements ServiceSubscriberInterface{use ServiceSubscriberTrait, TwigAware;publicfunctiondoWithTwig():void    {// $this->twig() ...    }}

Related to#60265

carsonbot added Status: Needs Review Bug labels

Apr 25, 2025

carsonbot added this to the7.2 milestone

Apr 25, 2025

StevenRenaux force-pushed thefix/Fix-nullable-argument-for-SubscribedService-attribute branch from1a9c3a2 to7d7e4a0Compare

April 25, 2025 09:12

welcoMattic requested a review fromnicolas-grekas

April 25, 2025 09:28

StevenRenaux force-pushed thefix/Fix-nullable-argument-for-SubscribedService-attribute branch 2 times, most recently fromdb4b606 to562600fCompare

April 25, 2025 09:33

Fix ServiceMethodsSubscriberTrait for nullable service

e67db9a

StevenRenaux force-pushed thefix/Fix-nullable-argument-for-SubscribedService-attribute branch from562600f toe67db9aCompare

April 25, 2025 09:37

Copy link

Member

stof commentedApr 25, 2025

#60268 should be merged first and upmerged to 7.2, so that this PR can then be only about the new things for 7.2

OskarStark added the Contracts label

Apr 25, 2025

carsonbot changed the title~~FixServiceMethodsSubscriberTrait for nullable service~~[Contracts] FixServiceMethodsSubscriberTrait for nullable service

Apr 25, 2025

Copy link

ContributorAuthor

StevenRenaux commentedApr 25, 2025•
edited
Loading

#60268 should be merged first and upmerged to 7.2, so that this PR can then be only about the new things for 7.2

@stof
I'm agree but since 6.4 the tests about ServiceSubscriberTraits was updated, you can see with LegacyParentTestService. Or we need to updated it after the merge.

Let me know