From b8e717cbf3a0d11cea725ceb1b1d6317e0486cfe Mon Sep 17 00:00:00 2001
From: Lucie Bader <167515375+Lucie-Bdr@users.noreply.github.com>
Date: Tue, 18 Feb 2025 13:31:12 +0100
Subject: [PATCH] Pas d'identifiant des lexiques en mode prod

---
 src/context_menu/custom_context_menu.js | 30 ++++++++++++++++++-------
 src/sidebar/sidebar.js                  | 20 ++++++++++++-----
 2 files changed, 36 insertions(+), 14 deletions(-)

diff --git a/src/context_menu/custom_context_menu.js b/src/context_menu/custom_context_menu.js
index 271e490..a4cb929 100644
--- a/src/context_menu/custom_context_menu.js
+++ b/src/context_menu/custom_context_menu.js
@@ -243,15 +243,27 @@ async function showPicker(event, selectedText) {
     const lexicons = await getLexicons(authToken);
     log("Lexicons récupérés :", lexicons);
 
+    const lexiconDescriptions = {};
+
     if (!Array.isArray(lexicons) || lexicons.length === 0) {
       picker.innerHTML = "<p style='color:#333;'>Aucun lexique trouvé.</p>";
     } else {
       for (const lex of lexicons) {
         const id = lex.id;
-        const name = lex.category === "User"
-          ? `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})`
-          : `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})`;
-          
+        let name = "";
+        if (lex.category === "User") {
+          name = DEBUG
+            ? `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})`
+            : `Lexique personnel : ${lex.user?.pseudo || "Inconnu"}`;
+        } else {
+          name = DEBUG
+            ? `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})`
+            : `Lexique de groupe : ${lex.group?.name || "Inconnu"}`;
+        }
+        if (lex.language) {
+          name += ` [${lex.language}]`;
+        }
+        lexiconDescriptions[id] = name;
         const color = await getColorForLexicon(id);
         const circleIcon = await createColorCircle(color, 28);
         
@@ -309,19 +321,21 @@ async function showPicker(event, selectedText) {
           log("Réponse API :", result);
           await new Promise(resolve => setTimeout(resolve, 300));
           browser.runtime.sendMessage({ action: "refreshUI" });
-          picker.innerHTML = `<p style="color: green;">✅ Mot ajouté avec succès dans : ${lexiconsToAdd.join(", ")}.</p>`;
+          const successMsg = `✅ Mot ajouté avec succès dans : ${lexiconsToAdd.map(id => lexiconDescriptions[id]).join(", ")}`;
+          picker.innerHTML = `<p style="color: green;">${successMsg}</p>`;
           setTimeout(() => picker.style.display = "none", 2000);
           browser.runtime.sendMessage({
             action: "addWordResult",
-            lexicons: `✅ Mot ajouté avec succès dans : ${lexiconsToAdd.join(", ")}.`
+            lexicons: successMsg
           });
         } catch (error) {
           console.error("❌ Erreur lors de l'ajout du mot :", error);
-          picker.innerHTML = `<p style="color: red;">❌ Erreur : ${error.message}</p>`;
+          const errorMsg = `❌ Erreur lors de l'ajout du mot : ${error.message}`;
+          picker.innerHTML = `<p style="color: red;">${errorMsg}</p>`;
           setTimeout(() => picker.style.display = "none", 3000);
           browser.runtime.sendMessage({
             action: "addWordResult",
-            lexicons: `❌ Erreur lors de l'ajout du mot : ${error.message}`
+            lexicons: errorMsg
           });
         }
       });
diff --git a/src/sidebar/sidebar.js b/src/sidebar/sidebar.js
index 652a75b..1233404 100644
--- a/src/sidebar/sidebar.js
+++ b/src/sidebar/sidebar.js
@@ -74,7 +74,7 @@ function toggleHighlightMessage(isLoggedIn) {
 }
 
 /**
- * Met à jour l'état global de la sidebar (bouton d'authentification, etc.)
+ * Met à jour l'état global de la barre latérale (bouton d'authentification, etc.)
  */
 async function refreshSidebarState() {
   log("🔄 Début de l'actualisation de la barre latérale...");
@@ -206,14 +206,18 @@ async function fetchLexicons() {
     lexicons.forEach((lex) => {
       let lexiconName = "";
       if (lex.category === "User") {
-        lexiconName = `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})`;
+        lexiconName = DEBUG 
+          ? `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})`
+          : `Lexique personnel : ${lex.user?.pseudo || "Inconnu"}`;
         if (lex.language) {
           lexiconName += ` [${lex.language}]`;
         }
       } else if (lex.category === "Group") {
-        lexiconName = `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})`;
+        lexiconName = DEBUG 
+          ? `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})`
+          : `Lexique de groupe : ${lex.group?.name || "Inconnu"}`
       } else {
-        lexiconName = `Lexique : ${lex.id}`;
+        lexiconName = DEBUG ? `Lexique : ${lex.id}` : "Lexique" ;
       }
       lexiconMap.set(lex.id, lexiconName);
     });
@@ -221,8 +225,12 @@ async function fetchLexicons() {
     displayLexiconsWithCheckbox(lexicons.map((lex) => ({
       lexiconName:
         lex.category === "User"
-          ? `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})${(lex.language) ? ` [${lex.language}]` : ""}`
-          : `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})${(lex.language) ? ` [${lex.language}]` : ""}`,
+          ? DEBUG
+            ? `Lexique personnel : ${lex.user?.pseudo || "Inconnu"} (${lex.id})${(lex.language) ? ` [${lex.language}]` : ""}`
+            : `Lexique personnel : ${lex.user?.pseudo || "Inconnu"}${(lex.language) ? ` [${lex.language}]` : ""}`
+          : DEBUG
+            ? `Lexique de groupe : ${lex.group?.name || "Inconnu"} (${lex.id})${(lex.language) ? ` [${lex.language}]` : ""}`
+            : `Lexique de groupe : ${lex.group?.name || "Inconnu"}${(lex.language) ? ` [${lex.language}]` : ""}`,
       lexiconId: lex.id,
       active: lex.active || false,
     })));
-- 
GitLab