IcingaCloneObjectForm: Fix cloning of hots in director branches

The cloned host must also have services and service sets that are under the original host.
Icinga · Jul 23, 2024 · 48f49ac · 48f49ac
1 parent 428a49f
commit 48f49ac
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/application/forms/IcingaCloneObjectForm.php b/application/forms/IcingaCloneObjectForm.php
@@ -146,7 +146,8 @@ public function onSuccess()
             $object->getObjectName()
         );
 
-        if ($object->isTemplate() && $this->branch && $this->branch->isBranch()) {
+        $isBranch = $this->branch && $this->branch->isBranch();
+        if ($object->isTemplate() && $isBranch) {
             throw new IcingaException('Cloning templates is not available for Branches');
         }
 
@@ -209,7 +210,11 @@ public function onSuccess()
                 );
 
                 if ($new instanceof IcingaHost) {
-                    $clone->set('host_id', $newId);
+                    if ($isBranch) {
+                        $clone->set('host', $newName);
+                    } else {
+                        $clone->set('host_id', $newId);
+                    }
                 } elseif ($new instanceof IcingaServiceSet) {
                     $clone->set('service_set_id', $newId);
                 }
@@ -220,7 +225,14 @@ public function onSuccess()
                 $newSet = IcingaServiceSet::fromPlainObject(
                     $set->toPlainObject(),
                     $connection
-                )->set('host_id', $newId);
+                );
+
+                if ($isBranch) {
+                    $newSet->set('host', $newName);
+                } else {
+                    $newSet->set('host_id', $newId);
+                }
+
                 $store->store($newSet);
             }