Add moderation system: tree view, public logs, representative requests, and permission wiring

- Drop course-scoped moderation — moderators can edit any document
- Add promoted_by FK on User to track who granted moderator rights
- Add moderation tree view (lobste.rs-style) showing the chain of trust
- Add public about page explaining the moderation system
- Fix document edit logging bug (old values were captured after form binding)
- Replace all hardcoded href="/" paths in templates with {% url %} tags
- Add isort skip for .venv to prevent corrupting installed packages
- Add comprehensive moderation test suite (18 tests)

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

Author: C4ptainCrunch

catalog/templates/catalog/course.html

@@ -246,7 +246,7 @@ <h4 class="alert-heading">
                     mais il se pourrait que certains documents soient mal rangés.
                     <br>
                     Si tu ne trouves plus des documents des années précédentes,
-                    <a href="/catalog/f/archives/">regarde dans les archives</a> !
+                    <a href="{% url 'catalog:finder' slugs='archives' %}">regarde dans les archives</a> !
                 </div>
                 <div>
                     <h3>Il n’y a encore rien dans ce cours…</h3>

catalog/templates/catalog/finder.html

@@ -33,7 +33,7 @@ <h4 class="alert-heading">
             </h4>
             Le programme de cours de l'ULB a parfois beaucoup changé au cours des années.
             Si tu ne trouves plus des documents des années précédentes,
-            <a href="/catalog/f/archives/">regarde dans les archives</a> !
+            <a href="{% url 'catalog:finder' slugs='archives' %}">regarde dans les archives</a> !
         </div>
     {% endif %}
     <div class="d-flex gap-2">

documents/models.py

@@ -154,15 +154,6 @@ def add_to_queue(self) -> None:
     def get_absolute_url(self) -> str:
         return reverse("document_show", args=(self.id,))
 
-    def write_perm(self, user, moderated_courses) -> bool:
-        if user.id == self.user_id:
-            return True
-
-        if self.course_id in moderated_courses:
-            return True
-
-        return False
-
     def tag_from_name(self) -> None:
         tags = logic.tags_from_name(self.name)
         self.tags.add(*tags)

documents/views.py

@@ -125,6 +125,10 @@ def document_edit(request, pk):
                 reverse("catalog:course_show", args=[doc.course.slug])
             )
 
+        old_name = doc.name
+        old_description = doc.description
+        old_tags = list(doc.tags.all())
+
         form = DocumentForm(request.POST, instance=doc)
 
         if form.is_valid():
@@ -133,12 +137,12 @@ def document_edit(request, pk):
                     user=request.user,
                     content_object=doc,
                     values={
-                        "name": (doc.name, form.cleaned_data["name"]),
+                        "name": (old_name, form.cleaned_data["name"]),
                         "description": (
-                            doc.description,
+                            old_description,
                             form.cleaned_data["description"],
                         ),
-                        "tags": (doc.tags.all(), form.cleaned_data["tags"]),
+                        "tags": (old_tags, form.cleaned_data["tags"]),
                     },
                 )
 
@@ -169,7 +173,6 @@ def document_reupload(request, pk):
 
     if not request.user.write_perm(obj=document):
         return HttpResponse("You may not edit this document.", status=403)
