From b5040b2fae09e593fc38411cc99e8b0656a8f769 Mon Sep 17 00:00:00 2001
From: Sukhwinder Dhillon <sukhwinder33445@gmail.com>
Date: Thu, 12 Jun 2025 13:07:54 +0200
Subject: [PATCH] EventRenderer: Fix `soft_state` event visual

- Only the CheckAttempt visual must be skipped in case $layout is minimal|header, not the whole condition body.
---
 library/Icingadb/View/EventRenderer.php | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/library/Icingadb/View/EventRenderer.php b/library/Icingadb/View/EventRenderer.php
index 0a9e3217..af900b3c 100644
--- a/library/Icingadb/View/EventRenderer.php
+++ b/library/Icingadb/View/EventRenderer.php
@@ -123,7 +123,7 @@ class EventRenderer implements ItemRenderer
 
                 break;
             case 'state_change':
-                if ($layout !== 'minimal' && $layout !== 'header' && $item->state->state_type === 'soft') {
+                if ($item->state->state_type === 'soft') {
                     $stateType = 'soft_state';
                     $previousStateType = 'previous_soft_state';
 
@@ -131,12 +131,14 @@ class EventRenderer implements ItemRenderer
                         $previousStateType = 'hard_state';
                     }
 
-                    $visual->addHtml(
-                        new CheckAttempt(
-                            (int) $item->state->check_attempt,
-                            (int) $item->state->max_check_attempts
-                        )
-                    );
+                    if ($layout !== 'minimal' && $layout !== 'header') {
+                        $visual->addHtml(
+                            new CheckAttempt(
+                                (int) $item->state->check_attempt,
+                                (int) $item->state->max_check_attempts
+                            )
+                        );
+                    }
                 } else {
                     $stateType = 'hard_state';
                     $previousStateType = 'previous_hard_state';