GeneaLabs\LaravelCaffeine\Http\Middleware\LaravelCaffeineDripMiddleware::handle()

Complexity

Conditions	5
Paths	4

Size

Total Lines	45
Code Lines	27

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php namespace GeneaLabs\LaravelCaffeine\Http\Middleware;
    public function handle(Request $request, Closure $next)
    {
        $response = $next($request);

        $content = $response->getContent();

        if (! is_string($content)) {
            return $response;
        }

        $shouldDripRegexp = $this->makeRegex([
            '<meta\s+',
            '(name\s*=\s*[\'"]caffeinated[\'"]\s+content\s*=\s*[\'"]false[\'"]',
            '|content\s*=\s*[\'"]false[\'"]\s+name\s*=\s*[\'"]caffeinated[\'"])',
        ]);

        $shouldNotDrip = preg_match($shouldDripRegexp, $content);

        if ($shouldNotDrip) {

The expression $shouldNotDrip of type integer|false is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== false instead.

            return $response;
        }

        $formTokenRegexp = $this->makeRegex([
            '<input([^>]*?[\n]?)*[^>]*?name\s*=\s*[\'"]_token[\'"]',
        ]);
        $metaTokenRegexp = $this->makeRegex([
            '<meta\s+',
            'name\s*=\s*[\'"]csrf[_-]token[\'"]',
        ]);
        $hasNoFormToken = ! preg_match($formTokenRegexp, $content);
        $hasNoMetaToken = ! preg_match($metaTokenRegexp, $content);

        if ($hasNoFormToken && $hasNoMetaToken) {
            return $response;
        }

        $dripper = (new Dripper);
        $content = str_replace(
            '</body>',
            "{$dripper->html}</body>",

The property html does not exist on GeneaLabs\LaravelCaffeine\Dripper. Since you implemented __get, consider adding a @property annotation.

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

        return $response;
    }