From 4964d6b08bd2548e52a62c426080c15c125a0de7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Tempez?= <frederic.tempez@outlook.com>
Date: Sun, 13 Mar 2022 10:14:56 +0100
Subject: [PATCH] delete fonte ok

---
 core/module/theme/theme.php | 25 +++++++++----------------
 1 file changed, 9 insertions(+), 16 deletions(-)

diff --git a/core/module/theme/theme.php b/core/module/theme/theme.php
index ef42c2b4..613d8513 100644
--- a/core/module/theme/theme.php
+++ b/core/module/theme/theme.php
@@ -597,7 +597,7 @@ class theme extends common {
 										: '',
 					$type !== 'websafe' ? 	template::button('themeFontDelete' . $fontId, [
 												'class' => 'themeFontDelete buttonRed',
-												'href' => helper::baseUrl() . $this->getUrl(0) . '/fontDelete/' . $fontId . '/' . $_SESSION['csrf'],
+												'href' => helper::baseUrl() . $this->getUrl(0) . '/fontDelete/' . $type . '/' . $fontId . '/' . $_SESSION['csrf'],
 												'value' => template::ico('cancel'),
 												'disabled' => !empty($fontUsed[$fontId])
 											])
@@ -712,7 +712,7 @@ class theme extends common {
 
 			// Valeurs en sortie
 			$this->addOutput([
-				'notification' => 'La fonte a été éditée',
+				'notification' => 'La fonte a été actualisée',
 				'redirect' => helper::baseUrl() . 'theme/fonts',
 				'state' => true
 			]);
@@ -729,7 +729,7 @@ class theme extends common {
 	 */
 	public function fontDelete() {
 		// Jeton incorrect
-		if ($this->getUrl(3) !== $_SESSION['csrf']) {
+		if ($this->getUrl(4) !== $_SESSION['csrf']) {
 			// Valeurs en sortie
 			$this->addOutput([
 				'redirect' => helper::baseUrl()  . 'theme/fonts',
@@ -739,26 +739,19 @@ class theme extends common {
 		// Suppression
 		else {
 
-			// Charger les données des fontes
-			$files = $this->getData(['fonts', 'files']);
-			$imported = $this->getData(['fonts', 'imported']);
+			// Effacer la fonte de la base
+			$this->deleteData(['fonts', $this->getUrl(2), $this->getUrl(3)]);
 
 			// Effacer le fichier existant
-			if ( file_exists(self::DATA_DIR . $files[$this->getUrl(2)]) ) {
-				unlink(self::DATA_DIR . $files[$this->getUrl(2)]);
+			if ( $this->getUrl(2) === 'file' &&
+				file_exists(self::DATA_DIR . $this->getUrl(2)) ) {
+				unlink(self::DATA_DIR . $this->getUrl(2));
 			}
 
-			// Supprimer les entrées
-			unset($files[$this->getUrl(2)]);
-			unset($imported[$this->getUrl(2)]);
-
-			// Mettre à jour le fichier des fontes
-			$this->setData(['fonts', 'files', $files ]);
-			$this->setData(['fonts', 'imported', $imported ]);
 			// Valeurs en sortie
 			$this->addOutput([
 				'redirect' => helper::baseUrl()  . 'theme/fonts',
-				'notification' => 'Fonte supprimée',
+				'notification' => 'La fonte a été supprimée',
 				'state' => true
 			]);
 		}