Création d'un historique de sanctions pour le staff

templates/member/profile.html

@@ -603,6 +603,8 @@ <h4>{% trans "Sanctions" %}</h4>
                         </form>
                     </li>
                 {% endif %}
+
+                <li><a href="{% url 'member-sanctions-history' usr.pk %}">Historique</a></li>
             </ul>
             {% endif %}
         </div>

templates/member/sanctions_history.html

@@ -0,0 +1,54 @@
+{% extends "member/base.html" %}
+{% load i18n %}
+{% load date %}
+
+
+{% block title %}
+    {% trans "Historique des sanctions de" %} {{ usr.username }}
+{% endblock %}
+
+
+
+{% block breadcrumb %}
+        <li><a href="{% url "member-detail" usr.username %}">{{ usr.username }}</a></li>
+        <li>{% trans "Historique des sanctions" %}</li>
+{% endblock %}
+
+
+
+{% block headline %}
+    {% trans "Historique des sanctions de" %} {{ usr.username }}
+{% endblock %}
+
+
+
+{% block content %}
+    {% if sanctions %}
+        <table class="fullwidth">
+            <thead>
+                <th>{% trans "Type" %}</th>
+                <th>{% trans "Explication" %}</th>
+                <th class="wide">{% trans "Modérateur" %}</th>
+                <th class="wide">{% trans "Date" %}</th>
+            </thead>
+            <tbody>
+                {% for sanction in sanctions %}
+                    <tr>
+                        <td>{{ sanction.type }}</td>
+                        <td>
+                            {% if sanction.note %}
+                                {{ sanction.note }}
+                            {% else %}
+                                –
+                            {% endif %}
+                        </td>
+                        <td class="wide"><a href="{% url "member-detail" sanction.moderator.username %}">{{ sanction.moderator.username }}</a></td>
+                        <td class="wide">{{ sanction.pubdate|format_date|capfirst }}</td>
+                    </tr>
+                {% endfor %}
+            </tbody>
+        </table>
+    {% else %}
+        <p>{{ usr.username }} {% trans "n'a jamais été sanctionné." %}</p>
+    {% endif %}
+{% endblock %}

zds/member/commons.py

@@ -224,7 +224,7 @@ class ReadingOnlySanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u'Lecture Seule')
+        return _(u'Lecture seule définitive')
 
     def get_text(self):
         return self.array_infos.get('ls-text', '')
@@ -246,7 +246,7 @@ class TemporaryReadingOnlySanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u'Lecture Seule Temporaire')
+        return _(u'Lecture seule temporaire')
 
     def get_text(self):
         return self.array_infos.get('ls-text', '')
@@ -271,7 +271,7 @@ class DeleteReadingOnlySanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u"Autorisation d'écrire")
+        return _(u'Fin de lecture seule')
 
     def get_text(self):
         return self.array_infos.get('unls-text', '')
@@ -293,7 +293,7 @@ class BanSanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u'Ban définitif')
+        return _(u'Bannissement définitif')
 
     def get_text(self):
         return self.array_infos.get('ban-text', '')
@@ -315,7 +315,7 @@ class TemporaryBanSanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u'Ban Temporaire')
+        return _(u'Bannissement temporaire')
 
     def get_text(self):
         return self.array_infos.get('ban-text', '')
