Keep compatibility with Icinga DB mysql v6 & pgsql v4

Icinga · Jan 13, 2025 · 6de9b18 · 6de9b18
1 parent ffd5126
commit 6de9b18
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 10 deletions.
diff --git a/application/controllers/ServiceController.php b/application/controllers/ServiceController.php
@@ -54,7 +54,7 @@ public function init()
                 Filter::equal('host.name', $hostName)
             ));
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $query->withColumns(['has_problematic_parent']);
         }
 

diff --git a/library/Icingadb/Common/Backend.php b/library/Icingadb/Common/Backend.php
@@ -165,4 +165,13 @@ public static function getRedis(): IcingaRedis
 
         return self::$redis;
     }
+
+    public static function supportDependencies(): bool
+    {
+        if (self::getDb()->getAdapter() instanceof Pgsql) {
+            return self::getDbSchemaVersion() >= 5;
+        }
+
+        return self::getDbSchemaVersion() >= 7;
+    }
 }
diff --git a/library/Icingadb/Model/Host.php b/library/Icingadb/Model/Host.php
@@ -116,7 +116,7 @@ public function getColumns()
             'command_endpoint_id'
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns[] = 'affected_children';
         }
 
@@ -166,7 +166,7 @@ public function getColumnDefinitions()
             'command_endpoint_id'       => t('Endpoint Id')
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns['affected_children'] = t('Affected Children');
         }
 

diff --git a/library/Icingadb/Model/HostState.php b/library/Icingadb/Model/HostState.php
@@ -57,7 +57,7 @@ public function getColumnDefinitions()
             'next_update'                   => t('Host Next Update')
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns['affects_children'] = t('Host Affects Children');
         }
 

diff --git a/library/Icingadb/Model/Service.php b/library/Icingadb/Model/Service.php
@@ -109,7 +109,7 @@ public function getColumns()
             'command_endpoint_id'
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns[] = 'affected_children';
         }
 
@@ -156,7 +156,7 @@ public function getColumnDefinitions()
             'command_endpoint_id'       => t('Endpoint Id'),
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns['affected_children'] = t('Affected Children');
         }
 
@@ -207,7 +207,7 @@ public function createBehaviors(Behaviors $behaviors)
             'command_endpoint_id'
         ]));
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $behaviors->add(new HasProblematicParent());
         }
     }

diff --git a/library/Icingadb/Model/ServiceState.php b/library/Icingadb/Model/ServiceState.php
@@ -59,7 +59,7 @@ public function getColumnDefinitions()
             'next_update'                   => t('Service Next Update')
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns['affects_children'] = t('Service Affects Children');
         }
 

diff --git a/library/Icingadb/Model/State.php b/library/Icingadb/Model/State.php
@@ -103,7 +103,7 @@ public function getColumns()
             'next_update'
         ];
 
-        if (Backend::getDbSchemaVersion() >= 6) {
+        if (Backend::supportDependencies()) {
             $columns[] = 'affects_children';
         }
 

diff --git a/library/Icingadb/Widget/Detail/ObjectDetail.php b/library/Icingadb/Widget/Detail/ObjectDetail.php
@@ -619,7 +619,7 @@ protected function fetchCustomVars()
      */
     protected function createRootProblems(): ?array
     {
-        if (Backend::getDbSchemaVersion() < 6) {
+        if (! Backend::supportDependencies()) {
             if ($this->object->state->is_reachable) {
                 return null;
             }