adjust cake 6.0 upgrade to current upgrade tooling rectors

[LordSimal]

Closes #8188

[LordSimal]

Should we always recommend named params for route building methods or only in certain situations?

[ADmad]

Always using named params in e.g. code keeps things consistent and more readable.

[LordSimal]

does this mean we should also add named params for 1st and 2nd param calls on all RouteBuilder calls in the docs?

[ADmad]

Always using named params in e.g. code keeps things consistent and more readable.

[LordSimal]

@ADmad this what you had in mind?

[ADmad]

@ADmad this what you had in mind?

Don't like the use of named arg for the 1st argument :) Sorry for not being clear earlier, I meant to use them for 2nd/3rd args because of the swapping of the args done in 6.x.

Your changes do highlight an inconsistency though. For RouterBuilder::connect() the 1st arg is called route, while for RouterBuilder::get() (and other RESTful shortcuts) it is called template. Should sync the argument names between the methods?

RouteBuilder::connect() also accepts a Route instance as 1st argument but in this case the $defaults argument is ignored (unless I am misreading the code). I am wondering if we should add a separate method RouterBuilder::add(Route $route, array $options = []) and make the 1st arg of RouteBuilder::connect() string only. Then we can rename it to template.