Core\Event\Event

Complexity

Total Complexity

2

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	data()	0	4	1

<?php
/**
 * CakeCMS Core
 *
 * This file is part of the of the simple cms based on CakePHP 3.
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @package   Core
 * @license   MIT
 * @copyright MIT License http://www.opensource.org/licenses/mit-license.php
 * @link      https://github.com/CakeCMS/Core".
 * @author    Sergey Kalistratov <kalistratov.s.m@gmail.com>
 */

namespace Core\Event;

use Cake\Event\Event as CakeEvent;

use JBZoo\Data\Data;

/**
 * Class Event
 *
 * @package Core\Event
 */
class Event extends CakeEvent
{

    /**
     * Custom data for the method that receives the event
     *
     * @var Data
     */
    public $data;

    /**
     * Event constructor.
     *
     * @param string $name
     * @param null|string $subject
     * @param null|array $data
     */
    public function __construct($name, $subject = null, $data = null)
    {
        parent::__construct($name, $subject, $data);

It seems like $subject defined by parameter $subject on line 44 can also be of type string; however, Cake\Event\Event::__construct() does only seem to accept object|null, maybe add an additional type check?

        $this->data = new Data($data);

It seems like $data defined by parameter $data on line 44 can also be of type null; however, JBZoo\Data\Data::__construct() does only seem to accept array|string, maybe add an additional type check?

    }

    /**
     * Access the event data/payload.
     *
     * @return array
     */
    public function data()
    {
        return $this->data->getArrayCopy();
    }
}