Remove unnecessary request override of look&feel in LookAndFeelServic…

…eProvider
pradoslaw · Dec 16, 2024 · 7356242 · 7356242
1 parent 1e74d40
commit 7356242
Showing 1 changed file with 2 additions and 20 deletions.
diff --git a/app/Feature/LookAndFeel/LookAndFeelServiceProvider.php b/app/Feature/LookAndFeel/LookAndFeelServiceProvider.php
@@ -25,7 +25,7 @@ public function boot(): void
 
         Route::middleware(['web', 'auth'])->group(function () {
             Route::get('/LookAndFeel/StyleGuide', function (StyleGuide $guide, StyleGuideView $view) {
-                if ($this->userSetting() === 'modern') {
+                if ($this->lookAndFeel() === 'modern') {
                     return $view->view($guide->getPrimitiveColorGroups());
                 }
                 return response(status:404);
@@ -36,12 +36,7 @@ public function boot(): void
         });
     }
 
-    private function lookAndFeel(): string
-    {
-        return $this->requestOverride() ?? $this->userSetting() ?? 'legacy';
-    }
-
-    private function userSetting(): ?string
+    private function lookAndFeel(): ?string
     {
         if (!auth()->check()) {
             return null;
@@ -56,19 +51,6 @@ private function userSetting(): ?string
         return null;
     }
 
-    private function requestOverride(): ?string
-    {
-        /** @var Request $request */
-        $request = $this->app[Request::class];
-        if ($request->query->get('lookAndFeel') === 'legacy') {
-            return 'legacy';
-        }
-        if ($request->query->has('lookAndFeel')) {
-            return 'modern';
-        }
-        return null;
-    }
-
     private function setLookAndFeel(bool $isModern): void
     {
         if (!auth()->check()) {