@@ -341,7 +341,7 @@ class DeleteBanSanction(MemberSanctionState):
     """
 
     def get_type(self):
-        return _(u'Autorisation de se connecter')
+        return _(u'Fin de bannissement')
 
     def get_text(self):
         return self.array_infos.get('unban-text', '')

zds/member/tests/tests_views.py

@@ -192,6 +192,54 @@ def test_details_member(self):
         )
         self.assertEqual(result.status_code, 404)
 
+    def test_sanctions_history(self):
+        user = ProfileFactory().user
+
+        ban = Ban(
+            user=user,
+            moderator=self.staff,
+            type='Lecture seule temporaire',
+            note='Test de LS'
+        )
+        ban.save()
+
+        # staff rights are required to view the history, check that
+        self.client.logout()
+        self.client.login(username=user.username, password='hostel77')
+
+        result = self.client.get(
+            reverse('member-sanctions-history', args=[user.pk]),
+            follow=False
+        )
+
+        self.assertEqual(result.status_code, 403)
+
+        self.client.logout()
+        self.client.login(username=self.staff.username, password='hostel77')
+
+        result = self.client.get(
+            reverse('member-sanctions-history', args=[user.pk]),
+            follow=False
+        )
+
+        self.assertEqual(result.status_code, 200)
+
+        # now, check that the history of a unknown user returns a 404
+        result = self.client.get(
+            reverse('member-sanctions-history', args=[0]),
+            follow=False
+        )
+
+        self.assertEqual(result.status_code, 404)
+
+        # finally, check that the history contains the note
+        result = self.client.get(
+            reverse('member-sanctions-history', args=[user.pk]),
+            follow=False
+        )
+
+        self.assertContains(result, 'Test de LS')
+
     def test_profile_page_of_weird_member_username(self):
 
         # create some user with weird username
@@ -635,7 +683,7 @@ def test_sanctions(self):
         self.assertIsNone(user.end_ban_write)
         self.assertIsNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-pubdate')[0]
-        self.assertEqual(ban.type, 'Lecture Seule')
+        self.assertEqual(ban.type, 'Lecture seule définitive')
         self.assertEqual(ban.note, 'Texte de test pour LS')
         self.assertEquals(len(mail.outbox), 1)
 
@@ -657,7 +705,7 @@ def test_sanctions(self):
         self.assertIsNone(user.end_ban_write)
         self.assertIsNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-id')[0]
-        self.assertEqual(ban.type, u'Autorisation d\'écrire')
+        self.assertEqual(ban.type, u'Fin de lecture seule')
         self.assertEqual(ban.note, 'Texte de test pour un-LS')
         self.assertEquals(len(mail.outbox), 2)
 
@@ -681,7 +729,7 @@ def test_sanctions(self):
         self.assertIsNotNone(user.end_ban_write)
         self.assertIsNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-id')[0]
-        self.assertEqual(ban.type, u'Lecture Seule Temporaire')
+        self.assertEqual(ban.type, u'Lecture seule temporaire')
         self.assertEqual(ban.note, u'Texte de test pour LS TEMP')
         self.assertEquals(len(mail.outbox), 3)
 
@@ -704,7 +752,7 @@ def test_sanctions(self):
         self.assertIsNone(user.end_ban_write)
         self.assertIsNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-id')[0]
-        self.assertEqual(ban.type, u'Ban définitif')
+        self.assertEqual(ban.type, u'Bannissement définitif')
         self.assertEqual(ban.note, u'Texte de test pour BAN')
         self.assertEquals(len(mail.outbox), 4)
 
@@ -727,7 +775,7 @@ def test_sanctions(self):
         self.assertIsNone(user.end_ban_write)
         self.assertIsNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-id')[0]
-        self.assertEqual(ban.type, u'Autorisation de se connecter')
+        self.assertEqual(ban.type, u'Fin de bannissement')
         self.assertEqual(ban.note, u'Texte de test pour BAN')
         self.assertEquals(len(mail.outbox), 5)
 
@@ -751,7 +799,7 @@ def test_sanctions(self):
         self.assertIsNone(user.end_ban_write)
         self.assertIsNotNone(user.end_ban_read)
         ban = Ban.objects.filter(user__id=user.user.id).order_by('-id')[0]
-        self.assertEqual(ban.type, u'Ban Temporaire')
+        self.assertEqual(ban.type, u'Bannissement temporaire')
         self.assertEqual(ban.note, u'Texte de test pour BAN TEMP')
         self.assertEquals(len(mail.outbox), 6)
 

zds/member/urls.py

@@ -5,7 +5,8 @@
 from zds.member.views import MemberList, MemberDetail, UpdateMember, UpdateAvatarMember, UpdatePasswordMember, \
     UpdateUsernameEmailMember, RegisterView, SendValidationEmailView, modify_karma, \
     modify_profile, settings_mini_profile, member_from_ip, tutorials, articles, settings_promote, login_view, \
-    logout_view, forgot_password, new_password, activate_account, generate_token_account, unregister, warning_unregister
+    logout_view, forgot_password, new_password, activate_account, generate_token_account, unregister, \
+    warning_unregister, sanctions_history
 
 urlpatterns = [
     # list
@@ -23,6 +24,7 @@
     # moderation
     url(r'^profil/karmatiser/$', modify_karma, name='member-modify-karma'),
     url(r'^profil/modifier/(?P<user_pk>\d+)/$', modify_profile, name='member-modify-profile'),
+    url(r'^profil/historique-sanctions/(?P<user_pk>\d+)/$', sanctions_history, name='member-sanctions-history'),
     url(r'^parametres/mini_profil/(?P<user_name>.+)/$', settings_mini_profile, name='member-settings-mini-profile'),
     url(r'^profil/multi/(?P<ip_address>.+)/$', member_from_ip, name='member-from-ip'),
 

zds/member/views.py

@@ -496,6 +496,23 @@ def modify_profile(request, user_pk):
     return redirect(profile.get_absolute_url())
 
 
+@login_required
+def sanctions_history(request, user_pk):
+    """Display the history of a member sanctions"""
+
+    if not request.user.has_perm('member.change_profile'):
+        raise PermissionDenied
+
+    user = get_object_or_404(User, pk=user_pk)
+
+    sanctions = Ban.objects.filter(user=user).order_by('-pubdate').select_related('moderator')
+
+    return render(request, 'member/sanctions_history.html', {
+        'usr': user,
+        'sanctions': sanctions,
+    })
+
+
 @login_required
 def tutorials(request):
     """Returns all tutorials of the authenticated user."""