PEIP\Dispatcher\EventClassDispatcher

Complexity

Total Complexity

6

Size/Duplication

Total Lines	75
Duplicated Lines	42.67 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	5
Bugs	5	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	notify()	17	18	3
A	buildAndNotify()	15	16	2
A	hasListeners()	0	7	1

<?php

namespace PEIP\Dispatcher;

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

/*
 * Description of EventClassDispatcher
 *
 * @author timo
 */
use PEIP\Event\EventBuilder;
use PEIP\Util\ReflectionPool;

class EventClassDispatcher extends \PEIP\Dispatcher\ObjectMapDispatcher
{
    /**
     * Notifies all listeners of a given event-object.
     *
     * @param string                $name   name of the event
     * @param \PEIP\INF\Event\Event $object an event object
     *
     * @return bool|null
     */
    public function notify($name, $object)

This method seems to be duplicated in your project.

    {
        if ($object instanceof \PEIP\INF\Event\Event) {
            if (is_object($object->getContent())) {
                return self::doNotify(
                    $this->getListeners(
                        $name,
                        $object
                        ),
                        $object->getContent()
                    );
            } else {
                throw new \InvalidArgumentException('instance of \PEIP\INF\Event\Event must contain subject');
            }
        } else {
            throw new \InvalidArgumentException('object must be instance of \PEIP\INF\Event\Event');
        }
    }

       //put your code here

    /**
     * Creates an event-object with given object as content/subject and notifies
     * all registers listeners of the event.
     *
     * @param string $name       name of the event
     * @param object $object     the subject of the event
     * @param array  $headers    headers of the event-object as key/value pairs
     * @param string $eventClass event-class to create instances from
     *
     * @return
     */
    public function buildAndNotify($name, $object, array $headers = [], $eventClass = false, $type = false)

This method seems to be duplicated in your project.

    {
        if (!$this->hasListeners($name, ($object))) {
            return false;
        }

        return $this->notify(
                $name,
                EventBuilder::getInstance($eventClass)->build(

It seems like $eventClass defined by parameter $eventClass on line 60 can also be of type false; however, PEIP\Event\EventBuilder::getInstance() does only seem to accept string, maybe add an additional type check?

                    $object,
                    $name,
                    $headers,
                    (string) $type

The call to EventBuilder::build() has too many arguments starting with (string) $type.

                )
        );
    }

    /**
     * Checks wether an object has a listener for an event.
     *
     * @param string $name   the event-name
     * @param object $object object to check for listeners
     *
     * @return bool
     */
    public function hasListeners($name, $object)
    {
        return parent::hasListeners(
            $name,
            ReflectionPool::getInstance($object)
        );
    }
}