-    # FIXME: log moderation action
 
     if document.state != Document.DocumentState.DONE:
         return HttpResponse(
@@ -196,13 +199,12 @@ def document_reupload(request, pk):
 
             document.reprocess(force=True)
 
-            # TODO Log new version upload
-            # action.send(
-            #     request.user,
-            #     verb="a uploadé une nouvelle version de",
-            #     action_object=document,
-            #     target=document.course,
-            # )
+            if request.user != document.user:
+                ModerationLog.track(
+                    user=request.user,
+                    content_object=document,
+                    values={"reupload": ("", file.name)},
+                )
 
             return HttpResponseRedirect(
                 reverse("catalog:course_show", args=(document.course.slug,))

moderation/models.py

@@ -81,6 +81,8 @@ def action_text(self):
             return "a accepté la demande de"
         elif self.target_field == "action_rejeter":
             return "a refusé la demande de"
+        elif self.target_field == "reupload":
+            return "a re-uploadé"
         return f"a modifié '{self.target_field}' sur"
 
     @property
@@ -92,6 +94,8 @@ def action_color(self):
             return "success"
         elif self.target_field == "action_rejeter":
             return "warning"
+        elif self.target_field == "reupload":
+            return "info"
         return "secondary"
 
     @property

moderation/templates/moderation/about.html

@@ -0,0 +1,72 @@
+{% extends "base.html" %}
+
+{% block title %}À propos de la modération{% endblock %}
+
+{% block content %}
+    <div class="container-xl mt-4" style="max-width: 800px;">
+        <h1 class="h3 mb-4">La modération sur DocHub</h1>
+
+        <div class="card shadow-sm border-0 mb-4">
+            <div class="card-body">
+                <h2 class="h5 card-title">Que peuvent faire les modérateurs ?</h2>
+                <p>
+                    Les modérateurs sont des étudiants bénévoles qui aident à garder DocHub propre et bien organisé.
+                    Ils peuvent :
+                </p>
+                <ul>
+                    <li>Modifier le titre, la description et les tags de n'importe quel document</li>
+                    <li>Cacher ou rendre visible un document (par exemple si le contenu n'est pas approprié)</li>
+                    <li>Re-uploader une nouvelle version d'un document</li>
+                    <li>Marquer un document comme "Staff pick"</li>
+                </ul>
+                <p class="mb-0 text-muted">
+                    Les modérateurs n'ont <strong>pas</strong> accès au panneau d'administration de Django, qui est réservé aux administrateurs système.
+                </p>
+            </div>
+        </div>
+
+        <div class="card shadow-sm border-0 mb-4">
+            <div class="card-body">
+                <h2 class="h5 card-title">Transparence totale</h2>
+                <p>
+                    Toutes les actions de modération sont enregistrées publiquement.
+                    N'importe quel utilisateur connecté peut consulter le journal de modération
+                    pour voir qui a fait quoi et quand.
+                </p>
+                <a href="{% url 'public_logs' %}" class="btn btn-outline-primary btn-sm">
+                    Consulter le journal de modération
+                </a>
+            </div>
+        </div>
+
+        <div class="card shadow-sm border-0 mb-4">
+            <div class="card-body">
+                <h2 class="h5 card-title">Comment devenir modérateur ?</h2>
+                <p>
+                    Tu es délégué d'année, délégué cours de ton cercle, membre d'un bureau étudiant
+                    ou simplement un étudiant motivé ? Tu peux demander les droits de modération !
+                </p>
+                <p>
+                    Remplis le formulaire de demande, et un modérateur existant examinera ta candidature.
+                    Si elle est acceptée, tu recevras immédiatement les droits.
+                </p>
+                <a href="{% url 'representative_request' %}" class="btn btn-outline-success btn-sm">
+                    Demander les droits de modération
+                </a>
+            </div>
+        </div>
+
+        <div class="card shadow-sm border-0 mb-4">
+            <div class="card-body">
+                <h2 class="h5 card-title">L'arbre de modération</h2>
+                <p>
+                    Tu peux voir qui a promu qui dans l'arbre de modération.
+                    Chaque branche montre la chaîne de confiance entre modérateurs.
+                </p>
+                <a href="{% url 'moderation_tree' %}" class="btn btn-outline-secondary btn-sm">
+                    Voir l'arbre de modération
+                </a>
+            </div>
+        </div>
+    </div>
+{% endblock content %}

moderation/templates/moderation/home.html

@@ -7,13 +7,18 @@
         <h1 class="d-flex align-items-center gap-2">
             Modération
         </h1>
-        <a class="btn btn-outline-primary btn-sm d-inline-flex align-items-center gap-1"
-           href="{% url 'moderators_list' %}">
-            <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-people-fill" viewBox="0 0 16 16">
-                <path d="M7 14s-1 0-1-1 1-4 5-4 5 3 5 4-1 1-1 1H7Zm4-6a3 3 0 1 0 0-6 3 3 0 0 0 0 6Zm-5.784 6A2.238 2.238 0 0 1 5 13c0-1.355.68-2.75 1.936-3.72A6.325 6.325 0 0 0 5 9c-4 0-5 3-5 4s1 1 1 1h4.216ZM4.5 8a2.5 2.5 0 1 0 0-5 2.5 2.5 0 0 0 0 5Z"/>
-            </svg>
-            <span>Gérer les modérateurs</span>
-        </a>
+        <div class="d-flex gap-2 flex-wrap">
+            <a class="btn btn-outline-primary btn-sm d-inline-flex align-items-center gap-1"
+               href="{% url 'moderators_list' %}">
+                <svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="currentColor" class="bi bi-people-fill" viewBox="0 0 16 16">
+                    <path d="M7 14s-1 0-1-1 1-4 5-4 5 3 5 4-1 1-1 1H7Zm4-6a3 3 0 1 0 0-6 3 3 0 0 0 0 6Zm-5.784 6A2.238 2.238 0 0 1 5 13c0-1.355.68-2.75 1.936-3.72A6.325 6.325 0 0 0 5 9c-4 0-5 3-5 4s1 1 1 1h4.216ZM4.5 8a2.5 2.5 0 1 0 0-5 2.5 2.5 0 0 0 0 5Z"/>
+                </svg>
+                <span>Gérer les modérateurs</span>
+            </a>
+            <a class="btn btn-outline-secondary btn-sm" href="{% url 'moderation_tree' %}">Arbre</a>
+            <a class="btn btn-outline-secondary btn-sm" href="{% url 'public_logs' %}">Journal</a>
+            <a class="btn btn-outline-secondary btn-sm" href="{% url 'moderation_about' %}">À propos</a>
+        </div>
     </header>
 {% endblock header %}
 

moderation/templates/moderation/representative_request_received.html

@@ -16,5 +16,5 @@ <h1>Demande de droits de modération</h1>
         </div>
     </div>
 
-    <a href="/" class="btn btn-primary">Retour</a>
+    <a href="{% url 'index' %}" class="btn btn-primary">Retour</a>
 {% endblock content %}

moderation/templates/moderation/tree.html

@@ -0,0 +1,75 @@
+{% extends "base.html" %}
+
+{% block title %}Arbre de modération{% endblock %}
+
+{% block content %}
+    <div class="container-xl mt-4">
+        <div class="d-flex align-items-center justify-content-between mb-4">
+            <h1 class="h3 mb-0">Arbre de modération</h1>
+            <a href="{% url 'moderation_about' %}" class="btn btn-outline-secondary btn-sm">Retour</a>
+        </div>
+
+        <p class="text-muted mb-4">
+            Cette page montre qui a promu qui en tant que modérateur.
+            Chaque branche représente une chaîne de confiance.
+        </p>
+
+        <style>
+            .mod-tree ul {
+                list-style: none;
+                padding-left: 1.5rem;
+                margin: 0;
+                position: relative;
+            }
+            .mod-tree ul::before {
+                content: '';
+                position: absolute;
+                left: 0.4rem;
+                top: 0;
+                bottom: 0.75rem;
+                border-left: 1px solid #ccc;
+            }
+            .mod-tree li {
+                position: relative;
+                padding: 0.15rem 0;
+            }
+            .mod-tree li::before {
+                content: '';
+                position: absolute;
+                left: -1.1rem;
+                top: 0.75rem;
+                width: 0.7rem;
+                border-top: 1px solid #ccc;
+            }
+            .mod-tree .tree-root {
+                list-style: none;
+                padding: 0;
+                margin: 0;
+            }
+            .mod-tree .tree-root::before {
+                display: none;
+            }
+            .mod-tree .tree-root > li::before {
+                display: none;
+            }
+            .mod-tree .doc-count {
+                color: #888;
+                font-size: 0.9em;
+            }
+        </style>
+
+        {% if tree %}
+            <div class="mod-tree">
+                <ul class="tree-root">
+                    {% for root in tree %}
+                        {% include "moderation/tree_node.html" with node=root %}
+                    {% endfor %}
+                </ul>
+            </div>
+        {% else %}
+            <div class="alert alert-light border text-center py-5 text-muted">
+                <p class="mb-0">Aucune promotion enregistrée.</p>
+            </div>
+        {% endif %}
+    </div>
+{% endblock content %}

moderation/templates/moderation/tree_node.html

@@ -0,0 +1,15 @@
+<li>
+    <strong>{{ node.user.netid }}</strong>
+    <span class="text-muted">{{ node.user.name }}</span>
+    <span class="doc-count">({{ node.user.document_count }})</span>
+    {% if node.user.is_staff %}
+        <span class="badge bg-danger ms-1" style="font-size:0.7em;">Admin</span>
+    {% endif %}
+    {% if node.children %}
+        <ul>
+            {% for child in node.children %}
+                {% include "moderation/tree_node.html" with node=child %}
+            {% endfor %}
+        </ul>
+    {% endif %}
+</li>