App\Http\Middleware\JaxonCallbacks

Complexity

Total Complexity

1

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

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

<?php

namespace App\Http\Middleware;

use App\Ajax\CallableClass;
use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request;
use Illuminate\Http\Response;
use Jaxon\Laravel\Jaxon;
use Closure;

use function app;

class JaxonCallbacks
{
    /**
     * Handle an incoming request.
     *
     * @param  Request  $request
     * @param  Closure(Request): (Response|RedirectResponse)  $next
     *
     * @return Response|RedirectResponse
     */
    public function handle(Request $request, Closure $next)
    {
        /** @var Jaxon */
        $jaxon = app()->make(Jaxon::class);
        $jaxon->callback()->init(function(CallableClass $callable) use($jaxon) {
            // Jaxon init
            $dialog = $jaxon->ajaxResponse()->dialog;

Accessing dialog on the interface Jaxon\Response\ResponseInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

            $callable->dialog = $dialog;
            $callable->notify = $dialog;
        });

        return $next($request);
    }
}