Fix - Update plugin's boot method to not directly reference Settings until DB connection is established

Author: Bradie Tilley

Plugin.php

@@ -2,14 +2,18 @@
 
 namespace ABWebDevelopers\ImageResize;
 
-use System\Classes\PluginBase;
 use ABWebDevelopers\ImageResize\Classes\Resizer;
 use ABWebDevelopers\ImageResize\Commands\ImageResizeClear;
 use ABWebDevelopers\ImageResize\Commands\ImageResizeGc;
 use ABWebDevelopers\ImageResize\Models\Settings;
 use ABWebDevelopers\ImageResize\ReportWidgets\ImageResizeClearWidget;
+use App;
+use Artisan;
+use DB;
 use Event;
-use Illuminate\Support\Facades\Artisan;
+use Illuminate\Database\QueryException;
+use Symfony\Component\Console\Output\ConsoleOutput;
+use System\Classes\PluginBase;
 
 class Plugin extends PluginBase
 {
@@ -92,11 +96,22 @@ public function boot()
             }
         });
 
-        if (Settings::cleanupOnCacheClear()) {
-            Event::listen('cache:cleared', function () {
-                Artisan::call('imageresize:clear');
+        Event::listen('cache:cleared', function () {
+            $this->ifDatabaseExists(function () {
+                if (Settings::cleanupOnCacheClear()) {
+                    if (App::runningInConsole()) {
+                        $output = new ConsoleOutput();
+                        $output->writeln('<info>Imagesizer: Deleting cached resized images...</info>');
+                    }
+
+                    Artisan::call('imageresize:clear');
+
+                    if (App::runningInConsole()) {
+                        $output->writeln('<info>Imagesizer: ' . Artisan::output() . '</info>');
+                    }
+                }
             });
-        }
+        });
     }
 
     /**
@@ -117,6 +132,9 @@ public function registerSchedule($schedule)
         $schedule->command('imageresize:gc')->everyFiveMinutes();
     }
 
+    /**
+     * @inheritDoc
+     */
     public function registerReportWidgets()
     {
         return [
@@ -126,4 +144,24 @@ public function registerReportWidgets()
             ],
         ];
     }
+
+    /**
+     * Run the callback only if/when the database exists (and system_settings table exists).
+     * 
+     * @param \Closure $callback
+     * @return mixed
+     */
+    public function ifDatabaseExists(\Closure $callback)
+    {
+        $canConnectToDatabase = false;
+        try {
+            // Test database connection (throw exception if no DB is configured yet)
+            $canConnectToDatabase = DB::table('system_settings')->exists();
+        } catch (QueryException $e) {
+        }
+
+        if ($canConnectToDatabase) {
+            return $callback();
+        }
+    }
 }

updates/version.yaml

@@ -36,4 +36,6 @@
 2.1.5:
   - Add new setting that runs `imageresize:clear` when `cache:clear` is run (if configured), and fix a Settings bug that was introduced from v2.1.0
 2.1.6:
-  - Fix migration (2.1.2 migration) that causes installations to fail (due to plugin settings not existing for new projects at time of execution)
+  - Fix migration (2.1.2 migration) that causes installations to fail (due to plugin settings not existing for new projects at time of execution)
+2.1.7:
+  - Fix - Update plugin's boot method to not directly reference Settings until DB connection is established