App\Gitter\Middlewares\GoogleSearchMiddleware

Complexity

Total Complexity

8

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	handle()	0	23	5
A	getUserLogin()	0	14	3

<?php
/**
 * This file is part of GitterBot package.
 *
 * @author Serafim <nesk@xakep.ru>
 * @date 27.01.2016 10:41
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace App\Gitter\Middlewares;

use App\Message;
use App\Gitter\Extensions\Middlewares\MiddlewareInterface;

/**
 * Class GoogleSearchMiddleware
 * @package App\Bot\Middlewares
 */
class GoogleSearchMiddleware implements MiddlewareInterface
{
    /**
     * @param Message $message
     * @param \Closure $next
     * @return string|null
     */
    public function handle(Message $message, \Closure $next)
    {
        $matches = $message->text->escape()->matches('(@.*?\s)?(?:погугли|гугли)\s(.*?)');

        if ($matches) {

The expression $matches of type string[] 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.

            if (!trim($matches[2])) {
                return $next($message);
            }

            $mention = $this->getUserLogin($message);

            return trim($matches[1]) && $mention
                ? trans('google.personal', [
                    'user'  => $mention->login,
                    'query' => urlencode($matches[2]),
                ])
                : trans('google.common', [
                    'query' => urlencode($matches[2]),
                ]);
        }

        return $next($message);
    }

    /**
     * @param Message $message
     * @return \App\User|null
     */
    protected function getUserLogin(Message $message)
    {
        $hasMentions = $message->mentions->count();
        $mention = null;

        if ($hasMentions) {
            $first = $message->mentions[0]->user;
            $mention = $first->login === \Auth::user()->login
                ? $message->user
                : $first;
        }

        return $mention;
    }
}