From 86db1f5a0c987a2bfdf2350065d02fd09e6cda00 Mon Sep 17 00:00:00 2001
From: isUnknown <adrien.payet@outlook.com>
Date: Thu, 15 Jan 2026 11:55:17 +0100
Subject: [PATCH] Fix collectLight() : inclure author, text, location pour
 l'affichage
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Problème : collectLight() ne retournait que id/type/isRead/date, causant
notification.author undefined dans le frontend.

Solution : Inclure tous les champs nécessaires à l'affichage (author, text,
location) mais toujours alléger en excluant les gros détails inutiles.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 .../src/NotificationCollector.php             | 29 ++++++++++++++-----
 1 file changed, 22 insertions(+), 7 deletions(-)

diff --git a/public/site/plugins/notifications/src/NotificationCollector.php b/public/site/plugins/notifications/src/NotificationCollector.php
index 83e47d1..ec1714b 100644
--- a/public/site/plugins/notifications/src/NotificationCollector.php
+++ b/public/site/plugins/notifications/src/NotificationCollector.php
@@ -59,7 +59,7 @@ class NotificationCollector
 
     /**
      * Collecte uniquement les données minimales des notifications (version allégée pour listing).
-     * Retourne seulement id, type, isRead, date pour économiser la mémoire.
+     * Retourne les champs nécessaires à l'affichage mais sans les détails lourds.
      *
      * @param Page $project Le projet à scanner
      * @param User $user L'utilisateur courant
@@ -72,18 +72,33 @@ class NotificationCollector
         foreach ($this->providers as $provider) {
             try {
                 $notifications = $provider->collect($project, $user);
-                // Ne garder que les champs essentiels
+                // Garder les champs nécessaires au frontend
                 foreach ($notifications as $notification) {
-                    $all[] = [
+                    $light = [
                         'id' => $notification['id'] ?? null,
                         'type' => $notification['type'] ?? null,
                         'isRead' => $notification['isRead'] ?? false,
                         'date' => $notification['date'] ?? null,
-                        // Garder location.project.uri pour le frontend
-                        'location' => [
-                            'project' => $notification['location']['project'] ?? []
-                        ]
+                        'text' => $notification['text'] ?? null,
+                        'author' => $notification['author'] ?? null,
+                        'location' => $notification['location'] ?? []
                     ];
+
+                    // Garder les champs optionnels s'ils existent
+                    if (isset($notification['dialogUri'])) {
+                        $light['dialogUri'] = $notification['dialogUri'];
+                    }
+                    if (isset($notification['_briefUri'])) {
+                        $light['_briefUri'] = $notification['_briefUri'];
+                    }
+                    if (isset($notification['_file'])) {
+                        $light['_file'] = $notification['_file'];
+                    }
+                    if (isset($notification['_projectUri'])) {
+                        $light['_projectUri'] = $notification['_projectUri'];
+                    }
+
+                    $all[] = $light;
                 }
             } catch (\Throwable $e) {
                 error_log("NotificationCollector: Error in {$provider->getType()}: " . $e->getMessage());