Add uuid to service sets.

application/controllers/ServicesetController.php

@@ -12,6 +12,7 @@ use Icinga\Module\Director\Web\Table\IcingaHostsMatchingFilterTable;
 use Icinga\Module\Director\Web\Table\IcingaServiceSetHostTable;
 use Icinga\Module\Director\Web\Table\IcingaServiceSetServiceTable;
 use gipfl\IcingaWeb2\Link;
+use Ramsey\Uuid\Uuid;
 
 class ServicesetController extends ObjectController
 {
@@ -102,11 +103,11 @@ class ServicesetController extends ObjectController
         $name = $this->object->getObjectName();
         $tabs->add('services', [
             'url'       => 'director/serviceset/services',
-            'urlParams' => ['name' => $name],
+            'urlParams' => ['uuid' => $this->object->getUniqueId()],
             'label'     => 'Services'
         ])->add('hosts', [
             'url'       => 'director/serviceset/hosts',
-            'urlParams' => ['name' => $name],
+            'urlParams' => ['uuid' => $this->object->getUniqueId()],
             'label'     => 'Hosts'
         ]);
 

library/Director/Objects/IcingaServiceSet.php

@@ -19,6 +19,7 @@ class IcingaServiceSet extends IcingaObject implements ExportInterface
 
     protected $defaultProperties = array(
         'id'                    => null,
+        'uuid'                  => null,
         'host_id'               => null,
         'object_name'           => null,
         'object_type'           => null,
@@ -26,6 +27,8 @@ class IcingaServiceSet extends IcingaObject implements ExportInterface
         'assign_filter'         => null,
     );
 
+    protected $uuidColumn = 'uuid';
+
     protected $keyName = array('host_id', 'object_name');
 
     protected $supportsImports = true;

library/Director/Web/Table/ObjectSetTable.php

@@ -8,6 +8,7 @@ use gipfl\IcingaWeb2\Link;
 use gipfl\IcingaWeb2\Table\ZfQueryBasedTable;
 use gipfl\IcingaWeb2\Url;
 use Icinga\Module\Director\Restriction\FilterByNameRestriction;
+use Ramsey\Uuid\Uuid;
 
 class ObjectSetTable extends ZfQueryBasedTable
 {
@@ -47,7 +48,10 @@ class ObjectSetTable extends ZfQueryBasedTable
         if ($row->object_type === 'apply') {
             $params['id'] = $row->id;
         } else {
-            $params = array('name' => $row->object_name);
+            $params = [
+                'uuid' => Uuid::fromBytes($row->uuid)->toString(),
+                'name' => $row->object_name
+                ];
         }
 
         $url = Url::fromPath("director/${type}set", $params);
@@ -70,6 +74,7 @@ class ObjectSetTable extends ZfQueryBasedTable
 
         $columns = [
             'id'             => 'os.id',
+            'uuid'           => 'os.uuid',
             'object_name'    => 'os.object_name',
             'object_type'    => 'os.object_type',
             'assign_filter'  => 'os.assign_filter',

schema/mysql.sql

@@ -656,12 +656,14 @@ ALTER TABLE icinga_host_template_choice
 
 CREATE TABLE icinga_service_set (
   id INT(10) UNSIGNED NOT NULL AUTO_INCREMENT,
+  uuid VARBINARY(16) NOT NULL,
   object_name VARCHAR(128) NOT NULL,
   object_type ENUM('object', 'template', 'external_object') NOT NULL,
   host_id INT(10) UNSIGNED DEFAULT NULL,
   description TEXT DEFAULT NULL,
   assign_filter TEXT DEFAULT NULL,
   PRIMARY KEY (id),
+  UNIQUE INDEX uuid (uuid),
   UNIQUE KEY object_key (object_name, host_id),
   CONSTRAINT icinga_service_set_host
     FOREIGN KEY host (host_id)