LAG\AdminBundle\Bridge\Doctrine\ORM\State\ORMDataProcessor::process()

Complexity

Conditions	4
Paths	4

Size

Total Lines	16
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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();
        }