Crummy\Phlack\Bot\AbstractBot

Complexity

Total Complexity

15

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	3

Test Coverage

Coverage

75%

Importance

Changes	9
Bugs	3	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	getMatcher()	0	4	1
A	setResponder()	0	6	1
A	say()	0	4	1
A	emote()	0	4	1
A	tell()	0	4	1
A	reply()	0	4	1
A	shout()	0	4	1
A	send()	0	4	1
A	__construct()	0	10	3
A	setMatcher()	0	13	4

<?php

namespace Crummy\Phlack\Bot;

use Crummy\Phlack\Common\Exception\InvalidArgumentException;
use Crummy\Phlack\Common\Matcher;
use Crummy\Phlack\Common\Responder\ResponderInterface;
use Crummy\Phlack\Message\MessageInterface;

abstract class AbstractBot implements ResponderAware, Matcher\MatcherAggregate
{
    private $matcher;

    /**
     * @var \Crummy\Phlack\Common\Responder\ResponderInterface
     */
    protected $responder;

    /**
     * @param Matcher\MatcherInterface|\Closure $matcher
     * @param ResponderInterface                $responder
     */
    public function __construct($matcher = null, ResponderInterface $responder = null)
    {
        $matcher = $matcher ?: new Matcher\DefaultMatcher();

        $this->setMatcher($matcher);

It seems like $matcher defined by $matcher ?: new \Crummy\Phlack\Common\Matcher\DefaultMatcher() on line 25 can also be of type object<Closure>; however, Crummy\Phlack\Bot\AbstractBot::setMatcher() does only seem to accept object<Crummy\Phlack\Common\Matcher\MatcherInterface>, maybe add an additional type check?

        if ($responder) {
            $this->setResponder($responder);
        }
    }

    /**
     * @param Matcher\MatcherInterface $matcher
     *
     * @throws \Crummy\Phlack\Common\Exception\InvalidArgumentException When given an invalid matcher.
     *
     * @return $this
     */
    public function setMatcher($matcher)
    {
        if (!$matcher instanceof Matcher\MatcherInterface && !is_callable($matcher)) {
            throw new InvalidArgumentException(sprintf(
                'The matcher must be callable, or implement \Crummy\Phlack\Common\Matcher\MatcherInterface. "%" given.',
                is_object($matcher) ? get_class($matcher) : gettype($matcher)
            ));
        }

        $this->matcher = $matcher;

        return $this;
    }

    /**
     * @return Matcher\MatcherInterface|callable
     */
    public function getMatcher()
    {
        return $this->matcher;
    }

    /**
     * @param ResponderInterface $responder
     *
     * @return self
     */
    public function setResponder(ResponderInterface $responder)
    {
        $this->responder = $responder;

        return $this;
    }

    /**
     * @param string $text
     *
     * @return \Crummy\Phlack\WebHook\Reply\Reply
     */
    protected function say($text)
    {
        return $this->responder->say($text);
    }

    /**
     * @param string $text
     *
     * @return \Crummy\Phlack\WebHook\Reply\Reply
     */
    protected function emote($text)
    {
        return $this->responder->emote($text);
    }

    /**
     * @param string $user The user_id to tell
     * @param string $text
     *
     * @return \Crummy\Phlack\WebHook\Reply\Reply
     */
    protected function tell($user, $text)
    {
        return $this->responder->tell($user, $text);
    }

    /**
     * @param \Crummy\Phlack\WebHook\CommandInterface $user The user_id, or a CommandInterface to inspect.
     * @param string                                  $text
     *
     * @return \Crummy\Phlack\WebHook\Reply\Reply
     */
    protected function reply($user, $text)
    {
        return $this->responder->reply($user, $text);
    }

    /**
     * @param string $text
     *
     * @return \Crummy\Phlack\WebHook\Reply\Reply
     */
    protected function shout($text)
    {
        return $this->responder->shout($text);
    }

    /**
     * @param MessageInterface $message
     *
     * @return \Crummy\Phlack\WebHook\Reply\EmptyReply
     */
    protected function send(MessageInterface $message)
    {
        return $this->responder->send($message);
    }
}