[5.1] Added ability to reference actions via an array

[robinvalk]

This pr adds the ability to reference actions via an array as followed:

return redirect()->action([OtherController::class => 'getIndex']);

instead of

return redirect()->action('\\' . OtherController::class . '@getIndex');

This is way cleaner and obvious in my opinion.

Also works in the routes.php

use App\Http\Controllers\OtherController;

Route::get('other', [OtherController::class => 'getIndex']);

[GrahamCampbell]

just return class_exists($class);

[thomasruiz]

Could be replaced by $class = $action[0]; since $method is not used afterwards.

[robinvalk]

@GrahamCampbell Should I fix the SC error or isn't it that strict?

[barryvdh]

Looks like a good idea, so we use the ::class method in a more clean way (for importing, refactoring, ctrl+clicking etc).

Should it be [OtherController::class, 'getIndex'] instead of [OtherController::class => 'getIndex'], to be more like a regular callable array?

[barryvdh]

Although that would conflict with actual (array) callbacks, that people could be using already.

[arrilot]

👎
Looks like seeking for troubles imo.
There is a nice @ convention in all parts of the framework,

Route::get('other', 'OtherController@getIndex'); is so much readable.
I don't see any reason to invent another one just because of the "Let's use ::class everywhere!" trend.

If you desperately want this ctrl clicking on your routes there is an excelent phpstorm Laravel plugin providing that

[koenvdheuvel]

@arrilot I respectfully disagree.

As @barryvdh mentioned this is added functionality and the big advantage of using ::class is that you can autocomplete in a lot of IDE's and also use it as a hyperlink within your IDE. It also makes refactoring a lot easier.

[GrahamCampbell]

The backslash is unneeded and should be avoided.

[GrahamCampbell]

If you don't want a namespace prefix, then set it to nothing in your route service provider.

[robinvalk]

This is now merged in: #24385

[robinvalk]

Actual action method is merged in: #24738