[10.x] Do not mutate underlying values on redirect

[timacdonald]

Fixes #46273

Due of the use of toArray the underlying values, e.g. an Eloquent model, are also cast to an array if they implement the Arrayable contract.

This means that the following will fail, as the explicit route model binding will return an eloquent model and the model will then be cast to an array and finally the array will be used as the "user" parameter.

Route::redirect('users/{user}', 'users/{user}/overview');

Route::get('users/{user}/overview', fn () => 'foo');

Route::bind('user', fn ($id) => User::findOrFail($id));

This should not be a breaking change, as the router can not handle parameters that are an array. The following will throw an "array to string exception"...

Route::redirect('users/{user}', 'users/{user}/overview');

Route::get('users/{user}/overview', fn () => 'foo');

// bind to an array
Route::bind('user', fn ($id) => [$id]);

[Propaganistas]

Seems like an elegant fix for my bug report. Thanks!