From 443ee4cb1a9706362239e6c4ea574e920b25def8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Wed, 28 Jan 2026 22:36:26 +0100
Subject: [PATCH] Update `insert` method in `AuditTrailRepository` to simplify
 nullable user handling

- Removed redundant null-check logic for `user_id` assignment.
- Updated query to directly handle optional users with safe navigation and nullable type support.
---
 .../ChillMainBundle/Repository/AuditTrailRepository.php     | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/src/Bundle/ChillMainBundle/Repository/AuditTrailRepository.php b/src/Bundle/ChillMainBundle/Repository/AuditTrailRepository.php
index 385a0dd55..e9030da31 100644
--- a/src/Bundle/ChillMainBundle/Repository/AuditTrailRepository.php
+++ b/src/Bundle/ChillMainBundle/Repository/AuditTrailRepository.php
@@ -48,15 +48,11 @@ class AuditTrailRepository extends ServiceEntityRepository
             ->setParameter('id', $auditTrail->getId())
             ->setParameter('action', $auditTrail->getAction())
             ->setParameter('occured_at', $auditTrail->getOccurredAt(), Types::DATETIMETZ_IMMUTABLE)
-            ->setParameter('user_id', null)
+            ->setParameter('user_id', $auditTrail->getUser()?->getId(), Types::INTEGER)
             ->setParameter('description', $auditTrail->getDescription())
             ->setParameter('targets', $auditTrail->getTargets(), Types::JSON)
             ->setParameter('metadata', $auditTrail->getMetadata(), Types::JSON);
 
-        if (null !== $user = $auditTrail->getUser()) {
-            $sql->setParameter('user_id', $user->getId());
-        }
-
         $sql->executeQuery();
     }
 }