Refactoring: utiliser la façon habituelle de créer des formsets

[auf_rh_dae.git] / project / dae / forms.py

diff --git a/project/dae/forms.py b/project/dae/forms.py
index d0402d1..159ec77 100644 (file)
--- a/project/dae/forms.py
+++ b/project/dae/forms.py
@@ -69,21 +69,14 @@ def _employe_choices(obj, request):
                   [('rh-%s' % p.id, option_label(p)) for p in rhv1],
                   key=lambda t: t[1])
 
-
 def label_poste_display(poste):
     """Formate un visuel pour un poste dans une liste déroulante"""
     label = u"%s - %s [%s]" %(poste.type_poste, poste.type_poste.famille_emploi.nom, poste.id)
     return label
 
-class PostePieceForm(inlineformset_factory(dae.Poste, dae.PostePiece)):
-    pass
-
-class DossierPieceForm(inlineformset_factory(dae.Dossier, dae.DossierPiece)):
-    pass
-
-class FinancementForm(inlineformset_factory(dae.Poste, dae.PosteFinancement, extra=2)):
-    pass
-
+PostePieceForm = inlineformset_factory(dae.Poste, dae.PostePiece)
+DossierPieceForm = inlineformset_factory(dae.Dossier, dae.DossierPiece)
+FinancementForm = inlineformset_factory(dae.Poste, dae.PosteFinancement, extra=2)
 
 class DossierComparaisonForm(forms.ModelForm):
     
@@ -93,8 +86,9 @@ class DossierComparaisonForm(forms.ModelForm):
     class Model:
         model = dae.DossierComparaison
 
-class DossierComparaisonForm(inlineformset_factory(dae.Dossier, dae.DossierComparaison, extra=3, max_num=3, form=DossierComparaisonForm)):
-    pass
+DossierComparaisonForm = inlineformset_factory(
+    dae.Dossier, dae.DossierComparaison, extra=3, max_num=3, form=DossierComparaisonForm
+)
 
 class PosteComparaisonForm(forms.ModelForm):
     
@@ -103,8 +97,9 @@ class PosteComparaisonForm(forms.ModelForm):
     class Model:
         model = dae.PosteComparaison
 
-class PosteComparaisonForm(inlineformset_factory(dae.Poste, dae.PosteComparaison, extra=3, max_num=3, form=PosteComparaisonForm)):
-    pass
+PosteComparaisonForm = inlineformset_factory(
+    dae.Poste, dae.PosteComparaison, extra=3, max_num=3, form=PosteComparaisonForm
+)
 
 class FlexibleRemunForm(forms.ModelForm):
 
@@ -125,9 +120,9 @@ class FlexibleRemunForm(forms.ModelForm):
         else:
             return devise
 
-
-class RemunForm(inlineformset_factory(dae.Dossier, dae.Remuneration, extra=5, form=FlexibleRemunForm)):
-    pass
+RemunForm = inlineformset_factory(
+    dae.Dossier, dae.Remuneration, extra=5, form=FlexibleRemunForm
+)
 
 class PosteForm(forms.ModelForm):
     """ Formulaire des postes. """