Generate correct URL's when using domains (#73)

Author: ivanvermeyen

src/UrlGenerator.php

@@ -55,11 +55,9 @@ public function route($name, $parameters = [], $absolute = true, $locale = null)
         $locale = $locale ?: $currentLocale;
 
         // Check if the locale is supported
-        if (!in_array($locale, config('localized-routes.supported-locales'))) {
+        if ( ! in_array($locale, $this->getSupportedLocales())) {
             // Use a fallback locale if provided
-            if (config('localized-routes.fallback_locale')) {
-                $locale = config('localized-routes.fallback_locale');
-            }
+            $locale = Config::get('localized-routes.fallback_locale', $locale);
         }
 
         // Normalize the route name by removing any locale prefix.

tests/Unit/Macros/LocalizedUrlMacroTest.php

@@ -374,6 +374,66 @@ public function it_returns_the_current_url_for_existing_non_localized_routes()
         ], $response->original);
     }
 
+    /** @test */
+    public function it_returns_the_url_for_existing_unnamed_localized_routes_using_domains()
+    {
+        $this->withoutExceptionHandling();
+        $this->setSupportedLocales([
+            'en' => 'domain.test',
+            'nl' => 'nl.domain.test',
+        ]);
+        $this->setAppLocale('en');
+        $this->setFallbackLocale('en');
+
+        Route::localized(function () {
+            Route::get('/', function () {
+                return [
+                    'current' => Route::localizedUrl(),
+                    'en' => Route::localizedUrl('en'),
+                    'nl' => Route::localizedUrl('nl'),
+                ];
+            });
+        });
+
+        $response = $this->call('GET', 'http://domain.test');
+        $response->assertOk();
+        $this->assertEquals([
+            'current' => 'http://domain.test',
+            'en' => 'http://domain.test',
+            'nl' => 'http://nl.domain.test',
+        ], $response->original);
+    }
+
+    /** @test */
+    public function it_returns_the_url_for_existing_named_localized_routes_using_domains()
+    {
+        $this->withoutExceptionHandling();
+        $this->setSupportedLocales([
+            'en' => 'domain.test',
+            'nl' => 'nl.domain.test',
+        ]);
+        $this->setAppLocale('en');
+        $this->setFallbackLocale('en');
+
+        Route::localized(function () {
+            Route::get('/', function () {
+                return [
+                    'current' => Route::localizedUrl(),
+                    'en' => Route::localizedUrl('en'),
+                    'nl' => Route::localizedUrl('nl'),
+                ];
+            })->name('route');
+        });
+
+        $response = $this->call('GET', 'http://domain.test');
+        $response->assertOk();
+        $this->assertEquals([
+            'current' => 'http://domain.test',
+            'en' => 'http://domain.test',
+            'nl' => 'http://nl.domain.test',
+        ], $response->original);
+    }
+
     /** @test */
     public function the_macro_does_not_blow_up_on_a_default_404_error()
     {