From 4b63fe81e437601132e4b2b748222946792bf775 Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Wed, 5 Jan 2022 01:03:25 +0100
Subject: [PATCH] fix: het sort

---
 app/scodoc/sco_report.py | 18 +++++++-----------
 app/scodoc/sco_utils.py  |  5 +++++
 2 files changed, 12 insertions(+), 11 deletions(-)

diff --git a/app/scodoc/sco_report.py b/app/scodoc/sco_report.py
index d480679b7..6c2582ef1 100644
--- a/app/scodoc/sco_report.py
+++ b/app/scodoc/sco_report.py
@@ -49,16 +49,12 @@ from app.scodoc import sco_etud
 from app.scodoc import sco_excel
 from app.scodoc import sco_formsemestre
 from app.scodoc import sco_formsemestre_inscriptions
-from app.scodoc import sco_formsemestre_status
 from app.scodoc import sco_parcours_dut
-from app.scodoc import sco_pdf
 from app.scodoc import sco_preferences
 import sco_version
 from app.scodoc.gen_tables import GenTable
 from app import log
 from app.scodoc.sco_codes_parcours import code_semestre_validant
-from app.scodoc.sco_exceptions import ScoValueError
-from app.scodoc.sco_pdf import SU
 
 MAX_ETUD_IN_DESCR = 20
 
@@ -121,9 +117,9 @@ def _categories_and_results(etuds, category, result):
         categories[etud[category]] = True
         results[etud[result]] = True
     categories = list(categories.keys())
-    categories.sort()
+    categories.sort(key=scu.heterogeneous_sorting_key)
     results = list(results.keys())
-    results.sort()
+    results.sort(key=scu.heterogeneous_sorting_key)
     return categories, results
 
 
@@ -166,7 +162,7 @@ def _results_by_category(
         l["sumpercent"] = "%2.1f%%" % ((100.0 * l["sum"]) / tot)
     #
     codes = list(results.keys())
-    codes.sort()
+    codes.sort(key=scu.heterogeneous_sorting_key)
 
     bottom_titles = []
     if C:  # ligne du bas avec totaux:
@@ -314,7 +310,7 @@ def formsemestre_report_counts(
                 "type_admission",
                 "boursier_prec",
             ]
-        keys.sort()
+        keys.sort(key=scu.heterogeneous_sorting_key)
         F = [
             """<form name="f" method="get" action="%s"><p>
               Colonnes: <select name="result" onchange="document.f.submit()">"""
@@ -497,7 +493,7 @@ def table_suivi_cohorte(
             P.append(p)
 
     # 4-- regroupe par indice de semestre S_i
-    indices_sems = list(set([s["semestre_id"] for s in sems]))
+    indices_sems = list({s["semestre_id"] for s in sems})
     indices_sems.sort()
     for p in P:
         p.nb_etuds = 0  # nombre total d'etudiants dans la periode
@@ -788,9 +784,9 @@ def _gen_form_selectetuds(
 ):
     """HTML form pour choix criteres selection etudiants"""
     bacs = list(bacs)
-    bacs.sort()
+    bacs.sort(key=scu.heterogeneous_sorting_key)
     bacspecialites = list(bacspecialites)
-    bacspecialites.sort()
+    bacspecialites.sort(key=scu.heterogeneous_sorting_key)
     # on peut avoir un mix de chaines vides et d'entiers:
     annee_bacs = [int(x) if x else 0 for x in annee_bacs]
     annee_bacs.sort()
diff --git a/app/scodoc/sco_utils.py b/app/scodoc/sco_utils.py
index 4c4734910..360eee977 100644
--- a/app/scodoc/sco_utils.py
+++ b/app/scodoc/sco_utils.py
@@ -898,6 +898,11 @@ def sort_dates(L, reverse=False):
         raise
 
 
+def heterogeneous_sorting_key(x):
+    "key to sort non homogeneous sequences"
+    return (float(x), "") if isinstance(x, (bool, float, int)) else (-1e34, str(x))
+
+
 def query_portal(req, msg="Portail Apogee", timeout=3):
     """Retreives external data using HTTP request
     (used to connect to Apogee portal, or ScoDoc server)