PiFinder\Http\Middleware\RemoveChartsFromHttp2ServerPush

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

64.29%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	12	3
A	shouldUseServerPush()	0	4	1

<?php

namespace PiFinder\Http\Middleware;

use Closure;
use Illuminate\Http\Request;

class RemoveChartsFromHttp2ServerPush
{
    /**
     * Handle an incoming request.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Closure  $next
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        $this->response = $next($request);

The property response does not exist. Did you maybe forget to declare it?

        if ($this->shouldUseServerPush($request) && ! $request->is('stats')) {
            app('server-push')->resources = collect(app('server-push')->resources)->reject(function ($resource) {
                return str_contains($resource['path'], '/js/charts.');
            })->toArray();
        }

        return $this->response;
    }

    /**
     * @param Request $request
     *
     * @return bool
     */
    protected function shouldUseServerPush(Request $request) : bool
    {
        return ! $request->ajax();
    }
}