Spatie\CookieConsent\CookieConsentMiddleware

Complexity

Total Complexity

7

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	18	4
A	containsBodyTag()	0	4	1
A	addCookieConsentScriptToResponse()	0	13	1
A	getLastClosingBodyTagPosition()	0	4	1

<?php

namespace Spatie\CookieConsent;

use Closure;
use Illuminate\Http\Response;

class CookieConsentMiddleware
{
    public function handle($request, Closure $next)
    {
        $response = $next($request);

        if (! config('cookie-consent.enabled')) {
            return $response;
        }

        if (! $response instanceof Response) {
            return $response;
        }

        if (! $this->containsBodyTag($response)) {
            return $response;
        }

        return $this->addCookieConsentScriptToResponse($response);
    }

    protected function containsBodyTag(Response $response): bool
    {
        return $this->getLastClosingBodyTagPosition($response->getContent()) !== false;

It seems like $response->getContent() targeting Symfony\Component\HttpFoundation\Response::getContent() can also be of type false; however, Spatie\CookieConsent\CookieConsentMiddleware::getLastClosingBodyTagPosition() does only seem to accept string, did you maybe forget to handle an error condition?

    }

    /**
     * @param \Illuminate\Http\Response $response
     *
     * @return $this
     */
    protected function addCookieConsentScriptToResponse(Response $response)
    {
        $content = $response->getContent();

        $closingBodyTagPosition = $this->getLastClosingBodyTagPosition($content);

It seems like $content defined by $response->getContent() on line 41 can also be of type false; however, Spatie\CookieConsent\CookieConsentMiddleware::getLastClosingBodyTagPosition() does only seem to accept string, did you maybe forget to handle an error condition?

        $content = ''
            .substr($content, 0, $closingBodyTagPosition)
            .view('cookieConsent::index')->render()

The method render does only exist in Illuminate\View\View, but not in Illuminate\Contracts\View\Factory.

            .substr($content, $closingBodyTagPosition);

        return $response->setContent($content);
    }

    protected function getLastClosingBodyTagPosition(string $content = '')
    {
        return strripos($content, '</body>');
    }
}