From bd20abb745e161f7cda215d4650a6a709909acb3 Mon Sep 17 00:00:00 2001
From: yolou <seri-khane.yolou@etu.u-pec.fr>
Date: Fri, 20 Mar 2026 03:06:30 +0100
Subject: [PATCH] MAJ

---
 parcoursup/app.riot                    | 55 ++++++++++++------
 parcoursup/components/auth-panel.riot  | 80 ++++++++++++++++++--------
 parcoursup/components/detail-view.riot | 16 +++++-
 parcoursup/components/search-bar.riot  | 31 +++++++---
 parcoursup/firebase.js                 | 39 +++++++++----
 parcoursup/formation.js                | 18 +++---
 6 files changed, 167 insertions(+), 72 deletions(-)

diff --git a/parcoursup/app.riot b/parcoursup/app.riot
index c682447..34d8e93 100644
--- a/parcoursup/app.riot
+++ b/parcoursup/app.riot
@@ -167,32 +167,36 @@
 
       onMounted() {
         var saved = localStorage.getItem('selectionFormations')
+        var self  = this
 
+        // Charger la sélection locale si elle existe
         if (saved) {
           try {
             this.state.selectedFormations = JSON.parse(saved)
           } catch (e) {
-            console.error(e)
+            console.error('Erreur lecture localStorage :', e)
           }
         }
 
-        var self = this
-
-        // Écouter les changements d'authentification Firebase
+        // Écouter les changements de connexion Firebase
         window.firebaseServices.onUserChanged(function(user) {
+
           self.update({ user: user })
 
+          // Si un utilisateur vient de se connecter, charger sa sélection depuis Firestore
           if (user) {
-            // Charger la sélection depuis Firestore
-            window.firebaseServices.loadUserData(user.uid).then(function(data) {
-              if (data && data.selection) {
-                self.update({ selectedFormations: data.selection })
-                localStorage.setItem('selectionFormations', JSON.stringify(data.selection))
-              }
-            }).catch(function(err) {
-              console.error('Erreur chargement Firestore:', err)
-            })
+            window.firebaseServices.loadUserData(user.uid)
+              .then(function(data) {
+                if (data && data.selection) {
+                  self.update({ selectedFormations: data.selection })
+                  localStorage.setItem('selectionFormations', JSON.stringify(data.selection))
+                }
+              })
+              .catch(function(err) {
+                console.error('Erreur chargement Firestore :', err)
+              })
           }
+
         })
 
         window.addEventListener('hashchange', function() {
@@ -202,26 +206,31 @@
         this.handleRoute()
       },
 
+      // Appelée après une connexion ou inscription réussie
       onUserAuth() {
-        // Appelé après connexion/inscription réussie — l'écouteur onUserChanged gère le reste
+        // L'écouteur onUserChanged dans onMounted gère automatiquement le reste
       },
 
+      // Appelée après une déconnexion
       onUserLogout() {
-        // Vider la sélection locale à la déconnexion → obligé de se reconnecter pour retrouver ses formations
+        // On vide la sélection : l'utilisateur doit se reconnecter pour retrouver ses formations
         this.update({ selectedFormations: [] })
         localStorage.removeItem('selectionFormations')
       },
 
+      // Sauvegarder la sélection en local ET dans Firestore si connecté
       async saveSelection(selection) {
+
         localStorage.setItem('selectionFormations', JSON.stringify(selection))
 
         if (this.state.user) {
           try {
             await window.firebaseServices.saveUserData(this.state.user.uid, { selection: selection })
           } catch (err) {
-            console.error('Erreur sauvegarde Firestore:', err)
+            console.error('Erreur sauvegarde Firestore :', err)
           }
         }
+
       },
 
       handleRoute() {
@@ -397,9 +406,17 @@
         this.saveSelection([])
       },
 
-      updateNote(e) { this.update({ note: Number(e.target.value) }) },
-      updateSerie(e) { this.update({ serie: e.target.value }) },
-      updateSort(e) { this.update({ sortBy: e.target.value }) },
+      updateNote(e) {
+        this.update({ note: Number(e.target.value) })
+      },
+
+      updateSerie(e) {
+        this.update({ serie: e.target.value })
+      },
+
+      updateSort(e) {
+        this.update({ sortBy: e.target.value })
+      },
 
       getSortedSelection() {
         var selection = this.state.selectedFormations.slice()
diff --git a/parcoursup/components/auth-panel.riot b/parcoursup/components/auth-panel.riot
index d601e3f..5baf79d 100644
--- a/parcoursup/components/auth-panel.riot
+++ b/parcoursup/components/auth-panel.riot
@@ -1,38 +1,39 @@
 <auth-panel>
-  <!-- Bouton compact dans le header quand pas connecté -->
+
+  <!-- Bouton connexion dans le header (utilisateur non connecté) -->
   <div class="auth-header-btn" if={ !props.user }>
     <button class="btn btn-auth" onclick={ openModal }>
       <span class="auth-icon">👤</span> Connexion
     </button>
   </div>
 
-  <!-- Info utilisateur connecté dans le header -->
+  <!-- Email + bouton déconnexion dans le header (utilisateur connecté) -->
   <div class="auth-user-info" if={ props.user }>
     <span class="auth-email">{ props.user.email }</span>
     <button class="btn btn-auth-logout" onclick={ handleLogout }>Déconnexion</button>
   </div>
 
-  <!-- Modale -->
+  <!-- Modale d'authentification -->
   <div class="auth-modal-overlay" if={ state.open } onclick={ closeOnOverlay }>
     <div class="auth-modal">
+
       <button class="auth-modal-close" onclick={ closeModal }>✕</button>
 
-      <h2 class="auth-modal-title">{ state.mode === 'login' ? 'Connexion' : 'Créer un compte' }</h2>
+      <h2 class="auth-modal-title">{ state.titre }</h2>
 
+      <!-- Onglets Connexion / Inscription -->
       <div class="auth-tabs">
-        <button
-          class={ state.mode === 'login' ? 'auth-tab active' : 'auth-tab' }
-          onclick={ () => switchMode('login') }>
+        <button class={ state.classBtnLogin } onclick={ switchToLogin }>
           Connexion
         </button>
-        <button
-          class={ state.mode === 'register' ? 'auth-tab active' : 'auth-tab' }
-          onclick={ () => switchMode('register') }>
+        <button class={ state.classBtnRegister } onclick={ switchToRegister }>
           Inscription
         </button>
       </div>
 
+      <!-- Formulaire -->
       <form onsubmit={ handleSubmit } class="auth-form">
+
         <div class="auth-field">
           <label>Adresse e-mail</label>
           <input
@@ -42,6 +43,7 @@
             required
           />
         </div>
+
         <div class="auth-field">
           <label>Mot de passe</label>
           <input
@@ -53,24 +55,31 @@
           />
         </div>
 
+        <!-- Message d'erreur -->
         <div class="auth-error" if={ state.error }>
           { state.error }
         </div>
 
         <button type="submit" class="btn btn-primary auth-submit" disabled={ state.loading }>
-          { state.loading ? 'Chargement...' : (state.mode === 'login' ? 'Se connecter' : 'Créer le compte') }
+          { state.labelBouton }
         </button>
+
       </form>
     </div>
   </div>
 
   <script>
     export default {
+
       state: {
         open: false,
         mode: 'login',
         loading: false,
-        error: null
+        error: null,
+        titre: 'Connexion',
+        labelBouton: 'Se connecter',
+        classBtnLogin: 'auth-tab active',
+        classBtnRegister: 'auth-tab'
       },
 
       openModal() {
@@ -87,19 +96,38 @@
         }
       },
 
-      switchMode(mode) {
-        this.update({ mode: mode, error: null })
+      switchToLogin() {
+        this.update({
+          mode: 'login',
+          error: null,
+          titre: 'Connexion',
+          labelBouton: 'Se connecter',
+          classBtnLogin: 'auth-tab active',
+          classBtnRegister: 'auth-tab'
+        })
+      },
+
+      switchToRegister() {
+        this.update({
+          mode: 'register',
+          error: null,
+          titre: 'Créer un compte',
+          labelBouton: 'Créer le compte',
+          classBtnLogin: 'auth-tab',
+          classBtnRegister: 'auth-tab active'
+        })
       },
 
       async handleSubmit(e) {
         e.preventDefault()
-        var email = e.target.email.value.trim()
+
+        var email    = e.target.email.value.trim()
         var password = e.target.password.value
+        var services = window.firebaseServices
 
         this.update({ loading: true, error: null })
 
         try {
-          var services = window.firebaseServices
 
           if (this.state.mode === 'register') {
             await services.createAccount(email, password)
@@ -109,22 +137,24 @@
 
           this.update({ open: false, loading: false })
           this.props.onauth && this.props.onauth()
+
         } catch (err) {
-          var msg = 'Une erreur est survenue.'
+
+          var message = 'Une erreur est survenue.'
 
           if (err.code === 'auth/email-already-in-use') {
-            msg = 'Cet e-mail est déjà utilisé.'
+            message = 'Cet e-mail est déjà utilisé.'
           } else if (err.code === 'auth/invalid-email') {
-            msg = 'Adresse e-mail invalide.'
+            message = 'Adresse e-mail invalide.'
           } else if (err.code === 'auth/wrong-password' || err.code === 'auth/invalid-credential') {
-            msg = 'E-mail ou mot de passe incorrect.'
+            message = 'E-mail ou mot de passe incorrect.'
           } else if (err.code === 'auth/weak-password') {
-            msg = 'Le mot de passe doit faire au moins 6 caractères.'
+            message = 'Le mot de passe doit faire au moins 6 caractères.'
           } else if (err.code === 'auth/user-not-found') {
-            msg = 'Aucun compte trouvé avec cet e-mail.'
+            message = 'Aucun compte trouvé avec cet e-mail.'
           }
 
-          this.update({ loading: false, error: msg })
+          this.update({ loading: false, error: message })
         }
       },
 
@@ -133,9 +163,11 @@
           await window.firebaseServices.logout()
           this.props.onlogout && this.props.onlogout()
         } catch (err) {
-          console.error('Erreur déconnexion:', err)
+          console.error('Erreur déconnexion :', err)
         }
       }
+
     }
   </script>
+
 </auth-panel>
diff --git a/parcoursup/components/detail-view.riot b/parcoursup/components/detail-view.riot
index 55c5dc0..57a513a 100644
--- a/parcoursup/components/detail-view.riot
+++ b/parcoursup/components/detail-view.riot
@@ -191,7 +191,7 @@
       </div>
     </div>
 
-    <button onclick={ () => props.onback() } class="btn-retour">Retour à la liste</button>
+    <button onclick={ goBack } class="btn-retour">Retour à la liste</button>
   </div>
 
   <script>
@@ -221,6 +221,11 @@
         }
       },
 
