[8.x] Subset in request's collect

[iraldoad]

This adds the ability to get a subset in request collect function.

Before

    public function collect($key = null)
    {
        return collect($this->input($key));
    }

After

    public function collect($key = null)
    {
        if (is_array($key))
            return collect($this->only($key));
        else
            return collect($this->input($key));
    }

fix #39190

[GrahamCampbell]

Thanks for the PR. Left a comment.

[GrahamCampbell]

please don;t use else after a return

[BrandonSurowiec]

It would be a good idea to add some test coverage. You can see the previous commit that added this method to give you an idea of where to add the test: iraldoad@4cb7660#diff-50be1df2e396478cd5abf036e618ba2a6c09c5dfcb6429d172faea92dae1c904

You could make this method a one-liner:

    public function collect($key = null)
    {
        return collect(is_array($key) ? $this->only($key) : $this->input($key));
    }

[taylorotwell]

Draft pending test coverage.

[iraldoad]

Working on it.

[BrandonSurowiec]

You're only ever passing one key into the collect() method on your test. It would be better to pass multiple keys in to really test that we're able to grab more than one key with your addition of only(). Expand your initial request to 3-4 parameters.

[BrandonSurowiec]

@iraldoad Great improvement, congrats on the merge! 🥳

[iraldoad]

@iraldoad Great improvement, congrats on the merge! 🥳

Thank you very much for the suggestions and reviews. Happy to contribute to this great project! 🥳