mvc5 / mvc5

src/Event.php

<?php
/**
 *
 */

namespace Mvc5;

class Event
    implements Event\Event
{
    /**
     *
     */
    use Event\Model;
    use Signal;

    /**
     * @var mixed
     */
    protected $model;

    /**
     * @param string|null $event
     * @param mixed $model
     */
    function __construct($event = null, $model = null)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->event = $event;
        $this->model = $model;
    }

    /**
     * @return array
     */
    protected function args()
    {
        return array_filter([
            Arg::EVENT => $this,
            Arg::MODEL => $this->model
        ]);
    }

    /**
     * @param callable $callable
     * @param array $args
     * @param callable $callback
     * @return mixed
     */
    function __invoke(callable $callable, array $args = [], callable $callback = null)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $model = $this->signal($callable, !$args || !is_string(key($args)) ? $args : $this->args() + $args, $callback);

The expression $args of type array 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.

        null !== $model
            && $this->model = $model;

        return $model;
    }
}