From 4e96b297c2775e2986d37e2d9f472d6cfab8437b Mon Sep 17 00:00:00 2001
From: odain <olivier.dain@combodo.com>
Date: Tue, 20 Jan 2026 16:21:52 +0100
Subject: [PATCH] =?UTF-8?q?N=C2=B08864=20-=20code=20readability?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 setup/wizardsteps.class.inc.php | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/setup/wizardsteps.class.inc.php b/setup/wizardsteps.class.inc.php
index 491e11fe9..bb0918dbd 100644
--- a/setup/wizardsteps.class.inc.php
+++ b/setup/wizardsteps.class.inc.php
@@ -2196,11 +2196,10 @@ class WizStepSummary extends WizardStep
 		$oPage->add('<div class="closed"><span class="title ibo-setup-summary-title">Extensions to be installed</span>');
 		$aExtensionsAdded = json_decode($this->oWizard->GetParameter('extensions_added'), true);
 
-		$sExtensionsAdded = '';
-		if (count($aExtensionsAdded)) {
+		if (count($aExtensionsAdded) > 0) {
 			$sExtensionsAdded = '<ul>';
 			foreach ($aExtensionsAdded as $sExtensionCode => $sLabel) {
-				$sExtensionsAdded .= '<li>'.$sLabel.'</li>';
+				$sExtensionsAdded .= "<li>$sLabel</li>'";
 			}
 			$sExtensionsAdded .= '</ul>';
 		} else {
@@ -2212,15 +2211,14 @@ class WizStepSummary extends WizardStep
 
 		$aExtensionsRemoved = json_decode($this->oWizard->GetParameter('removed_extensions'), true) ?? [];
 		$aExtensionsNotUninstallable = json_decode($this->oWizard->GetParameter('extensions_not_uninstallable'));
-		$sExtensionsRemoved = '';
 		if (count($aExtensionsRemoved) > 0) {
 			$sExtensionsRemoved = '<ul>';
 			foreach ($aExtensionsRemoved as $sExtensionCode => $sLabel) {
-				$sForcedUninstall = '';
 				if (in_array($sExtensionCode, $aExtensionsNotUninstallable)) {
-					$sForcedUninstall = ' (forced uninstallation)';
+					$sExtensionsRemoved .= "<li>$sLabel (forced uninstallation)</li>";
+				} else {
+					$sExtensionsRemoved .= "<li>$sLabel</li>";
 				}
-				$sExtensionsRemoved .= '<li>'.$sLabel.$sForcedUninstall.'</li>';
 			}
 			$sExtensionsRemoved .= '</ul>';
 		} else {