From dcd1f6b01894849ddbf47cc638e1bb0d121613c7 Mon Sep 17 00:00:00 2001
From: kmitresse <kmitresse@etud.univ-pau.fr>
Date: Wed, 1 May 2024 15:45:41 +0200
Subject: [PATCH] refacto: devWeb - unify language used for comments (front)

---
 .../src/main/webapp/WEB-INF/pages/game.jsp       |  1 -
 .../WEB-INF/tags/components/game-statistics.tag  |  3 ---
 .../webapp/WEB-INF/tags/components/navbar.tag    | 13 +++++--------
 .../WEB-INF/tags/forms/forgotten-password.tag    | 16 ++++++++--------
 .../src/main/webapp/WEB-INF/tags/forms/login.tag |  8 +++-----
 .../main/webapp/WEB-INF/tags/forms/new-game.tag  |  7 ++-----
 .../main/webapp/WEB-INF/tags/forms/profile.tag   |  8 +-------
 .../main/webapp/WEB-INF/tags/forms/register.tag  | 12 ++----------
 .../webapp/WEB-INF/tags/forms/reset-password.tag | 12 ------------
 S2/DevWeb/Projet/src/main/webapp/index.html      | 10 +++++-----
 10 files changed, 26 insertions(+), 64 deletions(-)

diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/pages/game.jsp b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/pages/game.jsp
index 00ca068..4f8a659 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/pages/game.jsp
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/pages/game.jsp
@@ -1,4 +1,3 @@
-<%@ page import="uppa.project.database.pojo.Card" %>
 <%@ page contentType="text/html;charset=UTF-8" language="java" %>
 <%@taglib prefix="layout" tagdir="/WEB-INF/tags/layouts" %>
 <%@taglib prefix="component" tagdir="/WEB-INF/tags/components" %>
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/game-statistics.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/game-statistics.tag
index 52e6cb4..40bba80 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/game-statistics.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/game-statistics.tag
@@ -1,18 +1,15 @@
 <%@ tag import="uppa.project.database.pojo.Player" %>
 <%@ tag import="uppa.project.database.pojo.Game" %>
-<%@ tag import="java.util.List" %>
 <%@ tag import="java.util.ArrayList" %>
 <%@ tag import="java.text.SimpleDateFormat" %>
 <%@ tag import="java.util.Date" %>
 <%@tag description="component/statistics" pageEncoding="UTF-8" %>
 
 <%
-    // Get game from request
     Game game = (Game) request.getAttribute("game");
     ArrayList<Player> players = (ArrayList<Player>) request.getAttribute("players");
 %>
 
-<%--<jsp:useBean id="game" class="uppa.project.database.pojo.Game" scope="request"/>--%>
 <h4 class="title is-6">Information sur la partie</h4>
 <div class="level">
     <div class="level-item has-text-centered has-text-on-top">
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/navbar.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/navbar.tag
index d140130..083e618 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/navbar.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/components/navbar.tag
@@ -17,8 +17,6 @@
 
         <% } %>
 
-
-
         <a role="button" class="navbar-burger" aria-label="menu" aria-expanded="false" data-target="navbarMenu">
             <span aria-hidden="true"></span>
             <span aria-hidden="true"></span>
@@ -68,21 +66,20 @@
 </nav>
 
 <script type="module" defer>
-    // Get all "navbar-burger" elements
+    // Récupération de tous les éléments de classe "navbar-burger"
     const $navbarBurgers = Array.prototype.slice.call(document.querySelectorAll('.navbar-burger'), 0);
 
-    // Add a click event on each of them
+    // Ajout d'un enventListener sur chaque élément
     $navbarBurgers.forEach(el => {
         el.addEventListener('click', () => {
 
-            // Get the target from the "data-target" attribute
+            // Récupere la valeur de l'attribut "data-target"
             const target = el.dataset.target;
             const $target = document.getElementById(target);
 
-            // Toggle the "is-active" class on both the "navbar-burger" and the "navbar-menu"
+            // Ajoute ou supprime la classe "is-active" sur les éléments
             el.classList.toggle('is-active');
             $target.classList.toggle('is-active');
-
         });
     });
-</script>
\ No newline at end of file
+</script>
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/forgotten-password.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/forgotten-password.tag
index bbbdb02..3fb7e65 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/forgotten-password.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/forgotten-password.tag
@@ -1,7 +1,7 @@
 <%@tag description="form/forgotten-password" pageEncoding="UTF-8" %>
 
 <form id="forgotten-password-form" action="${pageContext.request.contextPath}/forgotten-password" method="post">
