[make:decorator] Add new maker to create decorator

[WedgeSama]

Add new maker to create decorator.
#1401

Allow to create new decorator for existing services.

It try first to decorate by implements:

• The service ID used is an interface => implements this interface
• The service class implements one (or more) interface(s) and it do not declare more public method => implements those interfaces

If cannot implements, it will fallback to extends:

• Simply extends the service's class
• Do not work for final class

e.g.

bin/console make:decorator Symfony\Component\Routing\Generator\UrlGeneratorInterface MyUrlGenerator

<?php

namespace App;

use Symfony\Component\DependencyInjection\Attribute\AsDecorator;
use Symfony\Component\DependencyInjection\Attribute\AutowireDecorated;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Routing\RequestContext;

#[AsDecorator(UrlGeneratorInterface::class)]
final class MyUrlGenerator implements UrlGeneratorInterface
{
    public function __construct(
        #[AutowireDecorated]
        private readonly UrlGeneratorInterface $inner,
    ) {
    }

    public function generate(string $name, array $parameters = [], int $referenceType = self::ABSOLUTE_PATH): string
    {
        return $this->inner->generate($name, $parameters, $referenceType);
    }

    public function setContext(RequestContext $context): void
    {
        $this->inner->setContext($context);
    }

    public function getContext(): RequestContext
    {
        return $this->inner->getContext();
    }
}

[Neirda24]

based on the issue I'd suggest that the command accept not only the id but a classname and tries to guess the id based on it (if the service does not exists for example)

[WedgeSama]

I can try something like that:

1. If the class is use as ID => easy found
2. If no id with the classname, try to find one service using it
3. If more then one service, suggest to choose which service to decorate?
4. If nothing found, just exit with error

[Neirda24]

3 yes with fault tolerance maybe ? serializer vs serialiser There are example in symfony's code about this.

[WedgeSama]

Done. I also did guess from class/interface short name.

[Jean-Beru]

priorityand onInvalid could be added as options.

[WedgeSama]

Added

[Jean-Beru]

Maybe we can use this test to check different argument representations. Ex: without type, without default value, PHP8 attributes, variadic arguments, etc.)

[WedgeSama]

I test it here tests/Util/ClassSource/ClassMethodTest.php#L62
Is it enough?

[PierreCapel]

Suggested change

	return 'Create CRUD for Doctrine entity class';
	return 'Create a decorator of a service class';

[WedgeSama]

Done