chore: cleaner way of getting parameter

Author: Rens van Moorsel

app/controllers/public/home_controller.rb

@@ -15,13 +15,14 @@ def index
 
   def redirect_to_locale
     # Check if the URL already contains a locale
-    return if request.path.match?(/^\/\?l=(nl|en)(\/|$)/)
+    return if params[:l].in?(%w[nl en])
 
     # Get the language from the Accept-Language header
     locale = request.env['HTTP_ACCEPT_LANGUAGE'].to_s.start_with?('nl') ? 'nl' : 'en'
-
+    
     # Redirect to /?l=nl/... or /?l=en/...
-    redirect_to("/?l=#{ locale }#{ request.path }", allow_other_host: false)
+    new_params = request.query_parameters.merge(l: locale)
+    redirect_to url_for(params: new_params.merge(only_path: true)), allow_other_host: false
   end
 
   def create
@@ -36,11 +37,7 @@ def create
     if flash[:error].nil? && @member.save
       # create account and send welcome email
       user = User.create_on_member_enrollment!(@member)
-      if request.path.start_with?('/?l=nl')
-        user.language = 0
-      elsif request.path.start_with?('/?l=en')
-        user.language = 1
-      end
+      user.language = { 'nl' => 0, 'en' => 1 }[params[:l]] if params[:l].present?
       user.save
       user.resend_confirmation!(:activation_instructions)