App\Http\Middleware\RedirectCrawlers::handle()

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	3.7898

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
    }