Show root problem list for objects with problem and are part of dependency

library/Icingadb/Common/StateBadges.php

@@ -46,13 +46,6 @@ public function __construct($item)
  $this->url = $this->getBaseUrl();
  }
 
- /**
- * Get the badge base URL
- *
- * @return Url
- */
- abstract protected function getBaseUrl(): Url;
-
  /**
  * Get the type of the items
  *
@@ -67,14 +60,27 @@ abstract protected function getType(): string;
  */
  abstract protected function getPrefix(): string;
 
+ /**
+ * Get the badge base URL
+ *
+ * @return ?Url
+ */
+ protected function getBaseUrl(): ?Url
+ {
+ return null;
+ }
+
  /**
  * Get the integer of the given state text
  *
  * @param string $state
  *
  * @return int
  */
- abstract protected function getStateInt(string $state): int;
+ protected function getStateInt(string $state): int
+ {
+ return 0;
+ }
 
  /**
  * Get the badge URL
@@ -132,18 +138,25 @@ public function createLink($content, Filter\Rule $filter = null): Link
  * Create a state bade
  *
  * @param string $state
+ * @param bool $createLink Create link for the badge if true
  *
  * @return ?BaseHtmlElement
  */
- protected function createBadge(string $state)
+ protected function createBadge(string $state, bool $createLink = true)
  {
  $key = $this->prefix . "_{$state}";
 
  if (isset($this->item->$key) && $this->item->$key) {
- return Html::tag('li', $this->createLink(
- new StateBadge($this->item->$key, $state),
- Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state))
- ));
+ $stateBadge = new StateBadge($this->item->$key, $state);
+
+ if ($createLink) {
+ $this->createLink(
+ $stateBadge,
+ Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state))
+ );
+ }
+
+ return Html::tag('li', $stateBadge);
  }
 
  return null;
@@ -153,37 +166,50 @@ protected function createBadge(string $state)
  * Create a state group
  *
  * @param string $state
+ * @param bool $createLink Create link for the badge if true
  *
  * @return ?BaseHtmlElement
  */
- protected function createGroup(string $state)
+ protected function createGroup(string $state, bool $createLink = true)
  {
  $content = [];
  $handledKey = $this->prefix . "_{$state}_handled";
  $unhandledKey = $this->prefix . "_{$state}_unhandled";
 
  if (isset($this->item->$unhandledKey) && $this->item->$unhandledKey) {
- $content[] = Html::tag('li', $this->createLink(
- new StateBadge($this->item->$unhandledKey, $state),
- Filter::all(
- Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state)),
- Filter::equal($this->type . '.state.is_handled', 'n'),
- Filter::equal($this->type . '.state.is_reachable', 'y')
- )
- ));
+ $unhandledStateBadge = new StateBadge($this->item->$unhandledKey, $state);
+
+ if ($createLink) {
+ $unhandledStateBadge = $this->createLink(
+ $unhandledStateBadge,
+ Filter::all(
+ Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state)),
+ Filter::equal($this->type . '.state.is_handled', 'n'),
+ Filter::equal($this->type . '.state.is_reachable', 'y')
+ )
+ );
+ }
+
+ $content[] = Html::tag('li', $unhandledStateBadge);
  }
 
  if (isset($this->item->$handledKey) && $this->item->$handledKey) {
- $content[] = Html::tag('li', $this->createLink(
- new StateBadge($this->item->$handledKey, $state, true),
- Filter::all(
- Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state)),
- Filter::any(
- Filter::equal($this->type . '.state.is_handled', 'y'),
- Filter::equal($this->type . '.state.is_reachable', 'n')
+ $handledStateBadge = new StateBadge($this->item->$handledKey, $state, true);
+
+ if ($createLink) {
+ $handledStateBadge = $this->createLink(
+ $handledStateBadge,
+ Filter::all(
+ Filter::equal($this->type . '.state.soft_state', $this->getStateInt($state)),
+ Filter::any(
+ Filter::equal($this->type . '.state.is_handled', 'y'),
+ Filter::equal($this->type . '.state.is_reachable', 'n')
+ )
  )
- )
- ));
+ );
+ }
+
+ $content[] = Html::tag('li', $handledStateBadge);
  }
 
  if (empty($content)) {

library/Icingadb/Model/DependencyEdge.php

@@ -17,8 +17,8 @@
  * @property string $from_node_id
  * @property ?string $dependency_id
  *
- * @property DependencyNode|Query $from
- * @property DependencyNode|Query $to
+ * @property DependencyNode|Query $child
+ * @property DependencyNode|Query $parent
  * @property (?Dependency)|Query $dependency
  */
 class DependencyEdge extends Model
@@ -53,11 +53,17 @@ public function createBehaviors(Behaviors $behaviors): void
 
  public function createRelations(Relations $relations): void
  {
- $relations->belongsTo('from', DependencyNode::class)
+ $relations->belongsTo('child', DependencyNode::class)
  ->setCandidateKey('from_node_id');
- $relations->belongsTo('to', DependencyNode::class)
+ $relations->belongsTo('parent', DependencyNode::class)
  ->setCandidateKey('to_node_id');
  $relations->belongsTo('dependency', Dependency::class)
  ->setJoinType('LEFT');
+
+ // "from" and "to" are only necessary for sub-query filters.
+ $relations->belongsTo('from', DependencyNode::class)
+ ->setCandidateKey('from_node_id');
+ $relations->belongsTo('to', DependencyNode::class)
+ ->setCandidateKey('to_node_id');
  }
 }

