`languages.detect` setting removes query and params from redirected url

[JuKra00]

Description

When activating the option languages.detect, Kirby does correctly redirect the user to the detected language slug, but does remove any params from the redirected url. I think this is due to the way the language redirect is performed in kirby/src/Cms/LanguageRoutes.php

Expected behavior
When the user with e.g. german language request opens the url my-page.com/?myquery=param the automatic redirect should push the user to my-page.com/de?myquery=param

To reproduce

1. Install starterkit
2. Set "languages" => true, "languages.detect" => true
3. Add two languages, e.g. en.php and de.php
4. Set browser to request german language
5. Open the frontpage with a query parameter attached.

Your setup

Kirby 4 / Kirby 5

[distantnative]

@getkirby/backend It could be good as a first step to come up with a universal method in the Uri class that merges two URLs or inherits certain parts from one Uri to another. (EDIT: #7611)

Then we could use that method to base the redirect target locations off of $kirby->request()->url().

[distantnative]

@lukasbestle What do you think which Uri parts we would need to merge/migrate/forward?

• Query
• Params
• Fragment
  ?

[lukasbestle]

I'd say query and params. Fragment theoretically as well, but in practice the browser won't transmit them with the request, right? But if the fragment is present for whatever reason, I think we could migrate it as well.