From aa7ded9f20cce60c7bea368d6314088775320b12 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ma=C3=ABl=20Kervella?= <kervella.mael@gmail.com>
Date: Fri, 13 Apr 2018 17:30:48 +0000
Subject: [PATCH 1/2] Fix #95 by removing duplicates classes and functions

---
 cotisations/models.py |  3 +--
 machines/models.py    |  4 ++--
 users/admin.py        | 20 ++++----------------
 3 files changed, 7 insertions(+), 20 deletions(-)

diff --git a/cotisations/models.py b/cotisations/models.py
index 83f81d5b..082847e7 100644
--- a/cotisations/models.py
+++ b/cotisations/models.py
@@ -659,9 +659,8 @@ def cotisation_post_save(sender, **kwargs):
     regen('mailing')
 
 
-# TODO : should be name cotisation_post_delete
 @receiver(post_delete, sender=Cotisation)
-def vente_post_delete(sender, **kwargs):
+def cotisation_post_delete(sender, **kwargs):
     """
     Mark some services as needing a regeneration after the deletion of a
     cotisation. Indeed the membership status may have changed.
diff --git a/machines/models.py b/machines/models.py
index 5af11b36..2b4eb4a2 100644
--- a/machines/models.py
+++ b/machines/models.py
@@ -1452,7 +1452,7 @@ def iptype_post_save(sender, **kwargs):
 
 
 @receiver(post_save, sender=MachineType)
-def machine_post_save(sender, **kwargs):
+def machinetype_post_save(sender, **kwargs):
     """Mise à jour des interfaces lorsque changement d'attribution
     d'une machinetype (changement iptype parent)"""
     machinetype = kwargs['instance']
@@ -1539,7 +1539,7 @@ def srv_post_save(sender, **kwargs):
 
 
 @receiver(post_delete, sender=Srv)
-def text_post_delete(sender, **kwargs):
+def srv_post_delete(sender, **kwargs):
     """Regeneration dns après modification d'un SRV"""
     regen('dns')
 
diff --git a/users/admin.py b/users/admin.py
index a6b70009..727b129c 100644
--- a/users/admin.py
+++ b/users/admin.py
@@ -56,19 +56,6 @@ from .forms import (
 )
 
 
-class UserAdmin(admin.ModelAdmin):
-    """Administration d'un user"""
-    list_display = (
-        'surname',
-        'pseudo',
-        'email',
-        'school',
-        'shell',
-        'state'
-    )
-    search_fields = ('surname', 'pseudo')
-
-
 class LdapUserAdmin(admin.ModelAdmin):
     """Administration du ldapuser"""
     list_display = ('name', 'uidNumber', 'login_shell')
@@ -143,8 +130,9 @@ class UserAdmin(VersionAdmin, BaseUserAdmin):
         'is_admin',
         'shell'
     )
-    list_display = ('pseudo',)
-    list_filter = ()
+    # Need to reset the settings from BaseUserAdmin
+    # They are using fields we don't use like 'is_staff'
+    list_filter=()
     fieldsets = (
         (None, {'fields': ('pseudo', 'password')}),
         (
@@ -175,7 +163,7 @@ class UserAdmin(VersionAdmin, BaseUserAdmin):
             }
         ),
     )
-    search_fields = ('pseudo',)
+    search_fields = ('pseudo', 'surname')
     ordering = ('pseudo',)
     filter_horizontal = ()
 

From 4f1831d5f9a99058d82789af3131225eb5fac872 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ma=C3=ABl=20Kervella?= <kervella.mael@gmail.com>
Date: Sat, 14 Apr 2018 01:43:00 +0000
Subject: [PATCH 2/2] Little fix in cotis

---
 cotisations/views.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cotisations/views.py b/cotisations/views.py
index bb2540ae..631713e8 100644
--- a/cotisations/views.py
+++ b/cotisations/views.py
@@ -375,7 +375,7 @@ def credit_solde(request, user, userid):
         )
         return redirect(reverse('cotisations:index'))
     return form({
-        'factureform': facture,
+        'factureform': invoice,
         'action_name': _("Edit")
         }, 'cotisations/facture.html', request)