use request set in crud instead of global request instance

[pxpm]

WHY

BEFORE - What was wrong? What was happening before this PR?

I think this was a leftover from some old refactoring since we are already using the request defined in the CrudPanel in other methods, eg:

CRUD/src/app/Library/CrudPanel/CrudFilter.php

Line 111 in 77f783d

$input = $input ?? new ParameterBag($this->crud()->getRequest()->all());

AFTER - What is happening after this PR?

I replaced the global request instances for the requests defined in CrudPanel, in the end it would fallback to the global request instance in case a request is not set in CrudPanel.

Is it a breaking change?

I don't think so no.

How can we test the before & after?

This is testable if you are manually creating a request instance and attaching it to the CrudPanel.

        $request = request()->create('/admin/users', 'GET', ['my_custom_filter' => 'foo']);
        $request->setRouteResolver(function () use ($request) {
            return (new Route('GET', 'admin/users', ['UserCrudController', 'index']))->bind($request);
        });
        $this->crudPanel->setRequest($request);

The global \Request instance has no access to this Request that we set on the CrudPanel.

[scrutinizer-notifier]

The inspection completed: No new issues