From 07819d9f9e151e9fed6a6490dcad53ba2d57a550 Mon Sep 17 00:00:00 2001
From: polo <ordipolo@gmx.fr>
Date: Fri, 12 Sep 2025 17:12:04 +0200
Subject: correction erreur champs vides dans config.ini section [email]

---
 src/Config.php                     |  6 +++---
 src/EmailService.php               |  8 ++++++--
 src/view/templates/form_params.php | 14 +++++++-------
 3 files changed, 16 insertions(+), 12 deletions(-)

diff --git a/src/Config.php b/src/Config.php
index e25c180..baf4d6d 100644
--- a/src/Config.php
+++ b/src/Config.php
@@ -22,10 +22,10 @@ class Config
     static public string $smtp_host = '';
     static public string $smtp_username = '';
     static public string $smtp_password = '';
-    static public string $smtp_secure = 'tls'; // tls (smarttls) ou ssl (smtps)
-    static public string $email_from = '';
+    static public string $smtp_secure = ''; // tls (smarttls) ou ssl (smtps) ou plain_text/chaine vide
+    static public string $email_from = 'mon_adresse@email.fr';
     static public string $email_from_name = 'site web';
-    static public string $email_dest = '';
+    static public string $email_dest = 'mon_adresse@email.fr';
     static public string $email_dest_name = 'destinataire formulaire';
 
     // copier dans ce tableau les variables contenant des chemins
diff --git a/src/EmailService.php b/src/EmailService.php
index a9abc85..1bcca0f 100644
--- a/src/EmailService.php
+++ b/src/EmailService.php
@@ -20,7 +20,10 @@ class EmailService
 	    $smtp_secure = $form_data->getData()['smtp_secure'] ?? Config::$smtp_secure;
 		$smtp_username = $form_data->getData()['smtp_username'] ?? Config::$smtp_username;
 		$smtp_password = $form_data->getData()['smtp_password'] ?? Config::$smtp_password;
+		$email_from = $form_data->getData()['email_from'] ?? Config::$email_from; // une adresse bidon est donnée à setFrom()
+		$email_from_name = $form_data->getData()['email_from_name'] ?? Config::$email_from_name; // = site web
 		$email_dest = $form_data->getData()['email_dest'] ?? Config::$email_dest;
+		$email_dest_name = $form_data->getData()['email_dest_name'] ?? Config::$email_dest_name; // = destinataire formulaire
 
 	    try{
 	        // Paramètres du serveur
@@ -43,8 +46,9 @@ class EmailService
 	        //var_dump($mail->smtpConnect());die; // test de connexion
 
 	        // Expéditeur et destinataire
-	        $mail->setFrom(strtolower(Config::$email_from), Config::$email_from_name);  // paramètre modifiable uniquement dans le config.ini pour l'instant
-	        $mail->addAddress(strtolower($email_dest), Config::$email_dest_name);  // // paramètre modifiable uniquement dans le config.ini pour l'instant
+	        // $email_from, $email_from_name et $email_dest_name sont modifiables uniquement dans le config.ini pour l'instant
+	        $mail->setFrom(strtolower($email_from), $email_from_name);
+	        $mail->addAddress(strtolower($email_dest), $email_dest_name);
 
 	        // Contenu
 	        $mail->isHTML(true);
diff --git a/src/view/templates/form_params.php b/src/view/templates/form_params.php
index a7bf1d8..3559d25 100644
--- a/src/view/templates/form_params.php
+++ b/src/view/templates/form_params.php
@@ -22,23 +22,18 @@ declare(strict_types=1);
 	</p>
 	<p>
 	    <label for="smtp_username_<?= $node->getNodeData()->getId() ?>">Identifiant (adresse e-mail)</label>
-	    <input id="smtp_username_<?= $node->getNodeData()->getId() ?>" type="email" name="smtp_username" placeholder="mon-adresse@email.fr" value="<?= htmlspecialchars($smtp_username) ?>">
+	    <input id="smtp_username_<?= $node->getNodeData()->getId() ?>" type="email" name="smtp_username" autocomplete="new-password" placeholder="mon-adresse@email.fr" value="<?= htmlspecialchars($smtp_username) ?>">
 	    <input type="hidden" id="smtp_username_hidden_<?= $node->getNodeData()->getId() ?>" value="">
 	    <button onclick="setEmailParam('smtp_username', <?= $node->getNodeData()->getId() ?>)">Valider</button>
 	</p>
 	<p>
 	    <label for="smtp_password_<?= $node->getNodeData()->getId() ?>">Mot de passe</label>
-	    <input id="smtp_password_<?= $node->getNodeData()->getId() ?>" type="password" name="smtp_password">
+	    <input id="smtp_password_<?= $node->getNodeData()->getId() ?>" type="password" name="smtp_password" autocomplete="new-password">
 	    <input type="hidden" id="smtp_password_hidden_<?= $node->getNodeData()->getId() ?>" value="">
 	    <button onclick="setEmailParam('smtp_password', <?= $node->getNodeData()->getId() ?>)">Valider</button>
 	</p>
 	<p><i>Il s'agit du service qui acheminera les messages envoyés par ce formulaire. Les services d'envoi de courriels nécéssitent généralement de s'y connecter avec un identifiant et un mot de passe. Les adresses d'envoi et de réception peuvent être identiques. Le site web peut ne pas réussir à se connecter à certains fournisseurs.</i></p>
 </div>
-<div class="admin_form">
-	<p><button onclick="sendTestEmail(<?= $node->getNodeData()->getId() ?>)">Envoi d'un e-mail de test</button></p>
-	<p><i>Vérifie la connexion au serveur d'envoi. Pour tester la réception, consultez vos e-mails à l'adresse de réception.</i></p>
-	<p class="test_email_success_<?= $node->getNodeData()->getId() ?> full_width_column"></p>
-</div>
 <div class="admin_form">
 	<h3>Paramètres de réception</h3>
 	<p>
@@ -47,4 +42,9 @@ declare(strict_types=1);
 	    <input type="hidden" id="email_dest_hidden_<?= $node->getNodeData()->getId() ?>" value="">
 	    <button onclick="setEmailParam('email_dest', <?= $node->getNodeData()->getId() ?>)">Valider</button>
 	</p>
+</div>
+<div class="admin_form">
+	<p><button onclick="sendTestEmail(<?= $node->getNodeData()->getId() ?>)">Envoi d'un e-mail de test</button></p>
+	<p><i>Vérifie la connexion au serveur d'envoi. Pour tester la réception, consultez vos e-mails à l'adresse de réception.</i></p>
+	<p class="test_email_success_<?= $node->getNodeData()->getId() ?> full_width_column"></p>
 </div>
\ No newline at end of file
-- 
cgit v1.2.3