-    <div class="field"> <!-- Field Username -->
+    <div class="field">
         <label class="label" for="email">Email</label>
         <div class="control has-icons-left">
             <input id="email" placeholder="johndoe@exemple.com" class="input is-fullwidth" required/>
@@ -17,15 +17,15 @@
 <script defer type="module">
   const forgottenPasswordForm = document.querySelector("form#forgotten-password-form");
 
-  // Form fields
+  // Champ email
   const emailInput = document.querySelector("input#email");
 
-  // Add event listener to the form submission
+  // Ajout de l'écouteur d'événement sur la soumission du formulaire
   forgottenPasswordForm.addEventListener("submit", onSubmit)
 
   /**
-   * Handle the form submission with Ajax request
-   * @param event {Event} - Event of the form submission
+   * Gestion de la soumission du formulaire
+   * @param event {Event} - Événement de soumission du formulaire
    */
   function onSubmit(event) {
     event.preventDefault();
@@ -45,8 +45,8 @@
   }
 
   /**
-   * Handle the error of the form submission
-   * @param error {Error} - Error of the form submission
+   * Gestion des erreurs lors de la soumission du formulaire
+   * @param error {Error} - Erreur survenue lors de la soumission du formulaire
    */
   function onError(error) {
     console.error("Error:", error)
@@ -81,7 +81,7 @@
     setTimeout(() => notification.remove(), 5010);
   }
 
-  // Remove the shake animation at the end of animation
+  // Retrait des animations sur le champ email
   emailInput.addEventListener("animationend", () => emailInput.style.animation = "");
 </script>
 
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/login.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/login.tag
index 27b4367..1012822 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/login.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/login.tag
@@ -2,7 +2,6 @@
 
 <form id="login-form" action="${pageContext.request.contextPath}/login" method="post">
 
-    <!-- Field Username -->
     <div class="field">
         <label class="label" for="username">Nom d'utilisateur</label>
         <div class="control has-icons-left">
@@ -13,7 +12,6 @@
         </div>
     </div>
 
-    <!-- Field Password -->
     <div class="field">
         <label class="label" for="password">Mot de passe</label>
         <div class="control has-icons-left">
@@ -52,7 +50,7 @@
             .catch((error) => {
                 console.log(error)
 
-                // Input animation
+                // Animations des champs
                 inputs.forEach(input => {
                     input.classList.add("is-danger");
                     input.style.animation = "shake 0.5s ease-in-out"
@@ -79,12 +77,12 @@
                 notification.appendChild(notificationMessage);
                 document.body.appendChild(notification);
 
-                // Remove notification and animation after 5s
+                // Retirer la notification et les animations après 5 secondes
                 setTimeout(() => notification.remove(), 5010);
             });
     });
 
-    // On animation end, remove class and animation
+    // Retirer les animations des champs après la fin de l'animation
     inputs.forEach(input => input.addEventListener("animationend", () => input.style.animation = ""));
 </script>
 
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/new-game.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/new-game.tag
index 0cd0296..36eb0a2 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/new-game.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/new-game.tag
@@ -91,7 +91,6 @@
     const nbRound = document.querySelector("input[name='nbRounds']");
     const nbValues = document.querySelector("input[name='nbValues']");
     const nbColors = document.querySelector("input[name='nbColors']");
-    // Range inputs
     const rangeInputs = document.querySelectorAll("input[type='range']");
     rangeInputs.forEach(input => {
         const tooltip = document.querySelector(input.dataset.tooltip);
@@ -104,7 +103,6 @@
         });
     });
 
-    // Radio buttons
     const radioButtons = document.querySelectorAll('input[type="radio"]');
     radioButtons.forEach(radio => {
         radio.addEventListener('change', () => {
@@ -113,7 +111,6 @@
         });
     });
 
-    // Form submission
     const form = document.getElementById('new-game-form');
 
     form.addEventListener('submit', evt => {
@@ -132,7 +129,7 @@
             .then(data => {
                 if (data.code !== 200) throw new Error(data.message);
 
-                // Redirect to game page
+                // Redirection vers la page de jeu
                 window.location.href = "${pageContext.request.contextPath}/game?id=" + data.message;
             })
             .catch((error) => {
@@ -159,7 +156,7 @@
                 notification.appendChild(notificationMessage);
                 document.body.appendChild(notification);
 
-                // Remove notification and animation after 5s
+                // Retrait de la notification et des animations après 5 secondes
                 setTimeout(() => notification.remove(), 5010);
             });
     });
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/profile.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/profile.tag
index 7fce5ef..cf63c6b 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/profile.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/profile.tag
@@ -88,6 +88,7 @@
     const passwordFields = profileForm.querySelectorAll("div#old-password-field, div#password-field, div#repeat-password-field");
     const inputs = profileForm.querySelectorAll("input[type='text'], input[type='password']");
 
