From d3387330bde71b830c6433aff603709eb645877f Mon Sep 17 00:00:00 2001
From: fredtempez <frederic.tempez@outlook.com>
Date: Wed, 5 May 2021 16:14:17 +0200
Subject: [PATCH] suppression backup auto

---
 CHANGES.md                                    |  3 +-
 core/core.php                                 | 10 +++++
 core/module/config/config.php                 | 13 ------
 core/module/config/view/advanced/advanced.php | 44 ++++++++-----------
 4 files changed, 30 insertions(+), 40 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index 88ba1d4f..981f1824 100755
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -10,7 +10,8 @@
     - Configuration du site :
         - Activation et désactivation de la déconnexion automatique empêchant plusieurs sessions avec le même compte.
         - Suppression de l'option articles par pages.
-    - Pages : option d'affichage dans une fenêtre superposée (Lity).
+        - Suppression de l'option de backup individuel des fichiers de données, pour l'activer manuellement, créez un fichier vide .backup dans site/data
+        - Nouvelle option permettant de transférer les fichiers de sauvegarde automatiques dans le gestionnaire de fichiers.
 
 
 ## version 10.6.00
diff --git a/core/core.php b/core/core.php
index d87648b2..59bda8e6 100755
--- a/core/core.php
+++ b/core/core.php
@@ -1671,6 +1671,16 @@ class common {
 			$this->setData(['config', 'i18n','nl', false ]);
 			$this->setData(['config', 'i18n','pt', false ]);
 
+			// Supprimer les fichiers de backup 
+			if (file_exists('site/data/.backup')) unlink('site/data/.backup');
+			$path = realpath('site/data');
+			foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path)) as $filename)
+			{
+				if (strpos($filename,'backup.json')) {
+					unlink($filename);
+				}
+			}
+
 			$this->setData(['core', 'dataVersion', 11000]);
 		}
 	}
diff --git a/core/module/config/config.php b/core/module/config/config.php
index cd9d0dbf..b6aaf812 100755
--- a/core/module/config/config.php
+++ b/core/module/config/config.php
@@ -482,19 +482,6 @@ class config extends common {
 					]
 				]
 			]);
-			// Efface les fichiers de backup lorsque l'option est désactivée
-			if ($this->getInput('configAdvancedFileBackup', helper::FILTER_BOOLEAN) === false) {
-				$path = realpath('site/data');
-				foreach (new RecursiveIteratorIterator(new RecursiveDirectoryIterator($path)) as $filename)
-				{
-					if (strpos($filename,'backup.json')) {
-						unlink($filename);
-					}
-				}
-				if (file_exists('site/data/.backup')) unlink('site/data/.backup');
-			} else {
-				touch('site/data/.backup');
-			}
 			// Notice
 			if(self::$inputNotices === []) {
 				// Active la réécriture d'URL
diff --git a/core/module/config/view/advanced/advanced.php b/core/module/config/view/advanced/advanced.php
index f5a34783..7b286216 100644
--- a/core/module/config/view/advanced/advanced.php
+++ b/core/module/config/view/advanced/advanced.php
@@ -17,33 +17,26 @@
 		<div class="block">
 			<h4>Maintenance</h4>
 			<div class="row">
-				<div class="col12">
+				<div class="col4">
 						<?php echo template::checkbox('configAdvancedMaintenance', true, 'Site en maintenance', [
 							'checked' => $this->getData(['config', 'maintenance'])
 						]); ?>
 					</div>
-			</div>
-			<div class="row">
-				<div class="col4">
+				<div class="col3 offset1">
 					<?php echo template::button('configBackupButton', [
 						'href' => helper::baseUrl() . 'config/backup',
 						'value' => 'Sauvegarder',
 						'ico' => 'download-cloud'
 					]); ?>
 				</div>
-				<div class="col4">
+				<div class="col3 offset1">
 					<?php echo template::button('configRestoreButton', [
 						'href' => helper::baseUrl() . 'config/restore',
 						'value' => 'Restaurer',
 						'ico' => 'upload-cloud'
 					]); ?>
 				</div>
-				<div class="col4">
-					<?php echo template::button('configBackupCopyButton', [
-						'href' => helper::baseUrl() . 'config/copyBackups',
-						'value' => 'Backups Auto &#10140; FileManager'
-					]); ?>
-				</div>
+
 			</div>
 		</div>
 	</div>
@@ -90,21 +83,6 @@
 						'help' => 'Vérifiez d\'abord que votre serveur autorise l\'URL rewriting (ce qui n\'est pas le cas chez Free).'
 					]); ?>
 				</div>
-
-			</div>
-			<div class="row">
-				<div class="col6">
-					<?php echo template::checkbox('configAdvancedAutoBackup', true, 'Sauvegarde automatique quotidienne du site', [
-							'checked' => $this->getData(['config', 'autoBackup']),
-							'help' => 'Une archive contenant le dossier /site/data est copiée dans le dossier \'site/backup\'. La sauvegarde est conservée pendant 30 jours.</p><p>Les fichiers du site ne sont pas sauvegardés automatiquement. Activation recommandée.'
-						]); ?>
-				</div>
-				<div class="col6">
-					<?php echo template::checkbox('configAdvancedFileBackup', true, 'Créer un backup des données json', [
-							'checked' => file_exists('site/data/.backup'),
-							'help' => 'Un fichier .backup.json est généré à chaque édition ou effacement d\'une donnée. La désactivation entraîne la suppression de ces fichiers.'
-						]); ?>
-				</div>
 			</div>
 			<div class="row">
 				<div class="col6">
@@ -120,6 +98,20 @@
 						]); ?>
 				</div>
 			</div>
+			<div class="row">
+				<div class="col6">
+					<?php echo template::checkbox('configAdvancedAutoBackup', true, 'Sauvegarde automatique quotidienne du site', [
+							'checked' => $this->getData(['config', 'autoBackup']),
+							'help' => 'Une archive contenant le dossier /site/data est copiée dans le dossier \'site/backup\'. La sauvegarde est conservée pendant 30 jours puis effacée.</p><p>Les fichiers du site ne sont pas sauvegardés automatiquement. Activation recommandée.'
+						]); ?>
+				</div>
+				<div class="col3">
+					<?php echo template::button('configBackupCopyButton', [
+						'href' => helper::baseUrl() . 'config/copyBackups',
+						'value' => 'Backups Auto &#10140; FileManager'
+					]); ?>
+				</div>
+			</div>
 		</div>
 	</div>
 </div>