On peut changer de respos

Tout se fait simplement depuis la vue `/clubs/membres/<club_name>`.
Il est déconseillé de passer par l'interface admin.

gestioncof/templates/home.html

@@ -70,6 +70,7 @@
                     <li><a href="{% url "admin:index" %}">Administration générale</a></li>
                     <li><a href="{% url "petits-cours-demandes-list" %}">Demandes de petits cours</a></li>
                     <li><a href="{% url "gestioncof.views.registration" %}">Inscription d'un nouveau membre</a></li>
+                    <li><a href="{% url "liste-clubs" %}">Gestion des clubs</a></li>
                 </ul>
                 <ul>
                     <h4>Évènements & Sondages</h4>

gestioncof/templates/liste_clubs.html

@@ -8,7 +8,7 @@
 {% for club in owned_clubs %}
   <li>
     <a href="{% url "membres-club" club.name %}">
-      {{ club }} ({% for respo in club.respos.all %}{{ respo.get_full_name }}{% empty %}pas de respo{% endfor %})
+      {{ club }} ({% for respo in club.respos.all %}{{ respo.get_full_name }}, {% empty %}pas de respo{% endfor %})
     </a>
   </li>
 {% endfor %}
@@ -16,7 +16,7 @@
 {% for club in other_clubs %}
     <li>
       <p>
-        {{ club }} ({% for respo in club.respos.all %}{{ respo.get_full_name }}{% empty %}pas de respo{% endfor %})
+        {{ club }} ({% for respo in club.respos.all %}{{ respo.get_full_name }}, {% empty %}pas de respo{% endfor %})
       </p>
     </li>
 {% endfor %}

gestioncof/templates/membres_clubs.html

@@ -1,20 +1,41 @@
 {% extends "base_title.html" %}
 
+
 {% block realcontent %}
 <h2>{{ club }}</h2>
 
+
+{% if club.respos.exists %}
+<h3>Respo{{ club.respos.all|pluralize }}</h3>
+<table class="table table-striped">
+{% for member in club.respos.all %}
+  <tr>
+    <td>{{ member }}</td>
+    <td>{{ member.email }}</td>
+    <td><a class="glyphicon glyphicon-arrow-down"
+           href="{% url "change-respo" club.name member.id %}"></a></td>
+  </tr>
+{% endfor %}
+</table>
+{% else %}
+<h3>Pas de respo</h3>
+{% endif %}
+
+
 {% if club.membres.exists %}
 <h3>Liste des membres</h3>
 <table class="table table-striped">
-{% for member in club.membres.all %}
+{% for member in members_no_respo %}
-    <tr>
+  <tr>
-        <td>{{ member }}</td>
+    <td>{{ member }}</td>
-	<td>{{ member.email }}</td>
+    <td>{{ member.email }}</td>
-    </tr>
+    <td><a class="glyphicon glyphicon-arrow-up"
+           href="{% url "change-respo" club.name member.id %}"></a></td>
+  </tr>
 {% endfor %}
-</ul>
+</table>
 {% else %}
-Ce club ne contient actuellement aucun membre.
+Ce club ne comporte actuellement aucun membre.
 {% endif %}
 
 {% endblock %}

gestioncof/urls.py

@@ -55,5 +55,7 @@ calendar_patterns = [
 
 clubs_patterns = [
     url(r'^membres/(?P<name>\w+)', views.membres_club, name='membres-club'),
-    url(r'^liste', views.liste_clubs, name='liste-clubs')
+    url(r'^liste', views.liste_clubs, name='liste-clubs'),
+    url(r'^change_respo/(?P<club_name>\w+)/(?P<user_id>\d+)',
+        views.change_respo, name='change-respo'),
 ]

gestioncof/views.py

@@ -497,7 +497,23 @@ def membres_club(request, name):
     if not request.user.profile.is_buro \
             and club not in user.clubs_geres.all():
         return HttpResponseForbidden('<h1>Permission denied</h1>')
-    return render(request, 'membres_clubs.html', {'club': club})
+    members_no_respo = club.membres.exclude(clubs_geres=club).all()
+    return render(request, 'membres_clubs.html',
+                  {'club': club,
+                   'members_no_respo': members_no_respo})
+
+
+@buro_required
+def change_respo(request, club_name, user_id):
+    club = get_object_or_404(Club, name=club_name)
+    user = get_object_or_404(User, id=user_id)
+    if user in club.respos.all():
+        club.respos.remove(user)
+    elif user in club.membres.all():
+        club.respos.add(user)
+    else:
+        raise Http404
+    return redirect('membres-club', name=club_name)
 
 
 @cof_required