🐛 Bug - fixing visual diff controller

maelstrom-cms · Feb 9, 2021 · 4cb79f1 · 4cb79f1
1 parent 5327c4f
commit 4cb79f1
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 10 deletions.
diff --git a/app/Checkers/BrowserConsole.php b/app/Checkers/BrowserConsole.php
@@ -65,7 +65,7 @@ private function shouldReport($messages)
             return false;
         }
 
-        foreach (config('odin.ignore_console_errors') as $phrase) {
+        foreach (config('app.ignore_console_errors') as $phrase) {
             if (Str::contains($messages, $phrase)) {
                 return false;
             }

diff --git a/app/Http/Controllers/VisualDiffsController.php b/app/Http/Controllers/VisualDiffsController.php
@@ -39,7 +39,7 @@ public function __invoke(Request $request, Website $website)
             if ($result->count() !== 2) {
                 return null;
             }
-            
+
             $previous = $result->last();
             $current = $result->first();
 
@@ -54,7 +54,7 @@ public function __invoke(Request $request, Website $website)
                 'diff' => $current->diff_url,
                 'date' => $current->created_at,
             ];
-        })->filter();
+        })->filter()->values();
     }
 
     /**

diff --git a/config/app.php b/config/app.php
@@ -7,6 +7,10 @@
     // How many days of uptime to report on? This effects DB performance.
     'max_uptime_age' => env('MAX_UPTIME_AGE', 90),
 
+    'ignore_console_errors' => [
+        'app/Crawler/browser.js',
+    ],
+
     /*
     |--------------------------------------------------------------------------
     | Application Name

diff --git a/config/odin.php b/config/odin.php