Improve document history display with readable field labels and old/new values

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: C4ptainCrunch

moderation/models.py

@@ -66,7 +66,12 @@ class Meta:
     def __str__(self):
         return f"{self.user.get_short_name()} a fait une action le {self.timestamp.strftime('%d/%m/%Y')}"
 
-    ### Action translation logic ###
+    FIELD_LABELS = {
+        "name": "titre",
+        "description": "description",
+        "tags": "tags",
+        "hidden": "visibilité",
+    }
 
     @property
     def action_text(self):
@@ -83,7 +88,18 @@ def action_text(self):
             return "a refusé la demande de"
         elif self.target_field == "reupload":
             return "a re-uploadé"
-        return f"a modifié '{self.target_field}' sur"
+        label = self.FIELD_LABELS.get(self.target_field, self.target_field)
+        return f"a modifié '{label}' sur"
+
+    @property
+    def document_action_text(self):
+        """Action text for document history context (no trailing 'sur')."""
+        if self.target_field == "reupload":
+            return "re-upload"
+        if self.target_field == "hidden":
+            return "caché" if str(self.new_value) == "True" else "rendu visible"
+        label = self.FIELD_LABELS.get(self.target_field, self.target_field)
+        return f"modifié {label}"
 
     @property
     def action_color(self):

moderation/templates/moderation/document_history.html

@@ -31,6 +31,7 @@ <h1 class="mb-0">Historique de modération</h1>
                             <th class="ps-4">Date</th>
                             <th>Modérateur</th>
                             <th>Action</th>
+                            <th>Détails</th>
                         </tr>
                     </thead>
                     <tbody>
@@ -44,16 +45,21 @@ <h1 class="mb-0">Historique de modération</h1>
                                 </td>
                                 <td>
                                     <span class="badge bg-{{ log.action_color }} bg-opacity-10 text-{{ log.action_color }} border border-{{ log.action_color }}">
-                                        {{ log.action_text }}
+                                        {{ log.document_action_text }}
                                     </span>
-                                    {% if log.details_text %}
-                                        <div class="mt-1 small text-muted fst-italic">{{ log.details_text }}</div>
+                                </td>
+                                <td class="small text-muted">
+                                    {% if log.old_value and log.new_value and log.target_field != "reupload" %}
+                                        <span class="text-decoration-line-through">{{ log.old_value|truncatewords:15 }}</span>
+                                        &rarr; {{ log.new_value|truncatewords:15 }}
+                                    {% elif log.target_field == "reupload" %}
+                                        {{ log.new_value }}
                                     {% endif %}
                                 </td>
                             </tr>
                         {% empty %}
                             <tr>
-                                <td colspan="3" class="text-center text-muted py-5">
+                                <td colspan="4" class="text-center text-muted py-5">
                                     Aucune action de modération enregistrée sur ce document.
                                 </td>
                             </tr>