From 8a83e416986671eef0580441aceb90d94acc81ff Mon Sep 17 00:00:00 2001
From: Emmanuel Viennet <emmanuel.viennet@gmail.com>
Date: Fri, 24 Jun 2022 12:39:54 +0200
Subject: [PATCH] =?UTF-8?q?BUT:=20pond=C3=A9ration=20des=20UE=20dans=20les?=
 =?UTF-8?q?=20RCUEs?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/models/but_validations.py                 |  6 ++-
 app/models/ues.py                             |  3 ++
 app/scodoc/sco_edit_ue.py                     | 43 ++++++++++++++-----
 migrations/versions/c0c225192d61_coef_rcue.py | 28 ++++++++++++
 4 files changed, 68 insertions(+), 12 deletions(-)
 create mode 100644 migrations/versions/c0c225192d61_coef_rcue.py

diff --git a/app/models/but_validations.py b/app/models/but_validations.py
index 27aee4299c..651e6fc2d4 100644
--- a/app/models/but_validations.py
+++ b/app/models/but_validations.py
@@ -120,8 +120,10 @@ class RegroupementCoherentUE:
             self.moy_ue_2_val = 0.0
         # Calcul de la moyenne au RCUE
         if (self.moy_ue_1 is not None) and (self.moy_ue_2 is not None):
-            # Moyenne RCUE non pondérée (pour le moment -- TODO)
-            self.moy_rcue = (self.moy_ue_1 + self.moy_ue_2) / 2
+            # Moyenne RCUE (les pondérations par défaut sont 1.)
+            self.moy_rcue = (
+                self.moy_ue_1 * ue_1.coef_rcue + self.moy_ue_2 * ue_2.coef_rcue
+            ) / (ue_1.coef_rcue + ue_2.coef_rcue)
         else:
             self.moy_rcue = None
 
diff --git a/app/models/ues.py b/app/models/ues.py
index aa87e7ee2e..52cd3788ce 100644
--- a/app/models/ues.py
+++ b/app/models/ues.py
@@ -40,6 +40,9 @@ class UniteEns(db.Model):
     # coef UE, utilise seulement si l'option use_ue_coefs est activée:
     coefficient = db.Column(db.Float)
 
+    # coef. pour le calcul de moyennes de RCUE. Par défaut, 1.
+    coef_rcue = db.Column(db.Float, nullable=False, default=1.0, server_default="1.0")
+
     color = db.Column(db.Text())
 
     # BUT
diff --git a/app/scodoc/sco_edit_ue.py b/app/scodoc/sco_edit_ue.py
index fd40ac2b85..e8d8ba02bf 100644
--- a/app/scodoc/sco_edit_ue.py
+++ b/app/scodoc/sco_edit_ue.py
@@ -78,6 +78,7 @@ _ueEditor = ndb.EditableTable(
         "is_external",
         "code_apogee",
         "coefficient",
+        "coef_rcue",
         "color",
     ),
     sortkey="numero",
@@ -252,6 +253,7 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
         initvalues = {
             "semestre_idx": default_semestre_idx,
             "color": ue_guess_color_default(formation_id, default_semestre_idx),
+            "coef_rcue": 1.0,
         }
         submitlabel = "Créer cette UE"
         can_change_semestre_id = True
@@ -341,22 +343,43 @@ def ue_edit(ue_id=None, create=False, formation_id=None, default_semestre_idx=No
                 "allow_null": not is_apc,  # ects requis en APC
             },
         ),
-        (
-            "coefficient",
-            {
-                "size": 4,
-                "type": "float",
-                "title": "Coefficient",
-                "explanation": """les coefficients d'UE ne sont utilisés que
+    ]
+    if is_apc:  # coef pour la moyenne RCUE
+        form_descr.append(
+            (
+                "coef_rcue",
+                {
+                    "size": 4,
+                    "type": "float",
+                    "title": "Coef. RCUE",
+                    "explanation": """pondération utilisée pour le calcul de la moyenne du RCUE. Laisser à 1, sauf si votre établissement a explicitement décidé de pondérations.
+                    """,
+                    "defaut": 1.0,
+                    "allow_null": False,
+                    "enabled": is_apc,
+                },
+            )
+        )
+    else:  # non APC, coef d'UE
+        form_descr.append(
+            (
+                "coefficient",
+                {
+                    "size": 4,
+                    "type": "float",
+                    "title": "Coefficient",
+                    "explanation": """les coefficients d'UE ne sont utilisés que
                     lorsque l'option <em>Utiliser les coefficients d'UE pour calculer
                     la moyenne générale</em> est activée. Par défaut, le coefficient
                     d'une UE est simplement la somme des coefficients des modules dans
                     lesquels l'étudiant a des notes.
                     Jamais utilisé en BUT.
                     """,
-                "enabled": not is_apc,
-            },
-        ),
+                    "enabled": not is_apc,
+                },
+            )
+        )
+    form_descr += [
         (
             "ue_code",
             {
diff --git a/migrations/versions/c0c225192d61_coef_rcue.py b/migrations/versions/c0c225192d61_coef_rcue.py
new file mode 100644
index 0000000000..5a3166e842
--- /dev/null
+++ b/migrations/versions/c0c225192d61_coef_rcue.py
@@ -0,0 +1,28 @@
+"""coef_rcue
+
+Revision ID: c0c225192d61
+Revises: 4311cc342dbd
+Create Date: 2022-06-24 12:19:58.723862
+
+"""
+from alembic import op
+import sqlalchemy as sa
+
+
+# revision identifiers, used by Alembic.
+revision = 'c0c225192d61'
+down_revision = '4311cc342dbd'
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.add_column('notes_ue', sa.Column('coef_rcue', sa.Float(), server_default='1.0', nullable=False))
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_column('notes_ue', 'coef_rcue')
+    # ### end Alembic commands ###