From 2db4bf2df9a704a7b12a342ccf768b1cf5d737c3 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Tue, 12 Sep 2017 17:59:30 +0200
Subject: [PATCH] Rimozione dei messaggi duplicati

Rimozione automatica dei messaggi duplicati dalla sessione, con miglioramenti nello stile.
---
 core.php                   |  6 +++---
 lib/functions.php          | 12 +++---------
 modules/backup/actions.php |  4 +++-
 modules/backup/edit.php    |  5 ++++-
 4 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/core.php b/core.php
index e1fe30e7a..af1314f7e 100644
--- a/core.php
+++ b/core.php
@@ -157,9 +157,9 @@ if (!API::isAPIRequest()) {
     register_shutdown_function('translateTemplate');
 
     // Impostazione della sessione di base
-    $_SESSION['infos'] = (array) $_SESSION['infos'];
-    $_SESSION['warnings'] = (array) $_SESSION['warnings'];
-    $_SESSION['errors'] = (array) $_SESSION['errors'];
+    $_SESSION['infos'] = array_unique((array) $_SESSION['infos']);
+    $_SESSION['warnings'] = array_unique((array) $_SESSION['warnings']);
+    $_SESSION['errors'] = array_unique((array) $_SESSION['errors']);
 
     // Imposto il periodo di visualizzazione dei record dal 01-01-yyy al 31-12-yyyy
     if (!empty($_GET['period_start'])) {
diff --git a/lib/functions.php b/lib/functions.php
index 3b66056e5..ba558ec02 100644
--- a/lib/functions.php
+++ b/lib/functions.php
@@ -246,20 +246,14 @@ function do_backup($path = null)
 
         // Creazione dello zip
         if (extension_loaded('zip')) {
-            if (create_zip([$path, dirname($database_file)], $backup_dir.$backup_name.'.zip', $ignores)) {
-                $_SESSION['infos'][] = tr('Nuovo backup creato!');
-            } else {
-                $_SESSION['errors'][] = tr('Errore durante la creazione del backup!');
-            }
+            $result = create_zip([$path, dirname($database_file)], $backup_dir.$backup_name.'.zip', $ignores);
 
             // Rimozione cartella temporanea
             delete($database_file);
         }
         // Copia dei file di OSM
         else {
-            copyr($path, $backup_dir.$backup_name, $ignores);
-
-            $_SESSION['infos'][] = tr('Nuovo backup creato!');
+            $result = copyr($path, $backup_dir.$backup_name, $ignores);
         }
 
         // Eliminazione vecchi backup se ce ne sono
@@ -289,7 +283,7 @@ function do_backup($path = null)
             }
         }
 
-        return true;
+        return $result;
     }
 
     return false;
diff --git a/modules/backup/actions.php b/modules/backup/actions.php
index a8f15963b..1ca36b553 100644
--- a/modules/backup/actions.php
+++ b/modules/backup/actions.php
@@ -28,7 +28,9 @@ switch (filter('op')) {
         break;
 
     case 'backup':
-        if (!do_backup()) {
+        if (do_backup()) {
+            $_SESSION['infos'][] = tr('Nuovo backup creato correttamente!');
+        } else {
             $_SESSION['errors'][] = tr('Errore durante la creazione del backup!').' '.tr_replace('_DIR_', '"'.$backup_dir.'"', tr('Verifica che la cartella _DIR_ abbia i permessi di scrittura!'));
         }
 
diff --git a/modules/backup/edit.php b/modules/backup/edit.php
index 50ada0f5d..4888be610 100644
--- a/modules/backup/edit.php
+++ b/modules/backup/edit.php
@@ -50,7 +50,10 @@ echo '
     <div class="col-md-12 text-right">
     <button type="button" class="btn btn-primary pull-right" onclick="continue_backup()"><i class="fa fa-database"></i> '.tr('Crea backup').'...</button>
     </div>
-</div>';
+</div>
+
+<div class="clearfix"></div>
+<br>';
 
 //Lettura file di backup
 if (file_exists($backup_dir)) {