Check extension is enabled before setting listeners

src/Extend/Scout.php

@@ -6,6 +6,7 @@
 use ClarkWinkelmann\Scout\ScoutModelWrapper;
 use Flarum\Extend\ExtenderInterface;
 use Flarum\Extension\Extension;
+use Flarum\Extension\ExtensionManager;
 use Flarum\Foundation\ContainerUtil;
 use Illuminate\Contracts\Container\Container;
 use Illuminate\Contracts\Events\Dispatcher;
@@ -123,6 +124,18 @@ public function extend(Container $container, Extension $extension = null)
             return;
         }
 
+        /**
+         * @var $manager ExtensionManager
+         */
+        $manager = $container->make(ExtensionManager::class);
+
+        // By design extensions will always call the extender if Scout is installed without checking if it's enabled
+        // We could continue setting the container bindings which would just never be used
+        // But the model scope and event listeners must be skipped if the extension is disabled
+        if (!$manager->isEnabled('clarkwinkelmann-scout')) {
+            return;
+        }
+
         // It looks like we can safely repeat this call, a new scope instance of the same scope class will just
         // override the existing scope of the same class
         $this->modelClass::addGlobalScope(new FlarumSearchableScope());