Silverback\ApiComponentBundle\EventListener\Doctrine\TimestampedListener

Complexity

Total Complexity

4

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	11	3
A	__construct()	0	3	1

<?php

/*
 * This file is part of the Silverback API Component Bundle Project
 *
 * (c) Daniel West <daniel@silverback.is>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Silverback\ApiComponentBundle\EventListener\Doctrine;

use DateTime;
use DateTimeImmutable;
use Doctrine\ORM\EntityManagerInterface;
use Silverback\ApiComponentBundle\Entity\Utility\TimestampedInterface;
use Symfony\Component\HttpKernel\Event\ViewEvent;

/**
 * @author Daniel West <daniel@silverback.is>
 */
class TimestampedListener
{
    private EntityManagerInterface $entityManager;

    public function __construct(EntityManagerInterface $entityManager)
    {
        $this->entityManager = $entityManager;
    }

    public function __invoke(ViewEvent $event)
    {
        $entity = $event->getControllerResult();
        if (!$entity instanceof TimestampedInterface) {
            return;
        }

        if (!$this->entityManager->contains($entity)) {
            $entity->setCreated(new DateTimeImmutable());
        }
        $entity->modified = new DateTime();

Accessing modified on the interface Silverback\ApiComponentBundle\Entity\Utility\TimestampedInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}