Add return button/action to loans list

coin/static/css/local.css

@@ -201,6 +201,13 @@ table tr.placeholder td {
 	font-style: italic;
 }
 
+table .actions {
+	text-align: center;
+}
+
+table .actions .button {
+	width: 100%;
+}
 /* Footer */
 
 #footer {

hardware_provisioning/models.py

@@ -87,6 +87,9 @@ class Loan(models.Model):
         return 'prêt de {item} à {user}'.format(item=self.item,
                                                 user=self.user)
 
+    def user_can_close(self, user):
+        return (not self.item.is_available()) and (self.user == user)
+
     class Meta:
         verbose_name = 'prêt d’objet'
         verbose_name_plural = 'prêts d’objets'

hardware_provisioning/templates/hardware_provisioning/list.html

@@ -6,6 +6,9 @@
 {% else %}
 <h2>L'association me prête…</h2>
 {% endif %}
+{% for message in messages %}
+<div class="message {{ message.tags }}">{{ message }}</div>
+{% endfor %}
 
 <table id="member_loans" class="full-width">
     <thead>
@@ -16,6 +19,7 @@
             {% if view == 'old' %}<th>Date retour</th>{% endif %}
             <th>Addr. MAC</th>
             <th>Num. de série</th>
+            {% if view != 'old' %}<th>Actions</th>{% endif %}
         </tr>
     </thead>
     <tbody>
@@ -27,6 +31,15 @@
             {% if view == 'old' %}<td>{{ loan.loan_date_end }}</td>{% endif %}
             <td>{{ loan.item.mac_address|default:"n/a" }}</td>
             <td>{{ loan.item.serial|default:"n/a" }}</td>
+            {% if view != 'old' %}
+            <td class="actions">
+              <div class="button-group">
+                  <a href="{% url 'hardware_provisioning:loan-return' pk=loan.pk %}" class="small button">
+                     <i class="fa fa-times"></i>&nbsp;Rendre
+                 </a>
+              </div>
+            </td>
+            {% endif %}
         </tr>
         {% empty %}
         <tr class="placeholder"><td colspan="6">… rien du tout !</td></tr>

hardware_provisioning/templates/hardware_provisioning/return.html

@@ -0,0 +1,12 @@
+{% extends "base.html" %}
+
+{% block content %}
+<h2>Retour de matériel</h2>
+<p>
+    J'ai retourné le matériel <strong>{{loan.item }} ({{ loan.item.type }}</strong>) a l'association ce jour.
+</p>
+<form method='post'>{% csrf_token %}
+  <input class="success button" type="submit"
+         value="Oui oui, c'est bien ça."/>
+</form>
+{% endblock %}

hardware_provisioning/urls.py

@@ -7,5 +7,6 @@ from . import views
 
 urlpatterns = [
     url(r'^$', views.loan_list, name='loan-list'),
+    url(r'^(?P<pk>[0-9]+)/return$', views.loan_return, name='loan-return'),
     url(r'^(?P<pk>[0-9]+)$', views.loan_detail, name='loan-detail'),
 ]

hardware_provisioning/views.py

@@ -1,12 +1,39 @@
 # -*- coding: utf-8 -*-
 
 from __future__ import unicode_literals
-from django.shortcuts import render
+
+from django.shortcuts import render, redirect
 from django.contrib.auth.decorators import login_required
+from django.contrib import messages
+from django.http import HttpResponseForbidden
+from django.core.urlresolvers import reverse
+
+
 from .models import Item, Loan
 
 
 @login_required
+def loan_return(request, pk):
+    loan = Loan.objects.get(pk=pk)
+
+    if not loan.user_can_close(request.user):
+        return HttpResponseForbidden('Non autorisé')
+
+    if request.method == 'POST':
+        messages.success(
+            request,
+            'Le matériel {} a été marqué comme rendu'.format(
+                loan.item))
+        loan.item.give_back()
+        return redirect(reverse('hardware_provisioning:loan-list'))
+
+    else:
+        return render(request, 'hardware_provisioning/return.html', {
+            'loan': loan,
+        })
+
+
+@login_required
 def loan_list(request):
     view = 'old' if 'old' in request.GET else ''