Cleaning de code inutile
authorNilovna Bascunan-Vasquez <contact@nilovna.com>
Mon, 8 Aug 2011 14:54:38 +0000 (10:54 -0400)
committerNilovna Bascunan-Vasquez <contact@nilovna.com>
Mon, 8 Aug 2011 14:54:38 +0000 (10:54 -0400)
project/recrutement/admin.py
project/recrutement/api.py
project/recrutement/forms.py

index ddbddd8..546326a 100644 (file)
@@ -16,13 +16,6 @@ from recrutement.workflow import grp_administrateurs_recrutement,\
                             grp_evaluateurs_recrutement, grp_drh_recrutement
 from recrutement.forms import *
 
-"""
-class MetaAdmin(VersionAdmin):
-    def get_actions(self, request):
-    
-Pour refactoring
-"""
-
 class ProxyEvaluateur(Evaluateur.offres_emploi.through):
     """
     Ce proxy sert uniquement dans l'admin à disposer d'un libellé
@@ -133,19 +126,22 @@ class ProxyOffreEmploiAdmin(OffreEmploiAdmin):
                         'renumeration', )
         }),
     )        
-
+    inlines = []
     def get_actions(self, request):
         actions = super(ProxyOffreEmploiAdmin, self).get_actions(request)
         del actions['affecter_evaluateurs_offre_emploi']
         return actions
 
     def response_change(self, request, obj):
-        response = super(ProxyOffreEmploiAdmin, self).response_change(request, obj)
+        response = super(ProxyOffreEmploiAdmin, self).response_change(request, 
+                                                                            obj)
         user_groupes = request.user.groups.all()
         if grp_drh_recrutement in user_groupes or \
             request.user.is_superuser:
-            return HttpResponseRedirect(reverse('admin:recrutement_offreemploi_changelist'))
-        return HttpResponseRedirect(reverse('admin:recrutement_proxyoffreemploi_changelist'))
+            return HttpResponseRedirect(reverse\
+                                ('admin:recrutement_offreemploi_changelist'))
+        return HttpResponseRedirect(reverse\
+                            ('admin:recrutement_proxyoffreemploi_changelist'))
 
     def has_add_permission(self, request):
         return False
@@ -364,8 +360,10 @@ class ProxyCandidatAdmin(CandidatAdmin):
         user_groupes = request.user.groups.all()
         if grp_drh_recrutement in user_groupes or \
             request.user.is_superuser:
-            return HttpResponseRedirect(reverse('admin:recrutement_candidat_changelist'))
-        return HttpResponseRedirect(reverse('admin:recrutement_proxycandidat_changelist'))
+            return HttpResponseRedirect(reverse\
+                                    ('admin:recrutement_candidat_changelist'))
+        return HttpResponseRedirect(reverse\
+                                ('admin:recrutement_proxycandidat_changelist'))
 
     def has_add_permission(self, request):
         return False
index acc68d1..a6adbdf 100644 (file)
@@ -69,19 +69,20 @@ class API:
         offre = emploi.OffreEmploi.objects.get(id=offre_id)
         if self.request.method == "POST":
             try:
-
                 candidat = emploi.Candidat()
                 candidat.offre_emploi = offre
                 candidat.nom = self.request.POST['nom']
                 candidat.prenom = self.request.POST['prenom']
                 candidat.genre = self.request.POST['genre']
-                candidat.nationalite = ref.Pays.objects.get(id=self.request.POST['nationalite'])
+                candidat.nationalite = ref.Pays.objects.get\
+                                        (id=self.request.POST['nationalite'])
                 candidat.situation_famille = self.request.POST['situation_famille']
                 candidat.nombre_dependant = self.request.POST['nombre_dependant']
                 candidat.niveau_diplome = self.request.POST['niveau_diplome']
                 candidat.employeur_actuel = self.request.POST['employeur_actuel']
                 candidat.poste_actuel = self.request.POST['poste_actuel']
-                candidat.domaine_professionnel = self.request.POST['domaine_professionnel']
+                candidat.domaine_professionnel = self.request.\
+                                                    POST['domaine_professionnel']
                 candidat.telephone = self.request.POST['telephone']
                 candidat.email = self.request.POST['email']
                 candidat.adresse = self.request.POST['adresse']
@@ -118,20 +119,20 @@ class API:
                 offres_emploi.append(offre)
         if offres_emploi:
             return api_return(STATUS_OK, simplejson.dumps(
-                                [{"id": "%s" % offre.id, 
-                                "est_affiche": "%s" % offre.est_affiche,
-                                "statut": "%s" % offre.statut,
-                                "nom": "%s" % offre.nom, 
-                                "resume": "%s" % offre.resume,
-                                "description": "%s" % offre.description,
-                                "poste_nom": "%s" % offre.poste_nom,
-                                "region": "%s" % offre.region.id,
-                                "bureau": "%s" % offre.bureau.id,
-                                "date_limite": "%s" % offre.date_limite,
-                                "duree_affectation": "%s" % offre.duree_affectation,
-                                "renumeration": "%s" % offre.renumeration,
-                                "debut_affectation": "%s" % offre.debut_affectation,
-                                "lieu_affectation": "%s" % offre.lieu_affectation.id} 
+                        [{"id": "%s" % offre.id, 
+                        "est_affiche": "%s" % offre.est_affiche,
+                        "statut": "%s" % offre.statut,
+                        "nom": "%s" % offre.nom, 
+                        "resume": "%s" % offre.resume,
+                        "description": "%s" % offre.description,
+                        "poste_nom": "%s" % offre.poste_nom,
+                        "region": "%s" % offre.region.id,
+                        "bureau": "%s" % offre.bureau.id,
+                        "date_limite": "%s" % offre.date_limite,
+                        "duree_affectation": "%s" % offre.duree_affectation,
+                        "renumeration": "%s" % offre.renumeration,
+                        "debut_affectation": "%s" % offre.debut_affectation,
+                        "lieu_affectation": "%s" % offre.lieu_affectation.id} 
                                 for offre in offres_emploi]), json=True)
         return api_return(STATUS_OK)
         
@@ -158,6 +159,7 @@ class API:
                 "duree_affectation": "%s" % offre.duree_affectation,
                 "renumeration": "%s" % offre.renumeration,
                 "debut_affectation": "%s" % offre.debut_affectation,
-                "lieu_affectation": "%s" % offre.lieu_affectation.id} ), json=True)
+                "lieu_affectation": "%s" % offre.lieu_affectation.id} ), 
+                                                                    json=True)
         return api_return(STATUS_OK)
 
index 294013e..5b52584 100644 (file)
@@ -89,11 +89,6 @@ class OffreEmploiForm(ModelForm):
                 raise forms.ValidationError("La date limite ne peut pas être \
                         supérieure à la date d'affection.")
 
-        """if date_limite < datetime.date.today() or \
-            debut_affectation < datetime.date.today():
-            raise forms.ValidationError("La date limite et/ou la date \
-                d'affection doivent être supérieures à la date d'aujourdhui.")
-        """
         return cleaned_data
 
 ################################################################################