Change media permissions

media/views.py

@@ -10,7 +10,6 @@ from django.shortcuts import render, redirect
 from django.template.context_processors import csrf
 from django.utils import timezone
 from reversion import revisions as reversion
-from reversion.models import Version
 
 from med.settings import PAGINATION_NUMBER
 from users.models import User
@@ -25,7 +24,7 @@ def form(ctx, template, request):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.add_auteur')
 def add_auteur(request):
     auteur = AuteurForm(request.POST or None)
     if auteur.is_valid():
@@ -39,7 +38,7 @@ def add_auteur(request):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.edit_auteur')
 def edit_auteur(request, auteurid):
     try:
         auteur_instance = Auteur.objects.get(pk=auteurid)
@@ -51,14 +50,15 @@ def edit_auteur(request, auteurid):
         with transaction.atomic(), reversion.create_revision():
             auteur.save()
             reversion.set_user(request.user)
-            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(field for field in auteur.changed_data))
+            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(
+                field for field in auteur.changed_data))
         messages.success(request, "Auteur modifié")
         return redirect("/media/index_auteurs/")
     return form({'mediaform': auteur}, 'media/media.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.delete_auteur')
 def del_auteur(request, auteurid):
     try:
         auteur_instance = Auteur.objects.get(pk=auteurid)
@@ -71,11 +71,12 @@ def del_auteur(request, auteurid):
             reversion.set_user(request.user)
             messages.success(request, "L'auteur a été détruit")
         return redirect("/media/index_auteurs")
-    return form({'objet': auteur_instance, 'objet_name': 'auteur'}, 'media/delete.html', request)
+    return form({'objet': auteur_instance, 'objet_name': 'auteur'},
+                'media/delete.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.add_media')
 def add_media(request):
     media = MediaForm(request.POST or None)
     if media.is_valid():
@@ -89,7 +90,7 @@ def add_media(request):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.edit_media')
 def edit_media(request, mediaid):
     try:
         media_instance = Media.objects.get(pk=mediaid)
@@ -101,14 +102,15 @@ def edit_media(request, mediaid):
         with transaction.atomic(), reversion.create_revision():
             media.save()
             reversion.set_user(request.user)
-            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(field for field in media.changed_data))
+            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(
+                field for field in media.changed_data))
         messages.success(request, "Media modifié")
         return redirect("/media/index_medias/")
     return form({'mediaform': media}, 'media/media.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.delete_media')
 def del_media(request, mediaid):
     try:
         media_instance = Media.objects.get(pk=mediaid)
@@ -121,11 +123,12 @@ def del_media(request, mediaid):
             reversion.set_user(request.user)
             messages.success(request, "Le media a été détruit")
         return redirect("/media/index_medias")
-    return form({'objet': media_instance, 'objet_name': 'media'}, 'media/delete.html', request)
+    return form({'objet': media_instance, 'objet_name': 'media'},
+                'media/delete.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.add_jeu')
 def add_jeu(request):
     jeu = JeuForm(request.POST or None)
     if jeu.is_valid():
@@ -139,7 +142,7 @@ def add_jeu(request):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.edit_jeu')
 def edit_jeu(request, jeuid):
     try:
         jeu_instance = Jeu.objects.get(pk=jeuid)
@@ -151,14 +154,15 @@ def edit_jeu(request, jeuid):
         with transaction.atomic(), reversion.create_revision():
             jeu.save()
             reversion.set_user(request.user)
-            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(field for field in jeu.changed_data))
+            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(
+                field for field in jeu.changed_data))
         messages.success(request, "Media modifié")
         return redirect("/media/index_jeux/")
     return form({'mediaform': jeu}, 'media/media.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.delete_jeu')
 def del_jeu(request, jeuid):
     try:
         jeu_instance = Jeu.objects.get(pk=jeuid)
@@ -171,20 +175,23 @@ def del_jeu(request, jeuid):
             reversion.set_user(request.user)
             messages.success(request, "Le jeu a été détruit")
         return redirect("/media/index_jeux")
-    return form({'objet': jeu_instance, 'objet_name': 'jeu'}, 'media/delete.html', request)
+    return form({'objet': jeu_instance, 'objet_name': 'jeu'},
+                'media/delete.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.add_emprunt')
 def add_emprunt(request, userid):
     try:
         user = User.objects.get(pk=userid)
     except User.DoesNotExist:
         messages.error(request, u"Entrée inexistante")
         return redirect("/media/index_emprunts/")
-    emprunts_en_cours = Emprunt.objects.filter(date_rendu=None, user=user).count()
+    emprunts_en_cours = Emprunt.objects.filter(date_rendu=None,
+                                               user=user).count()
     if emprunts_en_cours >= user.maxemprunt:
-        messages.error(request, "Maximum d'emprunts atteint de l'user %s" % user.maxemprunt)
+        messages.error(request,
+                       "Maximum d'emprunts atteint de l'user %s" % user.maxemprunt)
         return redirect("/media/index_emprunts/")
     emprunt = EmpruntForm(request.POST or None)
     if emprunt.is_valid():
