From 0c49cb5b15098e5cc365255cae5905731882e2a5 Mon Sep 17 00:00:00 2001
From: Thomas Zilio <Dasc3er@users.noreply.github.com>
Date: Thu, 19 Jul 2018 14:34:52 +0200
Subject: [PATCH] Miglioramento gestione logger

---
 lib/helpers.php                 | 39 +++++++++++++++++++--------------
 src/API.php                     |  3 +--
 src/Database.php                |  4 ++--
 src/HTMLBuilder/HTMLBuilder.php |  4 +---
 src/Mail.php                    |  3 +--
 src/Update.php                  |  3 +--
 6 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/lib/helpers.php b/lib/helpers.php
index bb38174d1..71a3ad0b5 100644
--- a/lib/helpers.php
+++ b/lib/helpers.php
@@ -27,21 +27,6 @@ function database()
  * @return mixed
  */
 function prepare($parameter)
-{
-    return p($parameter);
-}
-
-/**
- * Prepara il parametro inserito per l'inserimento in una query SQL.
- * Attenzione: protezione di base contro SQL Injection.
- *
- * @param string $parameter
- *
- * @since 2.3
- *
- * @return mixed
- */
-function p($parameter)
 {
     return \Database::getConnection()->prepare($parameter);
 }
@@ -94,12 +79,12 @@ function get($param, $raw = false)
 
 /**
  * Legge il valore di un'impostazione dalla tabella zz_settings.
- * Se descrizione è 1 e il tipo dell'impostazione è 'query=' mi restituisce il valore del campo descrizione della query.
  *
  * @param string $name
- * @param bool   $descrizione
  * @param bool   $again
  *
+ * @since 2.4.2
+ *
  * @return string
  */
 function setting($nome, $again = false)
@@ -110,6 +95,8 @@ function setting($nome, $again = false)
 /**
  * Restituisce l'oggetto dedicato alla gestione dei messaggi per l'utente.
  *
+ * @since 2.4.2
+ *
  * @return \Util\Messages
  */
 function flash()
@@ -120,6 +107,8 @@ function flash()
 /**
  * Restituisce l'oggetto dedicato alla gestione dell'autenticazione degli utente.
  *
+ * @since 2.4.2
+ *
  * @return \Auth
  */
 function auth()
@@ -130,6 +119,8 @@ function auth()
 /**
  * Restituisce l'oggetto dedicato alla gestione della traduzione del progetto.
  *
+ * @since 2.4.2
+ *
  * @return \Translator
  */
 function trans()
@@ -140,6 +131,8 @@ function trans()
 /**
  * Restituisce l'oggetto dedicato alla gestione della conversione di numeri e date.
  *
+ * @since 2.4.2
+ *
  * @return \Intl\Formatter
  */
 function formatter()
@@ -170,3 +163,15 @@ if (!function_exists('_')) {
         return tr($string, $parameters, $operations);
     }
 }
+
+/**
+ * Restituisce l'oggetto dedicato alla gestione dei log.
+ *
+ * @since 2.4.2
+ *
+ * @return \Monolog\Logger
+ */
+function logger()
+{
+    return Monolog\Registry::getInstance('logs');
+}
diff --git a/src/API.php b/src/API.php
index f4beaecd6..165f79638 100644
--- a/src/API.php
+++ b/src/API.php
@@ -57,8 +57,6 @@ class API extends \Util\Singleton
      */
     public function retrieve($request)
     {
-        global $logger;
-
         $user = Auth::user();
 
         // Controllo sulla compatibilità dell'API
@@ -150,6 +148,7 @@ class API extends \Util\Singleton
                 }
             } catch (PDOException $e) {
                 // Log dell'errore
+                $logger = logger();
                 $logger->addRecord(\Monolog\Logger::ERROR, $e);
 
                 return self::error('internalError');
diff --git a/src/Database.php b/src/Database.php
index 27bb0f298..dc3c62954 100644
--- a/src/Database.php
+++ b/src/Database.php
@@ -827,8 +827,6 @@ class Database extends Util\Singleton
      */
     protected function signal($e, $message, $options = [])
     {
-        global $logger;
-
         $options = array_merge([
             'session' => true,
             'level' => \Monolog\Logger::ERROR,
@@ -852,6 +850,8 @@ class Database extends Util\Singleton
         if (!empty($options['throw'])) {
             throw new PDOException($error);
         } else {
+            $logger = logger();
+
             $logger->addRecord($options['level'], $error);
         }
     }
diff --git a/src/HTMLBuilder/HTMLBuilder.php b/src/HTMLBuilder/HTMLBuilder.php
index d4e6cd885..2539469ec 100644
--- a/src/HTMLBuilder/HTMLBuilder.php
+++ b/src/HTMLBuilder/HTMLBuilder.php
@@ -195,13 +195,11 @@ class HTMLBuilder
      */
     protected static function elaborate($json)
     {
-        global $records;
-
         $values = [];
         $extras = [];
 
         if (!empty($json)) {
-            // Conversione delle variabili con i campi di database ($records)
+            // Conversione delle variabili con i campi di database ($record)
             foreach ($json as $key => $value) {
                 if (empty($value) && !is_numeric($value)) {
                     unset($json[$key]);
diff --git a/src/Mail.php b/src/Mail.php
index bed9b9b43..1750db8fb 100644
--- a/src/Mail.php
+++ b/src/Mail.php
@@ -207,8 +207,6 @@ class Mail extends PHPMailer\PHPMailer\PHPMailer
 
     public function send()
     {
-        global $logger;
-
         if (empty($this->AltBody)) {
             $this->AltBody = strip_tags($this->Body);
         }
@@ -218,6 +216,7 @@ class Mail extends PHPMailer\PHPMailer\PHPMailer
         $this->SmtpClose();
 
         // Segnalazione degli errori
+        $logger = logger();
         foreach ($this->infos as $info) {
             $logger->addRecord(\Monolog\Logger::ERROR, $info);
         }
diff --git a/src/Update.php b/src/Update.php
index 67f072961..e8870de93 100644
--- a/src/Update.php
+++ b/src/Update.php
@@ -315,8 +315,6 @@ class Update
      */
     public static function doUpdate($rate = 20)
     {
-        global $logger;
-
         set_time_limit(0);
         ignore_user_abort(true);
 
@@ -399,6 +397,7 @@ class Update
 
                 return true;
             } catch (\Exception $e) {
+                $logger = logger();
                 $logger->addRecord(\Monolog\Logger::EMERGENCY, $e->getMessage());
             }