library/Icingadb/Model/DependencyNode.php

@@ -4,6 +4,7 @@
 
 namespace Icinga\Module\Icingadb\Model;
 
+use Icinga\Module\Icingadb\Model\Behavior\ReRoute;
 use ipl\Orm\Behavior\Binary;
 use ipl\Orm\Behaviors;
 use ipl\Orm\Model;
@@ -23,8 +24,6 @@
  * @property (?RedundancyGroup)|Query $redundancy_group
  * @property (?DependencyEdge)|Query $from
  * @property (?DependencyEdge)|Query $to
- * @property (?DependencyNode)|Query $child
- * @property (?DependencyNode)|Query $parent
  */
 class DependencyNode extends Model
 {
@@ -56,6 +55,10 @@ public function createBehaviors(Behaviors $behaviors): void
  'service_id',
  'redundancy_group_id'
  ]));
+ $behaviors->add(new ReRoute([
+ 'child' => 'to.from',
+ 'parent' => 'from.to'
+ ]));
  }
 
  public function createRelations(Relations $relations): void
@@ -74,15 +77,13 @@ public function createRelations(Relations $relations): void
  ->setForeignKey('to_node_id')
  ->setJoinType('LEFT');
 
- $relations->belongsToMany('child', self::class)
- ->through(DependencyEdge::class)
- ->setForeignKey('to_node_id')
- ->setTargetForeignKey('from_node_id')
- ->setJoinType('LEFT');
- $relations->belongsToMany('parent', self::class)
- ->through(DependencyEdge::class)
- ->setForeignKey('from_node_id')
- ->setTargetForeignKey('to_node_id')
- ->setJoinType('LEFT');
+ // TODO: This self join is only a work-around as when selecting nodes and filtering by child or parent,
+ // the ORM wants to join the base table as usual in case a sub-query is used. Though, in this case
+ // resolving e.g. child to "to.from" is reversed in a sub-query to "from.to" and the ORM does not
+ // detect that "to" is already the link to the base table.
+ // Given the path "dependency_node.to.from.host", the sub-query uses "host.from.to.dependency_node".
+ // "to.dependency_node" is the crucial part, as "dependency_node" is said self-join.
+ $relations->hasOne('dependency_node', self::class)
+ ->setForeignKey('id');
  }
 }

library/Icingadb/Model/Host.php

@@ -181,8 +181,8 @@ public function createBehaviors(Behaviors $behaviors)
  ]));
 
  $behaviors->add(new ReRoute([
- 'child' => 'dependency_node.child',
- 'parent' => 'dependency_node.parent',
+ 'child' => 'to.from',
+ 'parent' => 'from.to',
  'servicegroup' => 'service.servicegroup',
  'user' => 'notification.user',
  'usergroup' => 'notification.usergroup'
@@ -237,8 +237,6 @@ public function createDefaults(Defaults $defaults)
 
  public function createRelations(Relations $relations)
  {
- $relations->belongsTo('dependency_node', DependencyNode::class)
- ->setJoinType('LEFT');
  $relations->belongsTo('environment', Environment::class);
  $relations->belongsTo('eventcommand', Eventcommand::class);
  $relations->belongsTo('checkcommand', Checkcommand::class);
@@ -274,5 +272,14 @@ public function createRelations(Relations $relations)
  $relations->hasMany('notification', Notification::class)->setJoinType('LEFT');
  $relations->hasMany('notification_history', NotificationHistory::class);
  $relations->hasMany('service', Service::class)->setJoinType('LEFT');
+
+ $relations->belongsToMany('from', DependencyEdge::class)
+ ->setTargetCandidateKey('from_node_id')
+ ->setTargetForeignKey('id')
+ ->through(DependencyNode::class);
+ $relations->belongsToMany('to', DependencyEdge::class)
+ ->setTargetCandidateKey('to_node_id')
+ ->setTargetForeignKey('id')
+ ->through(DependencyNode::class);
  }
 }

library/Icingadb/Model/RedundancyGroup.php

@@ -21,6 +21,8 @@
  *
  * @property (?RedundancyGroupState)|Query $state
  * @property Dependency|Query $dependency
+ * @property DependencyEdge|Query $from
+ * @property DependencyEdge|Query $to
  */
 class RedundancyGroup extends Model
 {
@@ -48,20 +50,25 @@ public function createBehaviors(Behaviors $behaviors): void
  'id'
  ]));
  $behaviors->add(new ReRoute([
- 'child' => 'dependency_node.child',
- 'parent' => 'dependency_node.parent'
+ 'child' => 'to.from',
+ 'parent' => 'from.to'
  ]));
  }
 
  public function createRelations(Relations $relations): void
  {
- $relations->belongsTo('dependency_node', DependencyNode::class)
- ->setForeignKey('redundancy_group_id')
- ->setCandidateKey('id');
-
  $relations->hasOne('state', RedundancyGroupState::class)
  ->setJoinType('LEFT');
 
  $relations->hasMany('dependency', Dependency::class);
+
+ $relations->belongsToMany('from', DependencyEdge::class)
+ ->setTargetCandidateKey('from_node_id')
+ ->setTargetForeignKey('id')
+ ->through(DependencyNode::class);
+ $relations->belongsToMany('to', DependencyEdge::class)
+ ->setTargetCandidateKey('to_node_id')
+ ->setTargetForeignKey('id')
+ ->through(DependencyNode::class);
  }
 }