Refactor SitemapController and SitemapServiceProvider to use a default filename for generating the sitemap, instead of relying on a filename parameter. This simplifies the code and ensures consistency in generating the sitemap.

Author: thejmitchener

src/Http/Controllers/SitemapController.php

@@ -15,6 +15,7 @@
 class SitemapController extends BaseController
 {
     protected Sitemap $sitemap;
+    protected $defaultFilename = 'sitemap.xml';
 
     /**
      * SitemapController constructor.
@@ -31,9 +32,9 @@ public function __construct(Sitemap $sitemap)
      *
      * @throws FileNotFoundException
      */
-    public function __invoke(?string $filename): Response
+    public function __invoke(): Response
     {
-        $contents = $this->sitemap->getSitemapContents($filename);
+        $contents = $this->sitemap->getSitemapContents($this->defaultFilename);
 
         return response($contents, 200)
             ->header('Content-Type', 'application/xml');

src/SitemapServiceProvider.php

@@ -36,8 +36,7 @@ public function configurePackage(Package $package): void
      */
     public function PackageRegistered(): void
     {
-        Route::get('/{filename}', SitemapController::class)
-            ->where('filename', '.*\.xml$')
+        Route::get('/sitemap.xml', SitemapController::class)
             ->name('sitemap');
     }
 }