🐛 Test for debug module before adding the Elasticsearch panel

✨ Give an proper id the the Elasticsearch Debug module panel so that clicking the the ES button in the debug module opens the Elasticsearch tab Fixes #12
la-haute-societe · Sep 30, 2020 · 11cd620 · 11cd620
1 parent cc26dfc
commit 11cd620
Showing 1 changed file with 15 additions and 11 deletions.
diff --git a/src/Elasticsearch.php b/src/Elasticsearch.php
@@ -37,6 +37,7 @@
 use yii\elasticsearch\DebugPanel;
 use yii\elasticsearch\Exception;
 use yii\queue\ExecEvent;
+use yii\debug\Module as DebugModule;
 
 /**
  * @property  services\Elasticsearch          service
@@ -204,18 +205,21 @@ function () {
             );
         }
 
-        if (YII_DEBUG) {
-            // Add the Elasticsearch panel to the Yii debug bar
-            Event::on(
-                Application::class,
-                Application::EVENT_BEFORE_REQUEST,
-                function () {
-                    /** @var \yii\debug\Module $debugModule */
-                    $debugModule = Craft::$app->getModule('debug');
-                    $debugModule->panels['elasticsearch'] = new DebugPanel(['module' => $debugModule]);
+        // Add the Elasticsearch panel to the Yii debug bar
+        Event::on(
+            Application::class,
+            Application::EVENT_BEFORE_REQUEST,
+            function () {
+                /** @var DebugModule|null $debugModule */
+                $debugModule = Craft::$app->getModule('debug');
+                if ($debugModule) {
+                    $debugModule->panels['elasticsearch'] = new DebugPanel([
+                        'id'     => 'elasticsearch',
+                        'module' => $debugModule,
+                    ]);
                 }
-            );
-        }
+            }
+        );
 
         // Register variables
         Event::on(