@@ -202,7 +209,7 @@ def add_emprunt(request, userid):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.edit_emprunt')
 def edit_emprunt(request, empruntid):
     try:
         emprunt_instance = Emprunt.objects.get(pk=empruntid)
@@ -214,14 +221,15 @@ def edit_emprunt(request, empruntid):
         with transaction.atomic(), reversion.create_revision():
             emprunt.save()
             reversion.set_user(request.user)
-            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(field for field in emprunt.changed_data))
+            reversion.set_comment("Champs modifié(s) : %s" % ', '.join(
+                field for field in emprunt.changed_data))
         messages.success(request, "Emprunt modifié")
         return redirect("/media/index_emprunts/")
     return form({'mediaform': emprunt}, 'media/media.html', request)
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.edit_emprunt')
 def retour_emprunt(request, empruntid):
     try:
         emprunt_instance = Emprunt.objects.get(pk=empruntid)
@@ -238,7 +246,7 @@ def retour_emprunt(request, empruntid):
 
 
 @login_required
-@permission_required('perm')
+@permission_required('media.delete_emprunt')
 def del_emprunt(request, empruntid):
     try:
         emprunt_instance = Emprunt.objects.get(pk=empruntid)
@@ -251,7 +259,8 @@ def del_emprunt(request, empruntid):
             reversion.set_user(request.user)
             messages.success(request, "L'emprunt a été détruit")
         return redirect("/media/index_emprunts")
-    return form({'objet': emprunt_instance, 'objet_name': 'emprunt'}, 'media/delete.html', request)
+    return form({'objet': emprunt_instance, 'objet_name': 'emprunt'},
+                'media/delete.html', request)
 
 
 @login_required
@@ -283,7 +292,8 @@ def index_auteurs(request):
     except EmptyPage:
         # If page is out of range (e.g. 9999), deliver last page of results.
         auteurs_list = paginator.page(paginator.num_pages)
-    return render(request, 'media/index_auteurs.html', {'auteurs_list': auteurs_list})
+    return render(request, 'media/index_auteurs.html',
+                  {'auteurs_list': auteurs_list})
 
 
 @login_required
@@ -299,7 +309,8 @@ def index_medias(request):
     except EmptyPage:
         # If page is out of range (e.g. 9999), deliver last page of results.
         medias_list = paginator.page(paginator.num_pages)
-    return render(request, 'media/index_medias.html', {'medias_list': medias_list})
+    return render(request, 'media/index_medias.html',
+                  {'medias_list': medias_list})
 
 
 @login_required
@@ -308,7 +319,8 @@ def index(request):
         emprunts_list = Emprunt.objects.all()
     else:
         emprunts_list = Emprunt.objects.filter(user=request.user)
-    paginator = Paginator(emprunts_list.order_by('date_emprunt').reverse(), PAGINATION_NUMBER)
+    paginator = Paginator(emprunts_list.order_by('date_emprunt').reverse(),
+                          PAGINATION_NUMBER)
     page = request.GET.get('page')
     try:
         emprunts_list = paginator.page(page)
@@ -318,44 +330,5 @@ def index(request):
     except EmptyPage:
         # If page is out of range (e.g. 9999), deliver last page of results.
         emprunts_list = paginator.page(paginator.num_pages)
-    return render(request, 'media/index_emprunts.html', {'emprunts_list': emprunts_list})
-
-
-@login_required
-def history(request, object, id):
-    if object == 'auteur':
-        try:
-            object_instance = Auteur.objects.get(pk=id)
-        except Auteur.DoesNotExist:
-            messages.error(request, "Auteur inexistant")
-            return redirect("/media/index_auteurs")
-    elif object == 'media':
-        try:
-            object_instance = Media.objects.get(pk=id)
-        except Media.DoesNotExist:
-            messages.error(request, "Media inexistant")
-            return redirect("/media/index_medias")
-    elif object == 'emprunt':
-        try:
-            object_instance = Emprunt.objects.get(pk=id)
-        except Emprunt.DoesNotExist:
-            messages.error(request, "Emprunt inexistant")
-            return redirect("/media/index_emprunts")
-    elif object == 'jeu':
-        try:
-            object_instance = Jeu.objects.get(pk=id)
-        except Jeu.DoesNotExist:
-            messages.error(request, "Jeu inexistant")
-            return redirect("/media/index_jeux")
-    reversions = Version.objects.get_for_object(object_instance)
-    paginator = Paginator(reversions, PAGINATION_NUMBER)
-    page = request.GET.get('page')
-    try:
-        reversions = paginator.page(page)
-    except PageNotAnInteger:
-        # If page is not an integer, deliver first page.
-        reversions = paginator.page(1)
-    except EmptyPage:
-        # If page is out of range (e.g. 9999), deliver last page of results.
-        reversions = paginator.page(paginator.num_pages)
-    return render(request, 'media/history.html', {'reversions': reversions, 'object': object_instance})
+    return render(request, 'media/index_emprunts.html',
+                  {'emprunts_list': emprunts_list})