App\Repository\Traits\SaveEntityTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

60.87%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	saveAllInTransaction()	0	16	3
A	saveAll()	0	9	2
A	deleteAll()	0	9	2

<?php

declare(strict_types=1);

/*
 * This file is part of the zibios/sharep.
 *
 * (c) Zbigniew Ślązak
 */

namespace App\Repository\Traits;

use App\Entity\EntityInterface;
use Doctrine\ORM\EntityManagerInterface;

trait SaveEntityTrait
{
    public function saveAllInTransaction(EntityInterface ...$entities): void
    {
        /** @var EntityManagerInterface $entityManager */
        $entityManager = $this->getEntityManager();

It seems like getEntityManager() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $entityManager->getConnection()->beginTransaction();
        try {
            foreach ($entities as $entity) {
                $entityManager->persist($entity);
            }
            $entityManager->flush();
            $entityManager->getConnection()->commit();
        } catch (\Throwable $e) {
            $entityManager->getConnection()->rollBack();
            throw $e;
        }
    }

    public function saveAll(EntityInterface ...$entities): void
    {
        /** @var EntityManagerInterface $entityManager */
        $entityManager = $this->getEntityManager();

It seems like getEntityManager() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($entities as $entity) {
            $entityManager->persist($entity);
        }
        $entityManager->flush();
    }

    public function deleteAll(EntityInterface ...$entities): void
    {
        /** @var EntityManagerInterface $entityManager */
        $entityManager = $this->getEntityManager();

It seems like getEntityManager() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($entities as $entity) {
            $entityManager->remove($entity);
        }
        $entityManager->flush();
    }
}