Up PHP code style to 8.1

src/Assigner/ProductsAssigner.php

@@ -9,9 +9,6 @@
 
 final class ProductsAssigner implements ProductsAssignerInterface
 {
-    /**
-     * @inheritdoc
-     */
     public function assign(BrandInterface $brand, array $products): void
     {
         foreach ($products as $product) {

src/DependencyInjection/Configuration.php

@@ -21,9 +21,6 @@
 
 final class Configuration implements ConfigurationInterface
 {
-    /**
-     * @inheritdoc
-     */
     public function getConfigTreeBuilder(): TreeBuilder
     {
         if (method_exists(TreeBuilder::class, 'getRootNode')) {

src/DependencyInjection/LoevgaardSyliusBrandExtension.php

@@ -12,8 +12,6 @@
 final class LoevgaardSyliusBrandExtension extends AbstractResourceExtension
 {
     /**
-     * @inheritdoc
-     *
      * @throws \Exception
      */
     public function load(array $config, ContainerBuilder $container): void

src/Doctrine/ORM/BrandImageRepository.php

@@ -10,9 +10,6 @@
 
 class BrandImageRepository extends EntityRepository implements BrandImageRepositoryInterface
 {
-    /**
-     * @inheritdoc
-     */
     public function createListQueryBuilder(string $brandCode): QueryBuilder
     {
         return $this->createQueryBuilder('o')
@@ -22,9 +19,6 @@ public function createListQueryBuilder(string $brandCode): QueryBuilder
         ;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function createPaginatorForBrandAndType(BrandInterface $brand, string $type): iterable
     {
         $queryBuilder = $this->createQueryBuilder('o')

src/Doctrine/ORM/BrandRepository.php

@@ -8,9 +8,6 @@
 
 class BrandRepository extends EntityRepository implements BrandRepositoryInterface
 {
-    /**
-     * @inheritdoc
-     */
     public function findByPhrase(string $phrase): array
     {
         return $this->createQueryBuilder('o')

src/Doctrine/ORM/ProductRepositoryTrait.php

@@ -17,9 +17,6 @@ trait ProductRepositoryTrait
      */
     abstract public function createQueryBuilder($alias, $indexBy = null);
 
-    /**
-     * @inheritdoc
-     */
     public function createPaginatorForBrand(BrandInterface $brand): iterable
     {
         return $this->createQueryBuilder('o')

src/Event/BrandMenuBuilderEvent.php

@@ -11,14 +11,9 @@
 
 class BrandMenuBuilderEvent extends MenuBuilderEvent
 {
-    /** @var BrandInterface */
-    private $brand;
-
-    public function __construct(FactoryInterface $factory, ItemInterface $menu, BrandInterface $brand)
+    public function __construct(FactoryInterface $factory, ItemInterface $menu, private readonly BrandInterface $brand)
     {
         parent::__construct($factory, $menu);
-
-        $this->brand = $brand;
     }
 
     public function getBrand(): BrandInterface

src/EventListener/ImageUploadListener.php

@@ -11,12 +11,8 @@
 
 final class ImageUploadListener
 {
-    /** @var ImageUploaderInterface */
-    private $uploader;
-
-    public function __construct(ImageUploaderInterface $uploader)
+    public function __construct(private readonly ImageUploaderInterface $uploader)
     {
-        $this->uploader = $uploader;
     }
 
     public function uploadImage(ResourceControllerEvent $event): void

src/Factory/BrandImageFactory.php

@@ -10,17 +10,10 @@
 
 class BrandImageFactory implements BrandImageFactoryInterface
 {
-    /** @var FactoryInterface */
-    private $factory;
-
-    public function __construct(FactoryInterface $factory)
+    public function __construct(private readonly FactoryInterface $factory)
     {
-        $this->factory = $factory;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function createNew(): BrandImageInterface
     {
         /** @var BrandImageInterface $brandImage */
@@ -29,9 +22,6 @@ public function createNew(): BrandImageInterface
         return $brandImage;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function createForBrand(BrandInterface $brand): BrandImageInterface
     {
         /** @var BrandImageInterface $brandImage */

src/Fixture/Factory/BrandExampleFactory.php

@@ -103,7 +103,7 @@ protected function createImages(BrandInterface $brand, array $options): void
             if (is_array($imagePath)) {
                 $imagePath = $imagePath[array_key_first($imagePath)];
             }
-            $uploadedImage = new UploadedFile($imagePath, basename($imagePath));
+            $uploadedImage = new UploadedFile($imagePath, basename((string) $imagePath));
 
             /** @var BrandImageInterface $brandImage */
             $brandImage = $this->productImageFactory->createNew();

src/Form/Extension/Api/ProductTypeExtension.php

@@ -11,9 +11,6 @@
 
 class ProductTypeExtension extends AbstractTypeExtension
 {
-    /**
-     * @inheritdoc
-     */
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->add('brand', BrandChoiceType::class, [
@@ -23,9 +20,6 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
         ]);
     }
 
-    /**
-     * @inheritdoc
-     */
     public static function getExtendedTypes(): iterable
     {
         return [

src/Form/Extension/ProductTypeExtension.php

@@ -11,9 +11,6 @@
 
 class ProductTypeExtension extends AbstractTypeExtension
 {
-    /**
-     * @inheritdoc
-     */
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->add('brand', BrandAutocompleteChoiceType::class, [
@@ -23,9 +20,6 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
         ]);
     }
 
-    /**
-     * @inheritdoc
-     */
     public static function getExtendedTypes(): iterable
     {
         return [

src/Form/Type/BrandAutocompleteChoiceType.php

@@ -12,9 +12,6 @@
 
 final class BrandAutocompleteChoiceType extends AbstractType
 {
-    /**
-     * @inheritdoc
-     */
     public function configureOptions(OptionsResolver $resolver): void
     {
         $resolver->setDefaults([
@@ -24,26 +21,17 @@ public function configureOptions(OptionsResolver $resolver): void
         ]);
     }
 
-    /**
-     * @inheritdoc
-     */
     public function buildView(FormView $view, FormInterface $form, array $options): void
     {
         $view->vars['remote_criteria_type'] = 'contains';
         $view->vars['remote_criteria_name'] = 'phrase';
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getBlockPrefix(): string
     {
         return 'loevgaard_sylius_brand_autocomplete_choice';
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getParent(): string
     {
         return ResourceAutocompleteChoiceType::class;

src/Form/Type/BrandChoiceType.php

@@ -14,50 +14,32 @@
 
 final class BrandChoiceType extends AbstractType
 {
-    /** @var RepositoryInterface */
-    private $brandRepository;
-
-    public function __construct(RepositoryInterface $brandRepository)
+    public function __construct(private readonly RepositoryInterface $brandRepository)
     {
-        $this->brandRepository = $brandRepository;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         if ($options['multiple']) {
             $builder->addModelTransformer(new CollectionToArrayTransformer());
         }
     }
 
-    /**
-     * @inheritdoc
-     */
     public function configureOptions(OptionsResolver $resolver): void
     {
         $resolver->setDefaults([
-            'choices' => function (Options $options) {
-                return $this->brandRepository->findAll();
-            },
+            'choices' => fn (Options $options) => $this->brandRepository->findAll(),
             'choice_value' => 'code',
             'choice_label' => 'name',
             'choice_translation_domain' => false,
         ]);
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getParent(): string
     {
         return ChoiceType::class;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getBlockPrefix(): string
     {
         return 'loevgaard_sylius_brand_choice';

src/Form/Type/BrandImageType.php

@@ -8,9 +8,6 @@
 
 final class BrandImageType extends ImageType
 {
-    /**
-     * @inheritdoc
-     */
     public function getBlockPrefix(): string
     {
         return 'loevgaard_sylius_brand_brand_image';

src/Form/Type/BrandType.php

@@ -12,9 +12,6 @@
 
 final class BrandType extends AbstractResourceType
 {
-    /**
-     * @inheritdoc
-     */
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder
@@ -33,9 +30,6 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             ]);
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getBlockPrefix(): string
     {
         return 'loevgaard_sylius_brand_brand';

src/LoevgaardSyliusBrandPlugin.php

@@ -12,9 +12,6 @@ final class LoevgaardSyliusBrandPlugin extends AbstractResourceBundle
 {
     use SyliusPluginTrait;
 
-    /**
-     * @inheritdoc
-     */
     public function getSupportedDrivers(): array
     {
         return [

src/Menu/BrandFormMenuBuilder.php

@@ -16,17 +16,12 @@ final class BrandFormMenuBuilder
 {
     public const EVENT_NAME = 'loevgaard_sylius_brand.menu.admin.brand.form';
 
-    /** @var FactoryInterface */
-    private $factory;
-
     /** @var EventDispatcherInterface */
     private $eventDispatcher;
 
-    public function __construct(FactoryInterface $factory, EventDispatcherInterface $eventDispatcher)
+    public function __construct(private readonly FactoryInterface $factory, EventDispatcherInterface $eventDispatcher)
     {
-        $this->factory = $factory;
-
-        if (class_exists('Symfony\Component\EventDispatcher\LegacyEventDispatcherProxy')) {
+        if (class_exists(\Symfony\Component\EventDispatcher\LegacyEventDispatcherProxy::class)) {
             /**
              * It could return null only if we pass null, but we pass not null in any case
              *
@@ -59,7 +54,7 @@ public function createMenu(array $options = []): ItemInterface
             ->setLabel('sylius.ui.media')
         ;
 
-        if (class_exists('Symfony\Component\EventDispatcher\LegacyEventDispatcherProxy')) {
+        if (class_exists(\Symfony\Component\EventDispatcher\LegacyEventDispatcherProxy::class)) {
             $this->eventDispatcher->dispatch(
                 new BrandMenuBuilderEvent($this->factory, $menu, $options['brand']),
                 self::EVENT_NAME,

src/Model/Brand.php

@@ -28,17 +28,11 @@ public function __construct()
         $this->__productsAwareTraitConstruct();
     }
 
-    /**
-     * @inheritdoc
-     */
     public function __toString(): string
     {
         return (string) $this->getName();
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getId(): ?int
     {
         return $this->id;
@@ -60,17 +54,11 @@ public function setCode(?string $code): void
         $this->code = $code;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getName(): ?string
     {
         return $this->name;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function setName(?string $name): void
     {
         $this->name = $name;

src/Model/BrandImage.php

@@ -15,9 +15,6 @@ public function isLogo(): bool
         return $this->getType() === self::TYPE_LOGO;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function getBrand(): ?BrandInterface
     {
         /** @var BrandInterface|null $brand */
@@ -26,9 +23,6 @@ public function getBrand(): ?BrandInterface
         return $brand;
     }
 
-    /**
-     * @inheritdoc
-     */
     public function setBrand(?BrandInterface $brand): void
     {
         $this->setOwner($brand);