quelques ajustements pep 8
authorNicolas Cadou <ncadou@cadou.ca>
Wed, 6 Apr 2011 00:34:34 +0000 (20:34 -0400)
committerNicolas Cadou <ncadou@cadou.ca>
Wed, 6 Apr 2011 00:34:34 +0000 (20:34 -0400)
project/dae/forms.py
project/dae/models.py
project/dae/views.py

index 1a444cb..85a45ed 100644 (file)
@@ -28,10 +28,10 @@ class PosteForm(forms.ModelForm):
                   'comp_autre_min', 'comp_autre_max',)
         widgets = dict(statut_residence=forms.RadioSelect(),
                        appel=forms.RadioSelect(),
-                       nom=forms.TextInput(attrs={'size': 60 }),
+                       nom=forms.TextInput(attrs={'size': 60}),
 )
 
-    responsable=forms.ModelChoiceField(
+    responsable = forms.ModelChoiceField(
             queryset=rh.Poste.objects.select_related(depth=1))
 
     # La liste des choix est laissée vide. Voir __init__ pour la raison.
@@ -67,7 +67,7 @@ class PosteForm(forms.ModelForm):
         # Optimisation de la requête
         rhv1 = rhv1.select_related(depth=1)
 
-        return [('', 'Nouveau poste')]+ \
+        return [('', 'Nouveau poste')] + \
                sorted([('dae-%s' % p.id, unicode(p)) for p in dae_ | copies] +
                       [('rh-%s' % p.id, unicode(p)) for p in rhv1],
                       key=lambda t: t[1])
@@ -100,7 +100,7 @@ class ChoosePosteForm(forms.ModelForm):
         copies = dae.Poste.objects.exclude(id_rh__isnull=True)
         id_copies = [p.id_rh_id for p in copies.all()]
 
-        return [('', '----------')]+ \
+        return [('', '----------')] + \
                sorted([('dae-%s' % p.id, unicode(p)) for p in dae_ | copies],
                       key=lambda t: t[1])
 
index 9e44988..077c890 100644 (file)
@@ -21,7 +21,6 @@ POSTE_STATUT_CHOICES = (
 )
 
 
-
 class PosteManager(models.Manager):
     """
     Chargement de tous les objets FK existants sur chaque QuerySet.
@@ -38,19 +37,22 @@ class PosteManager(models.Manager):
             'valeur_point_min',
             'valeur_point_max',
         )
-        return super(PosteManager, self).get_query_set().select_related(*fkeys).all()
+        return super(PosteManager, self).get_query_set() \
+                                        .select_related(*fkeys).all()
 
 
 class Poste(models.Model):
     # Modèle existant
     id_rh = models.ForeignKey(rh.Poste, null=True, related_name='+',
-                              editable=False, verbose_name="Mise à jour du poste")
+                              editable=False,
+                              verbose_name="Mise à jour du poste")
     nom = models.CharField(verbose_name="Titre du poste", max_length=255)
     implantation = models.ForeignKey(ref.Implantation)
     type_poste = models.ForeignKey(rh.TypePoste, null=True, related_name='+')
-    service = models.ForeignKey(rh.Service, verbose_name=u"Direction/Service/Pôle support", related_name='+')
-    responsable = models.ForeignKey(rh.Poste, verbose_name="Poste du responsable", related_name='+')
-
+    service = models.ForeignKey(rh.Service, related_name='+',
+                                verbose_name=u"Direction/Service/Pôle support")
+    responsable = models.ForeignKey(rh.Poste, related_name='+',
+                                    verbose_name="Poste du responsable")
 
     regime_travail = models.DecimalField(max_digits=12, decimal_places=2,
                                          default=100)
@@ -59,11 +61,13 @@ class Poste(models.Model):
                                                           default=40)
 
     # Recrutement
-    statut_residence = models.CharField(max_length=10, default='MAD', verbose_name="Recrutement",
+    statut_residence = models.CharField(max_length=10, default='MAD',
+                                        verbose_name="Recrutement",
                                         choices=STATUT_RESIDENCE_CHOICES)
     # TODO null?
     mise_a_disposition = models.BooleanField()
-    appel = models.CharField(max_length=10, default='interne', verbose_name="Appel à candidature",
+    appel = models.CharField(max_length=10, default='interne',
+                             verbose_name="Appel à candidature",
                              choices=POSTE_APPEL_CHOICES)
 
     # Rémunération
@@ -111,8 +115,10 @@ class Poste(models.Model):
     # Méta
     date_creation = models.DateTimeField(auto_now_add=True)
     date_modification = models.DateTimeField(auto_now=True)
-    date_debut = models.DateField(verbose_name="Date de début", help_text="format: aaaa-mm-jj")
-    date_fin = models.DateField(null=True, verbose_name="Date de fin", help_text="format: aaaa-mm-jj")
+    date_debut = models.DateField(verbose_name="Date de début",
+                                  help_text="format: aaaa-mm-jj")
+    date_fin = models.DateField(null=True, verbose_name="Date de fin",
+                                help_text="format: aaaa-mm-jj")
     actif = models.BooleanField(default=True)
 
     # Managers
@@ -248,7 +254,7 @@ class Remuneration(models.Model):
     # TODO: what's that?
     # type_revalorisation = models.ForeignKey('TypeRevalorisation',
                                             # db_column='type_revalorisation')
-    montant = models.DecimalField(max_digits=12, decimal_places=2) # Annuel
+    montant = models.DecimalField(max_digits=12, decimal_places=2)  # Annuel
     devise = models.ForeignKey(rh.Devise, to_field='code',
                                db_column='devise', related_name='+')
     date_effective = models.DateField(null=True, blank=True)
index f6f5ccb..3f7e8c1 100644 (file)
@@ -56,13 +56,16 @@ def poste(request, key=None):
 
     return render_to_response('dae/poste.html', vars, RequestContext(request))
 
+
 def postes_liste(request):
     """ Liste des postes
     """
     postes = dae.Poste.objects.all()
     vars = dict()
     vars['postes'] = postes
-    return render_to_response('dae/postes_liste.html', vars, RequestContext(request))
+    return render_to_response('dae/postes_liste.html', vars,
+                              RequestContext(request))
+
 
 def financement(request, key=None, id=None):
     """ Formulaire pour une source de financement pour un poste. """
@@ -134,7 +137,6 @@ def embauche(request, key=None):
                 employe_form.save()
                 #dossier_form = DossierForm(request)
 
-
     return render_to_response('dae/embauche.html', vars,
                               RequestContext(request))
 
@@ -209,7 +211,7 @@ def dossier(request, poste_key, employe_key):
                      | dossiers.filter(poste2=poste_rh)
             if len(dossiers):
                 # Ce bloc ignore toutes les erreurs, car les données de rh
-                # manquantes peuvent en générer 
+                # manquantes peuvent en générer
                 d = dossiers[0]
                 try:
                     titulaire = d.employe