Fewer requests on bda.views.revente

bda/forms.py

@@ -15,7 +15,7 @@ class TokenForm(forms.Form):
 
 class AttributionModelMultipleChoiceField(forms.ModelMultipleChoiceField):
     def label_from_instance(self, obj):
-        return "%s" % obj.spectacle
+        return "%s" % str(obj.spectacle)
 
 
 class ResellForm(forms.Form):
@@ -27,9 +27,13 @@ class ResellForm(forms.Form):
 
     def __init__(self, participant, *args, **kwargs):
         super(ResellForm, self).__init__(*args, **kwargs)
-        self.fields['attributions'].queryset = participant.attribution_set\
+        self.fields['attributions'].queryset = (
-            .filter(spectacle__date__gte=timezone.now())\
+            participant.attribution_set
+            .filter(spectacle__date__gte=timezone.now())
             .exclude(revente__seller=participant)
+            .select_related('spectacle', 'spectacle__location',
+                            'participant__user')
+        )
 
 
 class AnnulForm(forms.Form):
@@ -41,11 +45,15 @@ class AnnulForm(forms.Form):
 
     def __init__(self, participant, *args, **kwargs):
         super(AnnulForm, self).__init__(*args, **kwargs)
-        self.fields['attributions'].queryset = participant.attribution_set\
+        self.fields['attributions'].queryset = (
+            participant.attribution_set
             .filter(spectacle__date__gte=timezone.now(),
                     revente__isnull=False,
                     revente__notif_sent=False,
                     revente__soldTo__isnull=True)
+            .select_related('spectacle', 'spectacle__location',
+                            'participant__user')
+        )
 
 
 class InscriptionReventeForm(forms.Form):
@@ -56,8 +64,11 @@ class InscriptionReventeForm(forms.Form):
 
     def __init__(self, tirage, *args, **kwargs):
         super(InscriptionReventeForm, self).__init__(*args, **kwargs)
-        self.fields['spectacles'].queryset = tirage.spectacle_set.filter(
+        self.fields['spectacles'].queryset = (
-                date__gte=timezone.now())
+            tirage.spectacle_set
+            .select_related('location')
+            .filter(date__gte=timezone.now())
+        )
 
 
 class SoldForm(forms.Form):
@@ -73,4 +84,6 @@ class SoldForm(forms.Form):
             .filter(revente__isnull=False,
                     revente__soldTo__isnull=False)
             .exclude(revente__soldTo=participant)
+            .select_related('spectacle', 'spectacle__location',
+                            'participant__user')
         )

bda/templates/bda/reventes.html

@@ -4,6 +4,8 @@
 {% block realcontent %}
 
 <h2>Revente de place</h2>
+{% with resell_attributions=resellform.attributions annul_attributions=annulform.attributions sold_attributions=soldform.attributions %}
+
 {% if resellform.attributions %}
 <h3>Places non revendues</h3>
 <form class="form-horizontal" action="" method="post">
@@ -15,14 +17,14 @@
 </form>
 {% endif %}
 <br>
-{% if annulform.attributions or overdue %}
+{% if annul_attributions or overdue %}
 <h3>Places en cours de revente</h3>
 <form action="" method="post">
     {% csrf_token %}
     <div class='form-group'>
         <div class='multiple-checkbox'>
             <ul>
-                {% for attrib in annulform.attributions %}
+                {% for attrib in annul_attributions %}
                     <li>{{attrib.tag}} {{attrib.choice_label}}</li>
                 {% endfor %}
                 {% for attrib in overdue %}
@@ -31,13 +33,13 @@
                         {{attrib.spectacle}}
                     </li>
                 {% endfor %}
-    {% if annulform.attributions %}
+    {% if annul_attributions %}
     <input type="submit" class="btn btn-primary" name="annul" value="Annuler les reventes sélectionnées">
     {% endif %}
 </form>
 {% endif %}
 <br>
-{% if soldform.attributions %}
+{% if sold_attributions %}
 <h3>Places revendues</h3>
     <form action="" method="post">
         {% csrf_token %}
@@ -46,8 +48,9 @@
         <button type="submit" class="btn btn-primary" name="reinit">Réinitialiser</button>
     </form>
 {% endif %}
-{% if not resellform.attributions and not soldform.attributions and not overdue and not annulform.attributions %}
+{% if not resell_attributions and not annul_attributions and not overdue and not sold_attributions %}
 <p>Plus de reventes possibles !</p>
 {% endif %}
 
+{% endwith %}
 {% endblock %}