[TYPO3-mvc] RFC #8344: Default controller should be removed from generated link in UriBuilder

[Sebastian Kurfürst]

Hey,

first of all, thanks for your RFC!

I think this is a good idea to strip the Controller from the URI if it
is the default one. However, your patch introduces an unwanted behavior,
thus I have to give my -1 to it.

Wanted behavior:
If no controller is given in a ViewHelper (or any other place), then the
controller which is currently executed should be used. This is what your
patch changes in an unwanted manner.

Of course, I think it makes sense to strip the controller from the URI
if it is the default controller. So, instead of removing the two lines
in the patch, one would need an additional condition which checks if the
resolved controller is the default controller in the current plugin, and
then removes the "controller" argument.

It'd be great if you could supply an improved version of your patch
without the above described side-behavior :-)

Greets, and thanks again for your contribution,
Sebastian