From 2d9d8faa3d2656b7bbc46f3641d846a129d63d8a Mon Sep 17 00:00:00 2001
From: Fred Tempez <frederic.tempez@outlook.com>
Date: Sun, 23 Jul 2023 18:51:35 +0200
Subject: [PATCH] =?UTF-8?q?G=C3=A9n=C3=A9re=20des=20erreurs=20500=20en=20c?=
 =?UTF-8?q?as=20d'erreur?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 core/module/install/install.php               | 35 ++++++++++++-------
 core/module/install/view/update/update.js.php | 10 +++---
 core/module/install/view/update/update.php    |  3 +-
 3 files changed, 27 insertions(+), 21 deletions(-)

diff --git a/core/module/install/install.php b/core/module/install/install.php
index 8b0ffaf8..129dab9e 100644
--- a/core/module/install/install.php
+++ b/core/module/install/install.php
@@ -356,6 +356,7 @@ class install extends common
 						$pharData->extractTo(__DIR__ . '/../../../', null, true);
 					} catch (Exception $e) {
 						$success = false;
+						http_response_code(500);
 					}
 					// Nettoyage du dossier
 					if (file_exists(self::TEMP_DIR . 'update.tar.gz')) {
@@ -396,7 +397,11 @@ class install extends common
 							'.htaccess',
 							$fileContent
 						);
-						$message = $success ? '' : helper::translate('La réécriture d\'URL n\'a pas été restaurée !');
+						if ($success === false) {
+							$message = helper::translate('La réécriture d\'URL n\'a pas été restaurée !');
+							http_response_code(500);
+						}
+
 					}
 					// Recopie htaccess
 					if (
@@ -404,8 +409,11 @@ class install extends common
 						$success && file_exists('.htaccess.bak')
 					) {
 						// L'écraser avec le backup
-						$success = $success || copy('.htaccess.bak', '.htaccess');
-						$message = $success ? '' : helper::translate('La copie de sauvegarde du fichier htaccess n\'a pas été restaurée !');
+						$success = copy('.htaccess.bak', '.htaccess');
+						if ($success === false) {
+							$message = helper::translate('La copie de sauvegarde du fichier htaccess n\'a pas été restaurée !');
+							http_response_code(500);
+						}
 						// Effacer le backup
 						unlink('.htaccess.bak');
 					}
@@ -413,18 +421,19 @@ class install extends common
 					/**
 					 * Met à jour les dictionnaires des langues depuis les modèles installés
 					 */
+					if ($success) {
+						// Langues installées
+						$installedUI = $this->getData(['language']);
 
-					// Langues installées
-					$installedUI = $this->getData(['language']);
+						// Langues disponibles avec la mise à jour
+						require_once('core/module/install/ressource/defaultdata.php');
+						$default = init::$defaultData['language'];
 
-					// Langues disponibles avec la mise à jour
-					require_once('core/module/install/ressource/defaultdata.php');
-					$default = init::$defaultData['language'];
-
-					foreach ($installedUI as $key => $value) {
-						if ($default[$key]['version'] > $value['version']) {
-							copy('core/module/install/ressource/i18n/' . $key . '.json', self::I18N_DIR . $key . '.json');
-							$this->setData(['language', $key, $default[$key]]);
+						foreach ($installedUI as $key => $value) {
+							if ($default[$key]['version'] > $value['version']) {
+								copy('core/module/install/ressource/i18n/' . $key . '.json', self::I18N_DIR . $key . '.json');
+								$this->setData(['language', $key, $default[$key]]);
+							}
 						}
 					}
 
diff --git a/core/module/install/view/update/update.js.php b/core/module/install/view/update/update.js.php
index 8b979ecf..7145afe6 100644
--- a/core/module/install/view/update/update.js.php
+++ b/core/module/install/view/update/update.js.php
@@ -9,11 +9,11 @@ function step(i, data) {
             data: data
         },
         success: function (result) {
-             if (result.success != "1") { // Vérification de la propriété "success"
+            // if (result.success != "1") { // Vérification de la propriété "success"
             // Appel de la fonction de gestion d'erreur
-                showError(i, result, errors);
-                return;
-            }
+            //    showError(i, result, errors);
+            //    return;
+            //}
             setTimeout((function () {
                 if (4 === i) {
                     $("#installUpdateSuccess").show();
@@ -55,8 +55,6 @@ function showError(step, message, errors) {
         console.log("Données du tableau:", jsonData);
         $("#installUpdateErrorMessage").html("<strong>Détails de l'erreur :</strong><br> " + jsonData.data.replace(/^"(.*)"$/, '$1') + "<br>" + warningMessage.replace(/<[^p].*?>/g, ""));
     } else {
-        // Si l'accolade ouvrante n'est pas trouvée ou en première position, afficher un message d'erreur
-        console.log("Aucune donnée JSON trouvée dans le message d'erreur.");
         // Vous pouvez également faire quelque chose d'autre ici, par exemple, afficher un message à l'utilisateur, etc.
         $("#installUpdateErrorMessage").html(message);
     }
diff --git a/core/module/install/view/update/update.php b/core/module/install/view/update/update.php
index 454378e2..728f2bd8 100644
--- a/core/module/install/view/update/update.php
+++ b/core/module/install/view/update/update.php
@@ -30,8 +30,7 @@
 			<div id="installUpdateError" class="message colorRed displayNone">
 				<?php echo template::ico('cancel'); ?>
 				<strong>
-					<?php echo helper::translate('Une erreur est survenue lors de l\'étape :'); ?>
-					&nbsp;
+					<?php echo helper::translate('Une erreur est survenue lors de l\'étape :') . '<br>'; ?>
 					<span id="installUpdateErrorStep"> </span>.
 				</strong>
 			</div>