💿 Correction des acteurs digitaux (#1761)

* Fix digital acteurs

* WIP

Author: fabienheureux

jinja2/qfdmo/carte/_digital_results.html

@@ -25,7 +25,7 @@ <h5 class="fr-text--xl fr-text--bold fr-mb-0">{{ adresse.libelle }}</h5>
                 </p>
             </div>
         {% endfor %}
-        {% if acteurs.count() == 0 %}
+        {% if acteurs|length == 0 %}
             <p class="qf-italic">
                 Aucune solution en ligne n'a été trouvée pour votre recherche.
             </p>

qfdmo/models/acteur.py

@@ -260,11 +260,20 @@ def with_bonus(self):
         return self.annotate(bonus=Exists(bonus_label_qualite))
 
     def digital(self):
-        return (
+        # 1. Get ordered primary keys
+        # List here ensures queryset is not re-evaluated, causing
+        # a re-ordering and key error in the return below
+        pks = list(
             self.filter(acteur_type__code=DIGITAL_ACTEUR_CODE)
             .annotate(min_action_order=Min("proposition_services__action__order"))
+            .values("min_action_order", "pk")
             .order_by("min_action_order", "?")
+            .values_list("pk", flat=True)[:100]
         )
+        # 2. Fetch digital acteurs and preserver ordering.
+        ordered_pks = {pk: index for index, pk in enumerate(pks)}
+        instances = list(self.filter(pk__in=pks))
+        return sorted(instances, key=lambda x: ordered_pks[x.pk])
 
     def physical(self):
         return self.exclude(acteur_type__code=DIGITAL_ACTEUR_CODE)

qfdmo/views/adresses.py

@@ -217,7 +217,7 @@ def get_context_data(self, **kwargs):
         acteurs = self._acteurs_from_sous_categorie_objet_and_actions()
 
         if self.get_data_from_request_or_bounded_form("digital") == "1":
-            acteurs = acteurs.digital()[:100]
+            acteurs = acteurs.digital()
         else:
             bbox, acteurs = self._bbox_and_acteurs_from_location_or_epci(acteurs)
             acteurs = acteurs[: self._get_max_displayed_acteurs()]