Fix 2: Review comments

Icinga · May 15, 2024 · 7d9ad13 · 7d9ad13
1 parent 3eeefa6
commit 7d9ad13
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/application/controllers/EventRuleController.php b/application/controllers/EventRuleController.php
@@ -134,11 +134,12 @@ public function indexAction(): void
         $buttonsWrapper->add([$eventRuleConfigSubmitButton, $discardChangesButton, $deleteButton]);
 
         if ($ruleId > 0) {
-            $incidents = Incident::on(Database::get())
+            $incidentCount = Incident::on(Database::get())
                 ->with('rule')
-                ->filter(Filter::equal('rule.id', $ruleId));
+                ->filter(Filter::equal('rule.id', $ruleId))
+                ->count();
 
-            if ($incidents->count() > 0) {
+            if ($incidentCount) {
                 $deleteButton->addAttributes([
                     'disabled' => true,
                     'title'    => t('There exist active incidents for this escalation and hence cannot be removed')

diff --git a/application/forms/EventRuleConfigForm.php b/application/forms/EventRuleConfigForm.php
@@ -270,8 +270,7 @@ public function populate($values): self
         foreach ($values['rule_escalation'] as $position => $escalation) {
             $conditions = explode('|', $escalation['condition'] ?? '');
             $conditionFormValues = [];
-            $conditionCount = count($conditions);
-            $conditionFormValues['condition-count'] = $conditionCount;
+            $conditionFormValues['condition-count'] = count($conditions);
             $conditionFormValues['id'] = $escalation['id'] ?? bin2hex(random_bytes(4));
 
             foreach ($conditions as $key => $condition) {
@@ -381,11 +380,12 @@ protected function createRemoveButton(string $prefix): SubmitButtonElement
         $escalationId = $escalations[$pos]['id'] ?? null;
 
         if ($escalationId && ctype_digit($escalationId)) {
-            $incident = Incident::on(Database::get())->with('rule_escalation');
-            $incident->filter(Filter::equal('rule_escalation.id', $escalationId));
-            if ($incident->count() > 0) {
-                $disableRemoveButton = true;
-            }
+            $incidentCount = Incident::on(Database::get())
+                ->with('rule_escalation')
+                ->filter(Filter::equal('rule_escalation.id', $escalationId))
+                ->count();
+
+            $disableRemoveButton = $incidentCount > 0;
         }
 
         $button = new SubmitButtonElement(