Améliore import/export formations BUT (ects par parcours)

app/models/ues.py

@@ -110,17 +110,19 @@ class UniteEns(db.Model):
         # ScoDoc7 output_formators
         e["ue_id"] = self.id
         e["numero"] = e["numero"] if e["numero"] else 0
-        e["ects"] = e["ects"]  # legacy
-        e["ects_by_parcours"] = {}
-        for up in UEParcours.query.filter_by(ue_id=self.id):
-            p = ApcParcours.query.get(up.parcours_id)
-            e["ects_by_parcours"][p.code] = self.get_ects(p)
-
         e["coefficient"] = e["coefficient"] if e["coefficient"] else 0.0
         e["code_apogee"] = e["code_apogee"] or ""  # pas de None
-        e["parcours"] = [
+        e["ects_by_parcours"] = {
-            parcour.to_dict(with_annees=False) for parcour in self.parcours
+            parcour.code: self.get_ects(parcour) for parcour in self.parcours
-        ]
+        }
+        e["parcours"] = []
+        for parcour in self.parcours:
+            p_dict = parcour.to_dict(with_annees=False)
+            ects = self.get_ects(parcour, only_parcours=True)
+            if ects is not None:
+                p_dict["ects"] = ects
+            e["parcours"].append(p_dict)
+
         if with_module_ue_coefs:
             if convert_objects:
                 e["module_ue_coefs"] = [

app/scodoc/sco_formations.py

@@ -127,11 +127,7 @@ def formation_export_dict(
                 ue_dict["apc_niveau_libelle"] = ue.niveau_competence.libelle
                 ue_dict["apc_niveau_annee"] = ue.niveau_competence.annee
                 ue_dict["apc_niveau_ordre"] = ue.niveau_competence.ordre
-            # Et les parcours:
+
-            if ue.parcours:
-                ue_dict["parcours"] = [
-                    parcour.to_dict(with_annees=False) for parcour in ue.parcours
-                ]
         # pour les coefficients:
         ue_dict["reference"] = ue.id if ue_reference_style == "id" else ue.acronyme
         if not export_ids:
@@ -367,14 +363,13 @@ def formation_import_xml(doc: str, import_tags=True, use_local_refcomp=False):
             assert ue
             if xml_ue_id:
                 ues_old2new[xml_ue_id] = ue_id
-            # parcours BUT
+
-            # TODO XXX
             # élément optionnel présent dans les exports BUT:
             ue_reference = ue_info[1].get("reference")
             if ue_reference:
                 ue_reference_to_id[int(ue_reference)] = ue_id
 
-            # -- create matieres
+            # -- Create matieres
             for mat_info in ue_info[2]:
                 # Backward compat: un seul parcours par UE (ScoDoc < 9.4.71)
                 if mat_info[0] == "parcour":
@@ -392,18 +387,20 @@ def formation_import_xml(doc: str, import_tags=True, use_local_refcomp=False):
                         log(f"Warning: parcours {code_parcours} inexistant !")
                     continue
                 elif mat_info[0] == "parcours":
-                    # Parcours (BUT), liste (ScoDoc > 9.4.70)
+                    # Parcours (BUT), liste (ScoDoc > 9.4.70), avec ECTS en option
-                    codes_parcours = mat_info[1]["code"]
+                    code_parcours = mat_info[1]["code"]
-                    for code_parcours in codes_parcours:
+                    ue_parcour_ects = mat_info[1].get("ects")
-                        parcour = ApcParcours.query.filter_by(
+                    parcour = ApcParcours.query.filter_by(
-                            code=code_parcours,
+                        code=code_parcours,
-                            referentiel_id=referentiel_competence_id,
+                        referentiel_id=referentiel_competence_id,
-                        ).first()
+                    ).first()
-                        if parcour:
+                    if parcour:
-                            ue.parcours.append(parcour)
+                        ue.parcours.append(parcour)
-                        else:
+                    else:
-                            flash(f"Attention: parcours {code_parcours} inexistant !")
+                        flash(f"Attention: parcours {code_parcours} inexistant !")
-                            log(f"Warning: parcours {code_parcours} inexistant !")
+                        log(f"Warning: parcours {code_parcours} inexistant !")
+                    if ue_parcour_ects is not None:
+                        ue.set_ects(ue_parcour_ects, parcour)
                     db.session.add(ue)
                     continue