1908: Cleaning code

diff --git a/project/recrutement/admin.py b/project/recrutement/admin.py
index 546326a..109b9cd 100644 (file)
--- a/project/recrutement/admin.py
+++ b/project/recrutement/admin.py
@@ -34,8 +34,8 @@ class OffreEmploiAdmin(VersionAdmin):
     date_hierarchy = 'date_creation'
     list_display = ('nom', 'date_limite', 'region',  'statut', 
                     'est_affiche', '_candidatsList', )
     date_hierarchy = 'date_creation'
     list_display = ('nom', 'date_limite', 'region',  'statut', 
                     'est_affiche', '_candidatsList', )

-    exclude = ('poste_nom', 'resume',)
-    list_filter = ('statut', 'est_affiche', )
+    exclude = ('actif', 'poste_nom', 'resume',)
+    list_filter = ('statut',)

     actions = ['affecter_evaluateurs_offre_emploi', ]
     form = OffreEmploiForm
     inlines = [EvaluateurInline, ]
     actions = ['affecter_evaluateurs_offre_emploi', ]
     form = OffreEmploiForm
     inlines = [EvaluateurInline, ]

@@ -107,10 +107,9 @@ class OffreEmploiAdmin(VersionAdmin):
 class ProxyOffreEmploiAdmin(OffreEmploiAdmin):
     list_display = ('nom', 'resume', 'date_limite', 'region', 'statut', 
                     'est_affiche')
 class ProxyOffreEmploiAdmin(OffreEmploiAdmin):
     list_display = ('nom', 'resume', 'date_limite', 'region', 'statut', 
                     'est_affiche')

-    readonly_fields = ('description', 'bureau',
-                        'duree_affectation', 'renumeration',
-                        'debut_affectation', 'lieu_affectation', 'nom',
-                        'resume', 'date_limite', 'region')
+    readonly_fields = ('description', 'bureau', 'duree_affectation', 
+                        'renumeration', 'debut_affectation', 'lieu_affectation',
+                        'nom', 'resume', 'date_limite', 'region')

     fieldsets = (
         ('Nom', {
             'fields': ('nom', )        
     fieldsets = (
         ('Nom', {
             'fields': ('nom', )        

@@ -187,11 +186,13 @@ class CandidatEvaluationInline(admin.TabularInline):
         return self.readonly_fields
 
 class CandidatAdmin(VersionAdmin):
         return self.readonly_fields
 
 class CandidatAdmin(VersionAdmin):

+    exclude = ('actif', )

     date_hierarchy = 'date_creation'
     list_display = ('nom', 'prenom', 'offre_emploi','statut',
                     'voir_offre_emploi', 'calculer_moyenne', 
                     'afficher_candidat',)
     list_filter = ('offre_emploi', )
     date_hierarchy = 'date_creation'
     list_display = ('nom', 'prenom', 'offre_emploi','statut',
                     'voir_offre_emploi', 'calculer_moyenne', 
                     'afficher_candidat',)
     list_filter = ('offre_emploi', )

+

     fieldsets = (
         ("Offre d'emploi", {
             'fields': ('offre_emploi', )
     fieldsets = (
         ("Offre d'emploi", {
             'fields': ('offre_emploi', )

@@ -263,7 +264,7 @@ class CandidatAdmin(VersionAdmin):
         notes = [evaluation.note for evaluation in evaluations.all() \
                     if evaluation.note is not None]
  
         notes = [evaluation.note for evaluation in evaluations.all() \
                     if evaluation.note is not None]
  

-        if len(notes) > 0 and offre_emploi.date_limite <= datetime.date.today():
+        if len(notes) > 0:

             moyenne_votes = float(sum(notes)) / len(notes)
         else:
             moyenne_votes = "Non disponible"
             moyenne_votes = float(sum(notes)) / len(notes)
         else:
             moyenne_votes = "Non disponible"

@@ -435,20 +436,19 @@ class CandidatEvaluationAdmin(VersionAdmin):
         un lien pour Évaluer le candidat.
         Sinon afficher la note.
         """
         un lien pour Évaluer le candidat.
         Sinon afficher la note.
         """

-        evaluateur = obj.evaluateur
-        candidat = obj.candidat
-        candidat_evaluation = CandidatEvaluation.objects.\
-                                get(candidat=candidat, evaluateur=evaluateur)
+        #evaluateur = obj.evaluateur
+        #candidat = obj.candidat
+        #candidat_evaluation = CandidatEvaluation.objects.get(candidat=candidat, evaluateur=evaluateur)
+        #import pdb;pdb.set_trace()

         if obj.note is None:
             return "<a href='%s'>Candidat non évalué</a>" % \
                 (reverse('admin:recrutement_candidatevaluation_change', 
         if obj.note is None:
             return "<a href='%s'>Candidat non évalué</a>" % \
                 (reverse('admin:recrutement_candidatevaluation_change', 

-                args=(candidat_evaluation.id,)))
+                args=(obj.id,)))

         return "<a href='%s'>%s</a>" % \
             (reverse('admin:recrutement_candidatevaluation_change', 
             args=(candidat_evaluation.id,)), obj.note)
         return "<a href='%s'>%s</a>" % \
             (reverse('admin:recrutement_candidatevaluation_change', 
             args=(candidat_evaluation.id,)), obj.note)

-        return 

     _note.allow_tags = True
     _note.allow_tags = True

-    _note.short_description = "Votre note"    
+    _note.short_description = "Note"    

     _note.admin_order_field = 'note'    
 
     def _candidat(self, obj):
     _note.admin_order_field = 'note'    
 
     def _candidat(self, obj):

diff --git a/project/recrutement/models.py b/project/recrutement/models.py
index e1bc8cf..7789294 100755 (executable)
--- a/project/recrutement/models.py
+++ b/project/recrutement/models.py
@@ -55,7 +55,7 @@ class Candidat(emploi.Candidat):
         notes = [evaluation.note for evaluation in evaluations.all() \
                     if evaluation.note is not None]
 
         notes = [evaluation.note for evaluation in evaluations.all() \
                     if evaluation.note is not None]
 

-        if len(notes) > 0 and offre_emploi.date_limite <= datetime.date.today():
+        if len(notes) > 0:

             moyenne_votes = float(sum(notes)) / len(notes)
         else:
             moyenne_votes = "Non disponible"
             moyenne_votes = float(sum(notes)) / len(notes)
         else:
             moyenne_votes = "Non disponible"

@@ -160,6 +160,8 @@ class CourrielTemplate(models.Model):
 
     class Meta:
         ordering = ['nom_modele',]
 
     class Meta:
         ordering = ['nom_modele',]

+        verbose_name = "Modèle de courriel"
+        verbose_name_plural = "Modèles de courriel"

 
 class CandidatCourriel(models.Model):
     candidats = models.ManyToManyField(Candidat, verbose_name="Candidats", )
 
 class CandidatCourriel(models.Model):
     candidats = models.ManyToManyField(Candidat, verbose_name="Candidats", )