Ne proposer que les établissements actifs et ne pas planter s'il y a des

doublons.

diff --git a/auf_savoirs_en_partage/chercheurs/forms.py b/auf_savoirs_en_partage/chercheurs/forms.py
index 20dc802..7bdcc6b 100644 (file)
--- a/auf_savoirs_en_partage/chercheurs/forms.py
+++ b/auf_savoirs_en_partage/chercheurs/forms.py
@@ -93,12 +93,12 @@ class ChercheurForm(forms.ModelForm):
     def save(self):
         nom_etablissement = self.cleaned_data['etablissement']
         pays_etablissement = self.cleaned_data['pays_etablissement']
-        try:
-            etablissement = Etablissement.objects.get(nom=nom_etablissement, pays=pays_etablissement)
-            self.instance.etablissement = etablissement
+        etablissements = Etablissement.objects.filter(nom=nom_etablissement, pays=pays_etablissement, actif=True)
+        if etablissements.count() > 0:
+            self.instance.etablissement = etablissements[0]
             self.instance.etablissement_autre = ''
             self.instance.etablissement_autre_pays = None
-        except Etablissement.DoesNotExist:
+        else:
             self.instance.etablissement = None
             self.instance.etablissement_autre_nom = nom_etablissement
             self.instance.etablissement_autre_pays = pays_etablissement

diff --git a/auf_savoirs_en_partage/chercheurs/views.py b/auf_savoirs_en_partage/chercheurs/views.py
index 2f1b0ce..5363d32 100644 (file)
--- a/auf_savoirs_en_partage/chercheurs/views.py
+++ b/auf_savoirs_en_partage/chercheurs/views.py
@@ -144,7 +144,7 @@ def conversion(request):
 
 def etablissements_autocomplete(request, pays=None):
     term = request.GET.get('term')
-    noms = Etablissement.objects.all().filter(membre=True)
+    noms = Etablissement.objects.all().filter(membre=True, actif=True)
     for word in term.split():
         noms = noms.filter(nom__icontains=word)
     if pays: