GpsLab\Bundle\DomainEvent\Event\Listener\DomainEventPublisher

Complexity

Total Complexity

7

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getSubscribedEvents()	0	11	2
A	onFlush()	0	5	1
A	postFlush()	0	19	3

<?php

/**
 * GpsLab component.
 *
 * @author    Peter Gribanov <info@peter-gribanov.ru>
 * @copyright Copyright (c) 2011, Peter Gribanov
 * @license   http://opensource.org/licenses/MIT
 */

namespace GpsLab\Bundle\DomainEvent\Event\Listener;

use Doctrine\Common\EventSubscriber;
use Doctrine\ORM\Event\OnFlushEventArgs;
use Doctrine\ORM\Event\PostFlushEventArgs;
use Doctrine\ORM\Events;
use GpsLab\Bundle\DomainEvent\Service\EventPuller;
use GpsLab\Domain\Event\Bus\EventBus;
use GpsLab\Domain\Event\Event;

class DomainEventPublisher implements EventSubscriber
{
    /**
     * @var EventPuller
     */
    private $puller;

    /**
     * @var EventBus
     */
    private $bus;

    /**
     * @var bool
     */
    private $enable;

    /**
     * @var Event[]
     */
    private $events = [];

    /**
     * @param EventPuller $puller
     * @param EventBus    $bus
     * @param bool        $enable
     */
    public function __construct(EventPuller $puller, EventBus $bus, $enable)
    {
        $this->bus = $bus;
        $this->puller = $puller;
        $this->enable = $enable;
    }

    /**
     * @return array
     */
    public function getSubscribedEvents()
    {
        if (!$this->enable) {
            return [];
        }

        return [
            Events::onFlush,
            Events::postFlush,
        ];
    }

    /**
     * @param OnFlushEventArgs $args
     */
    public function onFlush(OnFlushEventArgs $args)
    {
        // aggregate events from deleted entities
        $this->events = $this->puller->pull($args->getEntityManager());
    }

    /**
     * @param PostFlushEventArgs $args
     */
    public function postFlush(PostFlushEventArgs $args)
    {
        // aggregate PreRemove/PostRemove events
        $events = array_merge($this->events, $this->puller->pull($args->getEntityManager()));

        // clear aggregate events before publish it
        // it necessary for fix recursive publish of events
        $this->events = [];

        // flush only if has domain events
        // it necessary for fix recursive handle flush
        if ($events) {

The expression $events of type GpsLab\Domain\Event\Event[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($events as $event) {
                $this->bus->publish($event);
            }

            $args->getEntityManager()->flush();
        }
    }
}