Enable promo-based Piwigo groups

app/utils/auth/providers.py

@@ -173,10 +173,13 @@ def get_userinfo(self, user: models_users.CoreUser) -> dict[str, Any]:
 
         # For Piwigo, providing the username is sufficient. The name of the claim (here `"name"`) needs to be set in Piwigo oidc plugin configuration page.
         # A modified Piwigo oidc plugin allows managing groups from the oidc provider
+        promo = user.promo
         return {
             "sub": user.id,
             "name": user.full_name,
-            "groups": [group.name for group in user.groups] + [user.account_type.value],
+            "groups": [group.name for group in user.groups]
+            + [user.account_type.value]
+            + [str(promo) if promo is not None and promo >= 2014 else None],
             "email": user.email,
         }
 

migrations/versions/43_promo_above_2000.py

@@ -0,0 +1,101 @@
+"""43-promo-above-2000
+
+Create Date: 2025-09-07 09:54:34.421809
+"""
+
+import uuid
+from collections.abc import Sequence
+from typing import TYPE_CHECKING
+
+from app.core.schools.schools_type import SchoolType
+
+if TYPE_CHECKING:
+    from pytest_alembic import MigrationContext
+
+import sqlalchemy as sa
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision: str = "91fadc90f892"
+down_revision: str | None = "c4812e1ab108"
+branch_labels: str | Sequence[str] | None = None
+depends_on: str | Sequence[str] | None = None
+
+
+def upgrade() -> None:
+    op.execute("UPDATE core_user SET promo = 2000 + promo WHERE promo < 2000")
+
+
+def downgrade() -> None:
+    # we afterwards cannot distinguish those who had 2023 from 23
+    pass
+
+
+user_id_23 = str(uuid.uuid4())
+user_id_2023 = str(uuid.uuid4())
+user_id_null = str(uuid.uuid4())
+
+
+def pre_test_upgrade(
+    alembic_runner: "MigrationContext",
+    alembic_connection: sa.Connection,
+) -> None:
+    alembic_runner.insert_into(
+        "core_user",
+        [
+            {
+                "id": user_id_23,
+                "email": "email23",
+                "school_id": str(SchoolType.no_school.value),
+                "password_hash": "password_hash",
+                "account_type": "student",
+                "name": "name",
+                "firstname": "firstname",
+                "promo": 23,
+            },
+            {
+                "id": user_id_2023,
+                "email": "email2023",
+                "school_id": str(SchoolType.no_school.value),
+                "password_hash": "password_hash",
+                "account_type": "student",
+                "name": "name",
+                "firstname": "firstname",
+                "promo": 2023,
+            },
+            {
+                "id": user_id_null,
+                "email": "emailnull",
+                "school_id": str(SchoolType.no_school.value),
+                "password_hash": "password_hash",
+                "account_type": "student",
+                "name": "name",
+                "firstname": "firstname",
+                # promo is null
+            },
+        ],
+    )
+
+
+def test_upgrade(
+    alembic_runner: "MigrationContext",
+    alembic_connection: sa.Connection,
+) -> None:
+    assert (
+        alembic_connection.execute(
+            sa.text(f"SELECT promo FROM core_user WHERE id = '{user_id_23}'"),
+        ).all()[0][0]
+        == 2023
+    )
+    assert (
+        alembic_connection.execute(
+            sa.text(f"SELECT promo FROM core_user WHERE id = '{user_id_2023}'"),
+        ).all()[0][0]
+        == 2023
+    )
+    assert (
+        alembic_connection.execute(
+            sa.text(f"SELECT promo FROM core_user WHERE id = '{user_id_null}'"),
+        ).all()[0][0]
+        is None
+    )