+      // Retour à la liste des résultats
+      goBack() {
+        this.props.onback()
+      },
+
       renderCharts() {
         var f = this.props.formation
         if (!f) return
@@ -307,8 +312,13 @@
           var rows = ''
           for (var i = 0; i < hist.length; i++) {
             var h = hist[i]
-            var sizeCand = maxCandidats > 0 ? Math.round((h.candidats / maxCandidats) * 100) / 100 : 0
-            var sizeAdmis = maxCandidats > 0 ? Math.round((h.admis / maxCandidats) * 100) / 100 : 0
+            var sizeCand  = 0
+            var sizeAdmis = 0
+
+            if (maxCandidats > 0) {
+              sizeCand  = Math.round((h.candidats / maxCandidats) * 100) / 100
+              sizeAdmis = Math.round((h.admis / maxCandidats) * 100) / 100
+            }
 
             rows += '<tr>'
             rows += '<th scope="row">' + h.annee + '</th>'
diff --git a/parcoursup/components/search-bar.riot b/parcoursup/components/search-bar.riot
index 1511402..81c2937 100644
--- a/parcoursup/components/search-bar.riot
+++ b/parcoursup/components/search-bar.riot
@@ -12,7 +12,7 @@
 
   <div class="filters-toggle">
     <button class="btn btn-small btn-outline" onclick={ toggleFilters }>
-      { state.showFilters ? 'Masquer les filtres' : 'Filtres avancés' }
+      { state.labelFiltres }
     </button>
   </div>
 
@@ -91,6 +91,7 @@
       state: {
         query: '',
         showFilters: false,
+        labelFiltres: 'Filtres avancés',
         filiere: '',
         selectivite: '',
         region: '',
@@ -109,14 +110,30 @@
       },
 
       toggleFilters() {
-        this.update({ showFilters: !this.state.showFilters })
+        var visible = !this.state.showFilters
+        var label   = visible ? 'Masquer les filtres' : 'Filtres avancés'
+        this.update({ showFilters: visible, labelFiltres: label })
       },
 
-      updateFiliere(e) { this.update({ filiere: e.target.value }) },
-      updateSelectivite(e) { this.update({ selectivite: e.target.value }) },
-      updateRegion(e) { this.update({ region: e.target.value }) },
-      updateTauxMin(e) { this.update({ tauxMin: Number(e.target.value) }) },
-      updateTauxMax(e) { this.update({ tauxMax: Number(e.target.value) }) },
+      updateFiliere(e) {
+        this.update({ filiere: e.target.value })
+      },
+
+      updateSelectivite(e) {
+        this.update({ selectivite: e.target.value })
+      },
+
+      updateRegion(e) {
+        this.update({ region: e.target.value })
+      },
+
+      updateTauxMin(e) {
+        this.update({ tauxMin: Number(e.target.value) })
+      },
+
+      updateTauxMax(e) {
+        this.update({ tauxMax: Number(e.target.value) })
+      },
 
       submitSearch() {
         var filters = {
diff --git a/parcoursup/firebase.js b/parcoursup/firebase.js
index e8c1330..069dab1 100644
--- a/parcoursup/firebase.js
+++ b/parcoursup/firebase.js
@@ -1,4 +1,5 @@
 import { initializeApp } from "https://www.gstatic.com/firebasejs/11.6.1/firebase-app.js";
+
 import {
   getAuth,
   createUserWithEmailAndPassword,
@@ -6,6 +7,7 @@ import {
   signOut,
   onAuthStateChanged
 } from "https://www.gstatic.com/firebasejs/11.6.1/firebase-auth.js";
+
 import {
   getFirestore,
   doc,
@@ -13,44 +15,59 @@ import {
   getDoc
 } from "https://www.gstatic.com/firebasejs/11.6.1/firebase-firestore.js";
 
+
 const firebaseConfig = {
-    apiKey: "AIzaSyDr1jMgGm0Oj_bOiWY-8Gy27IlzkmAzlOM",
-    authDomain: "parcoursupp-expl.firebaseapp.com",
-    projectId: "parcoursupp-expl",
-    storageBucket: "parcoursupp-expl.firebasestorage.app",
-    messagingSenderId: "973054617217",
-    appId: "1:973054617217:web:4d52af4280396976228f80"
-  };
+  apiKey: "AIzaSyDr1jMgGm0Oj_bOiWY-8Gy27IlzkmAzlOM",
+  authDomain: "parcoursupp-expl.firebaseapp.com",
+  projectId: "parcoursupp-expl",
+  storageBucket: "parcoursupp-expl.firebasestorage.app",
+  messagingSenderId: "973054617217",
+  appId: "1:973054617217:web:4d52af4280396976228f80"
+};
 
-const app = initializeApp(firebaseConfig);
+
+const app  = initializeApp(firebaseConfig);
 const auth = getAuth(app);
-const db = getFirestore(app);
+const db   = getFirestore(app);
 
+
+// Créer un compte avec email et mot de passe
 async function createAccount(email, password) {
   return createUserWithEmailAndPassword(auth, email, password);
 }
 
+// Se connecter avec email et mot de passe
 async function login(email, password) {
   return signInWithEmailAndPassword(auth, email, password);
 }
 
+// Se déconnecter
 async function logout() {
   return signOut(auth);
 }
 
+// Écouter les changements d'état de connexion
 function onUserChanged(callback) {
   return onAuthStateChanged(auth, callback);
 }
 
+// Sauvegarder les données d'un utilisateur dans Firestore
 async function saveUserData(uid, data) {
   await setDoc(doc(db, "users", uid), data, { merge: true });
 }
 
+// Charger les données d'un utilisateur depuis Firestore
 async function loadUserData(uid) {
-  const snap = await getDoc(doc(db, "users", uid));
-  return snap.exists() ? snap.data() : null;
+  var snap = await getDoc(doc(db, "users", uid));
+
+  if (snap.exists()) {
+    return snap.data();
+  } else {
+    return null;
+  }
 }
 
+
 export {
   auth,
   db,
diff --git a/parcoursup/formation.js b/parcoursup/formation.js
index 0bfa12b..517af8d 100644
--- a/parcoursup/formation.js
+++ b/parcoursup/formation.js
@@ -1,10 +1,17 @@
 export function createFormation(raw) {
-  let taux = 0
+  var taux      = 0
+  var latitude  = null
+  var longitude = null
 
   if (raw.voe_tot && raw.voe_tot > 0) {
     taux = Math.round((raw.acc_tot / raw.voe_tot) * 100)
   }
 
+  if (raw.g_olocalisation_des_formations) {
+    latitude  = raw.g_olocalisation_des_formations.lat
+    longitude = raw.g_olocalisation_des_formations.lon
+  }
+
   return {
     id: raw.cod_uai + "-" + raw.lib_for_voe_ins,
 
@@ -25,13 +32,8 @@ export function createFormation(raw) {
     admis: raw.acc_tot,
     tauxAcces: taux,
 
-    latitude: raw.g_olocalisation_des_formations
-      ? raw.g_olocalisation_des_formations.lat
-      : null,
-
-    longitude: raw.g_olocalisation_des_formations
-      ? raw.g_olocalisation_des_formations.lon
-      : null,
+    latitude: latitude,
+    longitude: longitude,
 
     pctFemmes: raw.pct_f,
     pctBoursiers: raw.pct_bours,