From b693ee32f29025448d301a48574b182d7a0d8fe4 Mon Sep 17 00:00:00 2001
From: Bartuccio Antoine <klmp200@klmp200.net>
Date: Wed, 10 Apr 2019 14:11:31 +0200
Subject: [PATCH] Group: add a label on users to delete

---
 core/templates/core/group_detail.jinja | 1 +
 core/views/group.py                    | 5 ++++-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/core/templates/core/group_detail.jinja b/core/templates/core/group_detail.jinja
index a9572cc4..e7da48b9 100644
--- a/core/templates/core/group_detail.jinja
+++ b/core/templates/core/group_detail.jinja
@@ -12,6 +12,7 @@
     {% else %}
         <form action="{{ url('core:group_detail', object.id) }}" method="post" id="add_users">
             {% csrf_token %}
+            <label for="{{ form.users_removed.id_for_label }}">{{ form.users_removed.label }} :</label>
             {{ form.users_removed.errors }}
             {% for user in form.users_removed %}
                 <label for="{{ user.id_for_label }}">
diff --git a/core/views/group.py b/core/views/group.py
index 72f9b4f4..b1e1b0f4 100644
--- a/core/views/group.py
+++ b/core/views/group.py
@@ -52,7 +52,7 @@ class EditMembersForm(forms.Form):
         super(EditMembersForm, self).__init__(*args, **kwargs)
         self.fields["users_removed"] = forms.ModelMultipleChoiceField(
             User.objects.filter(id__in=self.current_users).all(),
-            label=None,
+            label=_("Users to delete"),
             required=False,
             widget=forms.CheckboxSelectMultiple,
         )
@@ -65,6 +65,9 @@ class EditMembersForm(forms.Form):
     )
 
     def clean_users_added(self):
+        """
+            Check that the user is not trying to add an user already in the group
+        """
         cleaned_data = super(EditMembersForm, self).clean()
         users_added = cleaned_data.get("users_added", None)
         if not users_added: