Add types to implementations of RewriteColumnBehavior::rewriteColumn()

Add types and missing return statements for compatibility with ipl-orm.

Author: BastianLedererIcinga

library/Notifications/Model/Behavior/IcingaCustomVars.php

@@ -26,7 +26,7 @@ public function isSelectableColumn(string $name): bool
             || str_starts_with($name, self::SERVICE_PREFIX);
     }
 
-    public function rewriteColumn($column, ?string $relation = null)
+    public function rewriteColumn(mixed $column, ?string $relation = null): null
     {
         return null;
     }

library/Notifications/Model/Behavior/IdTagAggregator.php

@@ -35,7 +35,7 @@ public function setQuery(Query $query): static
         return $this;
     }
 
-    public function rewriteColumn($column, ?string $relation = null)
+    public function rewriteColumn(mixed $column, ?string $relation = null): ?AliasedExpression
     {
         if ($column === 'id_tags') {
             $path = ($relation ?? $this->query->getModel()->getTableAlias()) . '.object_id_tag';
@@ -63,6 +63,8 @@ public function rewriteColumn($column, ?string $relation = null)
                 $this->query->getResolver()->qualifyColumn('value', $pathAlias)
             ));
         }
+
+        return null;
     }
 
     public function isSelectableColumn(string $name): bool

library/Notifications/Model/Behavior/ObjectTags.php

@@ -48,7 +48,7 @@ public function rewriteCondition(Filter\Condition $condition, $relation = null):
         return $filterAll;
     }
 
-    public function rewriteColumn($column, $relation = null): AliasedExpression
+    public function rewriteColumn(mixed $column, $relation = null): AliasedExpression
     {
         /** @var string $relation */
         /** @var string $column */