LAG\AdminBundle\Bridge\Doctrine\ORM\State\ORMDataProcessor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	process()	0	16	4

<?php

declare(strict_types=1);

namespace LAG\AdminBundle\Bridge\Doctrine\ORM\State;

use Doctrine\Bundle\DoctrineBundle\Registry;
use LAG\AdminBundle\Metadata\Create;
use LAG\AdminBundle\Metadata\Delete;
use LAG\AdminBundle\Metadata\OperationInterface;
use LAG\AdminBundle\Metadata\Update;
use LAG\AdminBundle\State\DataProcessorInterface;

class ORMDataProcessor implements DataProcessorInterface
{
    public function __construct(
        private Registry $registry,
    ) {
    }

    public function process(
        mixed $data,
        OperationInterface $operation,
        array $uriVariables = [],
        array $context = []
    ): void {
        $manager = $this->registry->getManagerForClass($operation->getResource()->getDataClass());

It seems like $operation->getResource()->getDataClass() can also be of type null; however, parameter $class of Doctrine\Persistence\AbstractManagerRegistry::getManagerForClass() does only seem to accept string, maybe add an additional type check?

        if ($operation instanceof Create || $operation instanceof Update) {
            $manager->persist($data);
            $manager->flush();
        }

        if ($operation instanceof Delete) {
            $manager->remove($data);
            $manager->flush();
        }
    }
}