+    // Afficher les champs de mot de passe si le lien est cliqué
     changePassword.addEventListener("click", (e) => {
         e.preventDefault();
         passwordFields.forEach(field => {
@@ -130,10 +131,6 @@
         .catch(onError)
     }
 
-    /**
-     * Handle the error of the form submission
-     * @param error {Error} - Error of the form submission
-     */
     function onError(error) {
       console.log(error)
 
@@ -167,9 +164,6 @@
       }, 5010);
     }
 
-    /**
-     * Handle the success of the form submission
-     */
     function onSuccess() {
       console.log("Succès:", "Modifications effectuées avec succès")
 
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/register.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/register.tag
index 31085b5..90eb374 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/register.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/register.tag
@@ -59,16 +59,11 @@
 <script defer type="module">
     const registerForm = document.querySelector("form#register-form");
 
-    // Form fields
     const inputs = registerForm.querySelectorAll("input, select");
 
-    // Add event listener to the form submission
     registerForm.addEventListener("submit", onSubmit)
 
-    /**
-     * Handle the form submission with Ajax request
-     * @param event {Event} - Event of the form submission
-     */
+
     function onSubmit(event) {
         event.preventDefault();
 
@@ -95,10 +90,7 @@
             .catch(onError)
     }
 
-    /**
-     * Handle the error of the form submission
-     * @param error {Error} - Error of the form submission
-     */
+
     function onError(error) {
         console.error("Error:", error)
 
diff --git a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/reset-password.tag b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/reset-password.tag
index 8bf91da..797d6e3 100644
--- a/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/reset-password.tag
+++ b/S2/DevWeb/Projet/src/main/webapp/WEB-INF/tags/forms/reset-password.tag
@@ -26,19 +26,12 @@
     const submitButton = document.querySelector("input[type=submit]");
     const inputs = resetPasswordForm.querySelectorAll("input[type='text'], input[type='password']");
 
-
-    // Form fields
     const tokenInput = document.querySelector("input#token");
     const passwordInput = document.querySelector("input#password");
     const repasswordInput = document.querySelector("input#repassword");
 
-    // Add event listener to the form submission
     resetPasswordForm.addEventListener("submit", onSubmit)
 
-    /**
-     * Handle the form submission with Ajax request
-     * @param event {Event} - Event of the form submission
-     */
     function onSubmit(event) {
         event.preventDefault();
 
@@ -65,14 +58,9 @@
             .finally(() => submitButton.classList.remove("is-loading"));
     }
 
-    /**
-     * Handle the error of the form submission
-     * @param error {Error} - Error of the form submission
-     */
     function onError(error) {
       console.error("Error:", error)
 
-      // Input fields in red
       inputs.forEach(input => {
         input.classList.add("is-danger");
         input.style.animation = "shake 0.5s ease-in-out"
diff --git a/S2/DevWeb/Projet/src/main/webapp/index.html b/S2/DevWeb/Projet/src/main/webapp/index.html
index 5bdaaf3..d3d7497 100644
--- a/S2/DevWeb/Projet/src/main/webapp/index.html
+++ b/S2/DevWeb/Projet/src/main/webapp/index.html
@@ -52,18 +52,18 @@
 </nav>
 
 <script type="module" defer>
-    // Get all "navbar-burger" elements
+    // Récupération de tous les éléments de classe "navbar-burger"
     const $navbarBurgers = Array.prototype.slice.call(document.querySelectorAll('.navbar-burger'), 0);
 
-    // Add a click event on each of them
+    // Ajout d'un eventListener sur chaque élément
     $navbarBurgers.forEach(el => {
         el.addEventListener('click', () => {
 
-            // Get the target from the "data-target" attribute
+            // Récupération de la cible à partir de l'attribut "data-target"
             const target = el.dataset.target;
             const $target = document.getElementById(target);
 
-            // Toggle the "is-active" class on both the "navbar-burger" and the "navbar-menu"
+            // Toggle de la classe "is-active" sur les éléments
             el.classList.toggle('is-active');
             $target.classList.toggle('is-active');
 
@@ -94,4 +94,4 @@
     </div>
 </footer>
 </body>
-</html>
\ No newline at end of file
+</html>