[Fix] Exceptions handling in initialization (#299)

* [Fix] Refactored baseTableExists to isEnabled, added exceptions handling for cache/db

* Move isEnabled call inside the callback func to prevent calling DB and Cache in every app bootstrap

* Fixed StyleCI errors

Author: o-kima

src/Providers/ConsoleServiceProvider.php

@@ -17,17 +17,17 @@ class ConsoleServiceProvider extends ServiceProvider
      */
     public function boot()
     {
-        if (Totem::baseTableExists()) {
-            $this->app->resolving(Schedule::class, function ($schedule) {
+        $this->app->resolving(Schedule::class, function ($schedule) {
+            if (Totem::isEnabled()) {
                 $this->schedule($schedule);
-            });
-        }
+            }
+        });
     }
 
     /**
      * Prepare schedule from tasks.
      *
-     * @param Schedule $schedule
+     * @param  Schedule  $schedule
      */
     public function schedule(Schedule $schedule)
     {

src/Totem.php

@@ -7,6 +7,7 @@
 use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Facades\Schema;
 use Symfony\Component\Console\Command\Command;
+use Throwable;
 
 class Totem
 {
@@ -20,7 +21,7 @@ class Totem
     /**
      * Determine if the given request can access the Totem dashboard.
      *
-     * @param \Illuminate\Http\Request $request
+     * @param  \Illuminate\Http\Request  $request
      *
      * @return bool
      */
@@ -34,7 +35,7 @@ public static function check($request)
     /**
      * Set the callback that should be used to authenticate Totem users.
      *
-     * @param \Closure $callback
+     * @param  \Closure  $callback
      *
      * @return static
      */
@@ -91,16 +92,20 @@ public static function getCommands()
     /**
      * @return bool
      */
-    public static function baseTableExists(): bool
+    public static function isEnabled(): bool
     {
-        if (Cache::get('totem.table.'.TOTEM_TABLE_PREFIX.'tasks')) {
-            return true;
-        }
+        try {
+            if (Cache::get('totem.table.'.TOTEM_TABLE_PREFIX.'tasks')) {
+                return true;
+            }
 
-        if (Schema::hasTable(TOTEM_TABLE_PREFIX.'tasks')) {
-            Cache::forever('totem.table.'.TOTEM_TABLE_PREFIX.'tasks', true);
+            if (Schema::hasTable(TOTEM_TABLE_PREFIX.'tasks')) {
+                Cache::forever('totem.table.'.TOTEM_TABLE_PREFIX.'tasks', true);
 
-            return true;
+                return true;
+            }
+        } catch (Throwable $e) {
+            return false;
         }
 
         return false;