Canvas\Providers\EventsManagerProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	attachEvents()	0	17	3
A	register()	0	13	1

<?php

namespace Canvas\Providers;

use Phalcon\Di\ServiceProviderInterface;

The type Phalcon\Di\ServiceProviderInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Phalcon\DiInterface;

The type Phalcon\DiInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Canvas\Listener\Subscription;
use Canvas\EventsManager;
use Canvas\Listener\User;

class EventsManagerProvider implements ServiceProviderInterface
{
    /**
     * List of the listeners use by the app.
     *
     * [
     *  'eventName' => 'className'
     * ];
     *
     * @var array
     */
    protected $listeners = [
    ];

    protected $canvasListeners = [
        'subscription' => Subscription::class,
        'user' => User::class,
    ];

    /**
     * @param DiInterface $container
     */
    public function register(DiInterface $container)
    {
        $container->setShared(
            'events',
            function () {
                $eventsManager = new EventsManager();

                return $eventsManager;
            }
        );

        $this->attachEvents($container, $this->listeners);
        $this->attachEvents($container, $this->canvasListeners);
    }

    /**
     * given the DI attach the list of containers.
     *
     * @param DiInterface $container
     * @param array $listeners
     * @return void
     */
    protected function attachEvents(DiInterface $container, array $listeners): bool
    {
        if (empty($listeners)) {
            return false;

The expression return false returns the type false which is incompatible with the documented return type void.

        }
        $eventsManager = $container->get('eventsManager');

        //navigate the list of listener and create the events
        foreach ($listeners as $key => $listen) {
            //create the events given the key
            $eventsManager->attach(
                $key,
                new $listen()
            );
        }

        return true;

The expression return true returns the type true which is incompatible with the documented return type void.

    }
}