Track staff pick changes in moderation log and clean up history details

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

Author: C4ptainCrunch

documents/views.py

@@ -129,6 +129,7 @@ def document_edit(request, pk):
         old_name = doc.name
         old_description = doc.description
         old_tags = list(doc.tags.all())
+        old_staff_pick = doc.staff_pick
 
         form = DocumentForm(request.POST, instance=doc)
 
@@ -144,6 +145,7 @@ def document_edit(request, pk):
                             form.cleaned_data["description"],
                         ),
                         "tags": (old_tags, form.cleaned_data["tags"]),
+                        "staff_pick": (old_staff_pick, form.cleaned_data["staff_pick"]),
                     },
                 )
 

moderation/models.py

@@ -71,6 +71,7 @@ def __str__(self):
         "description": "description",
         "tags": "tags",
         "hidden": "visibilité",
+        "staff_pick": "staff pick",
     }
 
     @property
@@ -98,6 +99,12 @@ def document_action_text(self):
             return "re-upload"
         if self.target_field == "hidden":
             return "caché" if str(self.new_value) == "True" else "rendu visible"
+        if self.target_field == "staff_pick":
+            return (
+                "ajouté staff pick"
+                if str(self.new_value) == "True"
+                else "retiré staff pick"
+            )
         label = self.FIELD_LABELS.get(self.target_field, self.target_field)
         return f"modifié {label}"
 

moderation/templates/moderation/document_history.html

@@ -49,11 +49,11 @@ <h1 class="mb-0">Historique de modération</h1>
                                     </span>
                                 </td>
                                 <td class="small text-muted">
-                                    {% if log.old_value and log.new_value and log.target_field != "reupload" %}
+                                    {% if log.target_field == "reupload" %}
+                                        {{ log.new_value }}
+                                    {% elif log.target_field != "hidden" and log.target_field != "staff_pick" and log.old_value and log.new_value %}
                                         <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>