Do not register table listeners if Contao bundles are not installed

terminal42 · Nov 27, 2023 · 0a2ee69 · 0a2ee69
1 parent 3af2014
commit 0a2ee69
Show file tree

Hide file tree

Showing 2 changed files with 76 additions and 40 deletions.
diff --git a/src/EventListener/CallbackSetupListener.php b/src/EventListener/CallbackSetupListener.php
@@ -4,6 +4,7 @@
 
 namespace Terminal42\ChangeLanguage\EventListener;
 
+use Composer\InstalledVersions;
 use Contao\CoreBundle\ServiceAnnotation\Hook;
 use Terminal42\ChangeLanguage\EventListener\BackendView\ArticleViewListener;
 use Terminal42\ChangeLanguage\EventListener\BackendView\PageViewListener;
@@ -19,43 +20,56 @@
  */
 class CallbackSetupListener
 {
-    private static array $listeners = [
-        'tl_page' => [
-            PageViewListener::class,
-        ],
-        'tl_article' => [
-            ArticleListener::class,
-            PageViewListener::class,
-            ArticleViewListener::class,
-        ],
-        'tl_content' => [
-            ArticleViewListener::class,
-            ParentChildViewListener::class,
-        ],
-        'tl_news_archive' => [ParentTableListener::class],
-        'tl_calendar' => [ParentTableListener::class],
-        'tl_faq_category' => [ParentTableListener::class],
-        'tl_news' => [
-            NewsListener::class,
-            ParentChildViewListener::class,
-        ],
-        'tl_calendar_events' => [
-            CalendarEventsListener::class,
-            ParentChildViewListener::class,
-        ],
-        'tl_faq' => [
-            FaqListener::class,
-            ParentChildViewListener::class,
-        ],
-    ];
+    private static $listeners;
 
     public function __invoke(string $table): void
     {
-        if (\array_key_exists($table, self::$listeners)) {
-            foreach (self::$listeners[$table] as $class) {
+        $listeners = self::getListeners();
+
+        if (\array_key_exists($table, $listeners)) {
+            foreach ($listeners[$table] as $class) {
                 $listener = new $class($table);
                 $listener->register();
             }
         }
     }
+
+    private static function getListeners(): array
+    {
+        if (null !== self::$listeners) {
+            return self::$listeners;
+        }
+
+        $listeners = [
+            'tl_page' => [
+                PageViewListener::class,
+            ],
+            'tl_article' => [
+                ArticleListener::class,
+                PageViewListener::class,
+                ArticleViewListener::class,
+            ],
+            'tl_content' => [
+                ArticleViewListener::class,
+                ParentChildViewListener::class,
+            ],
+        ];
+
+        if (InstalledVersions::isInstalled('contao/news-bundle')) {
+            $listeners['tl_news_archive'] = [ParentTableListener::class];
+            $listeners['tl_news'] = [NewsListener::class, ParentChildViewListener::class];
+        }
+
+        if (InstalledVersions::isInstalled('contao/calendar-bundle')) {
+            $listeners['tl_calendar'] = [ParentTableListener::class];
+            $listeners['tl_calendar_events'] = [CalendarEventsListener::class, ParentChildViewListener::class];
+        }
+
+        if (InstalledVersions::isInstalled('contao/faq-bundle')) {
+            $listeners['tl_faq_category'] = [ParentTableListener::class];
+            $listeners['tl_faq'] = [FaqListener::class, ParentChildViewListener::class];
+        }
+
+        return self::$listeners = $listeners;
+    }
 }
diff --git a/src/EventListener/DataContainer/MissingLanguageIconListener.php b/src/EventListener/DataContainer/MissingLanguageIconListener.php
@@ -4,6 +4,7 @@
 
 namespace Terminal42\ChangeLanguage\EventListener\DataContainer;
 
+use Composer\InstalledVersions;
 use Contao\ArticleModel;
 use Contao\Backend;
 use Contao\BackendUser;
@@ -27,13 +28,7 @@
  */
 class MissingLanguageIconListener
 {
-    private static array $callbacks = [
-        'tl_page' => 'onPageLabel',
-        'tl_article' => 'onArticleLabel',
-        'tl_news' => 'onNewsChildRecords',
-        'tl_calendar_events' => 'onCalendarEventChildRecords',
-        'tl_faq' => 'onFaqChildRecords',
-    ];
+    private static $callbacks;
 
     private Security $security;
 
@@ -47,10 +42,12 @@ public function __construct(Security $security)
      */
     public function __invoke(string $table): void
     {
-        if (\array_key_exists($table, self::$callbacks)) {
+        $callbacks = self::getCallbacks();
+
+        if (\array_key_exists($table, $callbacks)) {
             LabelCallback::createAndRegister(
                 $table,
-                fn (array $args, $previousResult) => $this->{self::$callbacks[$table]}($args, $previousResult),
+                fn (array $args, $previousResult) => $this->{$callbacks[$table]}($args, $previousResult),
             );
         }
     }
@@ -213,4 +210,29 @@ private function generateLabelWithWarning($label, $imgStyle = '')
             $imgStyle,
         );
     }
+
+    private static function getCallbacks(): array {
+        if (null !== self::$callbacks) {
+            return self::$callbacks;
+        }
+
+        $callbacks = [
+            'tl_page' => 'onPageLabel',
+            'tl_article' => 'onArticleLabel',
+        ];
+
+        if (InstalledVersions::isInstalled('contao/news-bundle')) {
+            $callbacks['tl_news'] = 'onNewsChildRecords';
+        }
+
+        if (InstalledVersions::isInstalled('contao/calendar-bundle')) {
+            $callbacks['tl_calendar_events'] = 'onCalendarEventChildRecords';
+        }
+
+        if (InstalledVersions::isInstalled('contao/faq-bundle')) {
+            $callbacks['tl_faq'] = 'onFaqChildRecords';
+        }
+
+        return self::$callbacks = $callbacks;
+    }
 }