App\Http\Middleware\RedirectCrawlers

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

55.56%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	19	3

<?php

namespace App\Http\Middleware;

use App\Tournament;
use Closure;
use Illuminate\Support\Facades\Route;

class RedirectCrawlers
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request $request
     * @param  \Closure $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        $crawlers = [
            'facebookexternalhit/1.1',
            'facebookexternalhit/1.1 (+http://www.facebook.com/externalhit_uatext.php)',
            'Facebot',
            'Twitterbot',
        ];

        $userAgent = $request->header('User-Agent');

        if (str_contains($userAgent, $crawlers)) {

It seems like $userAgent can also be of type array; however, parameter $haystack of str_contains() does only seem to accept string, maybe add an additional type check?

            switch (Route::getCurrentRoute()->getName()) {
                case "register.category":
                    $tournament = Tournament::where('slug', $request->tournament)->first();
                    return view('public/register', compact('tournament'));
            }
        }
        return $next($request);
    }
}