1 files changed, 7 insertions, 1 deletions

diff --git a/src/view/FormBuilder.php b/src/view/FormBuilder.php
index e2389b0..7505510 100644
--- a/src/view/FormBuilder.php
+++ b/src/view/FormBuilder.php
@@ -30,7 +30,8 @@ class FormBuilder extends AbstractBuilder
         $smtp_username = $smtp_username ?? Config::$smtp_username;
         $email_dest = $email_dest ?? Config::$email_dest;
         $keep_emails = (bool)$keep_emails ?? false; // (bool) est inutile mais plus clair
-        $retention_period = (int)($retention_period ?? App\Entity\Email::DEFAULT_RETENTION_PERIOD); // (int) est nécessaire à cause du stockage JSON
+        $retention_period = $this->getRetentionPeriod($retention_period ?? null, App\Entity\Email::DEFAULT_RETENTION_PERIOD);
+        $retention_period_sensible = $this->getRetentionPeriod($retention_period_sensible ?? null, App\Entity\Email::DEFAULT_RETENTION_PERIOD_SENSITIVE);
 
         $admin_content = '';
         if($_SESSION['admin'])
@@ -44,4 +45,9 @@ class FormBuilder extends AbstractBuilder
         require self::VIEWS_PATH . $node->getName() . '.php';
         $this->html = ob_get_clean(); // pas de concaténation ici, on écrase
     }
+
+    private function getRetentionPeriod(mixed $period, int $default_period): int
+    {
+        return ($period === null || (int)$period <= 0) ? $default_period : (int)$period; // (int) est nécessaire à cause du stockage JSON
+    }
 }
\ No newline at end of file