TelegramBot\Api\Events\Event

Complexity

Total Complexity

6

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getAction()	0	3	1
A	getChecker()	0	3	1
A	__construct()	0	4	1
A	executeAction()	0	3	1
A	executeChecker()	0	7	2

<?php

namespace TelegramBot\Api\Events;

use TelegramBot\Api\Types\Update;

class Event
{
    /**
     * @var \Closure|null
     */
    protected $checker;

    /**
     * @var \Closure
     */
    protected $action;

    /**
     * Event constructor.
     *
     * @param \Closure $action
     * @param \Closure|null $checker
     */
    public function __construct(\Closure $action, \Closure $checker = null)
    {
        $this->action = $action;
        $this->checker = $checker;
    }

    /**
     * @return \Closure
     */
    public function getAction()
    {
        return $this->action;
    }

    /**
     * @return \Closure|null
     */
    public function getChecker()
    {
        return $this->checker;
    }

    /**
     * @return mixed
     */
    public function executeChecker(Update $message)
    {
        if (is_callable($this->checker)) {
            return call_user_func($this->checker, $message);

It seems like $this->checker can also be of type null; however, parameter $callback of call_user_func() does only seem to accept callable, maybe add an additional type check?

        }

        return false;
    }

    /**
     * @return mixed
     */
    public function executeAction(Update $message)
    {
        return call_user_func($this->action, $message);
    }
}