Merge pull request 'dev' (#58) from dev into master

Reviewed-on: #58
2025-11-07 15:31:18 +00:00 · 2025-11-07 15:31:18 +00:00 · e7deba52e9
commit e7deba52e9
parent 8a14f58ce5 d95c173fa8
10 changed files with 78 additions and 42 deletions
--- a/src/main/java/fr/titionfire/ffsaf/domain/service/AffiliationService.java
+++ b/src/main/java/fr/titionfire/ffsaf/domain/service/AffiliationService.java
@ -267,7 +267,9 @@ public class AffiliationService {
                                }).call(m -> Panache.withTransaction(() -> combRepository.persist(m)));
                    }
                })
-                .call(m -> ((m.getUserId() == null) ? keycloakService.initCompte(m.getId()) :
+                .call(m -> ((m.getUserId() == null) ? keycloakService.initCompte(m.getId())
+                        .onFailure().invoke(t -> LOGGER.warnf("Failed to init account: %s", t.getMessage())).onFailure()
+                        .recoverWithNull() :
                        keycloakService.setClubGroupMembre(m, club).map(__ -> m.getUserId()))
                        .call(userId -> keycloakService.setAutoRoleMembre(userId, m.getRole(), m.getGrade_arbitrage()))
                        .call(userId -> keycloakService.setEmail(userId, m.getEmail())))
@ -275,7 +277,7 @@ public class AffiliationService {
                        .call(l1 -> l1 != null && l1.stream().anyMatch(l -> l.getSaison() == saison) ?
                                Uni.createFrom().nullItem() :
                                Panache.withTransaction(() -> licenceRepository.persist(
-                                        new LicenceModel(null, m, club.getId(), saison, null, true, false)))
+                                                new LicenceModel(null, m, club.getId(), saison, null, true, false)))
                                        .call(licenceModel -> ls.logA(LogModel.ActionType.ADD, m.getObjectName(),
                                                licenceModel))));
    }
--- a/src/main/java/fr/titionfire/ffsaf/domain/service/LicenceService.java
+++ b/src/main/java/fr/titionfire/ffsaf/domain/service/LicenceService.java
@ -18,6 +18,7 @@ import io.smallrye.mutiny.unchecked.Unchecked;
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.inject.Inject;
 import org.hibernate.reactive.mutiny.Mutiny;
+import org.jboss.logging.Logger;

 import java.util.List;
 import java.util.function.Consumer;
@ -26,6 +27,7 @@ import java.util.function.Function;
@WithSession
@ApplicationScoped
 public class LicenceService {
+    private static final Logger LOGGER = Logger.getLogger(LicenceService.class);

    @Inject
    LicenceRepository repository;
@ -125,7 +127,9 @@ public class LicenceService {
                        .chain(() -> combRepository.persist(membreModel))
                : Uni.createFrom().nullItem())
                .call(__ -> (membreModel.getUserId() == null) ?
-                        keycloakService.initCompte(membreModel.getId())
+                        keycloakService.initCompte(membreModel.getId()).onFailure()
+                                .invoke(t -> LOGGER.infof("Failed to init account: %s", t.getMessage())).onFailure()
+                                .recoverWithNull()
                        : Uni.createFrom().nullItem());
    }

--- a/src/main/java/fr/titionfire/ffsaf/domain/service/MembreService.java
+++ b/src/main/java/fr/titionfire/ffsaf/domain/service/MembreService.java
@ -210,11 +210,12 @@ public class MembreService {
                    return repository.list("licence IN ?1 OR LOWER(lname || ' ' || fname) IN ?2 OR email IN ?3",
                            data2.stream().map(SimpleMembreInOutData::getLicence).filter(Objects::nonNull).toList(),
                            data2.stream().map(o -> (o.getNom() + " " + o.getPrenom()).toLowerCase()).toList(),
-                            data2.stream().map(SimpleMembreInOutData::getEmail).filter(o -> o != null && !o.isBlank()).toList());
+                            data2.stream().map(SimpleMembreInOutData::getEmail).filter(o -> o != null && !o.isBlank())
+                                    .toList());
                })
                .call(Unchecked.function(membres -> {
                    for (MembreModel membreModel : membres) {
-                        if (!Objects.equals(membreModel.getClub(), clubModel.get())){
+                        if (!Objects.equals(membreModel.getClub(), clubModel.get())) {
                            LOGGER.info("Similar membres found: " + membreModel);
                            throw new DForbiddenException(
                                    "Le membre n°" + membreModel.getLicence() + " n'appartient pas à votre club");
@ -225,7 +226,8 @@ public class MembreService {
                        MembreModel model = membres.stream()
                                .filter(m -> Objects.equals(m.getLicence(), dataIn.getLicence()) || m.getLname()
                                        .equals(dataIn.getNom()) && m.getFname().equals(dataIn.getPrenom()) ||
-                                        Objects.equals(m.getFname(), dataIn.getEmail())).findFirst()
+                                        (dataIn.getEmail() != null && !dataIn.getEmail().isBlank() && Objects.equals(
+                                                m.getFname(), dataIn.getEmail()))).findFirst()
                                .orElseGet(() -> {
                                    MembreModel mm = new MembreModel();
                                    mm.setClub(clubModel.get());
--- a/src/main/java/fr/titionfire/ffsaf/rest/from/AffiliationRequestForm.java
+++ b/src/main/java/fr/titionfire/ffsaf/rest/from/AffiliationRequestForm.java
@ -11,7 +11,7 @@ import org.eclipse.microprofile.openapi.annotations.media.Schema;
 import org.jboss.resteasy.reactive.PartType;

@Getter
-@ToString
+@ToString(exclude = {"status", "logo"})
 public class AffiliationRequestForm {
    @Schema(description = "L'identifiant de l'affiliation. (null si nouvelle demande d'affiliation)")
    @FormParam("id")
--- a/src/main/webapp/src/App.jsx
+++ b/src/main/webapp/src/App.jsx
@ -1,11 +1,11 @@
 import {useEffect, useRef} from 'react'
 import {Nav} from "./components/Nav.jsx";
-import {createBrowserRouter, Outlet, RouterProvider, useRouteError} from "react-router-dom";
+import {createBrowserRouter, Outlet, RouterProvider, useLocation, useRouteError} from "react-router-dom";
 import {Home} from "./pages/Homepage.jsx";
 import {AdminRoot, getAdminChildren} from "./pages/admin/AdminRoot.jsx";
 import {AuthCallback} from "./components/auhCallback.jsx";
-import {KeycloakContextProvider, useAuthDispatch} from "./hooks/useAuth.jsx";
-import {check_validity} from "./utils/auth.js";
+import {KeycloakContextProvider, useAuth, useAuthDispatch} from "./hooks/useAuth.jsx";
+import {check_validity, login} from "./utils/auth.js";
 import {ToastContainer} from "react-toastify";

 import './App.css'
@ -14,6 +14,7 @@ import {ClubRoot, getClubChildren} from "./pages/club/ClubRoot.jsx";
 import {DemandeAff, DemandeAffOk} from "./pages/DemandeAff.jsx";
 import {MePage} from "./pages/MePage.jsx";
 import {CompetitionRoot, getCompetitionChildren} from "./pages/competition/CompetitionRoot.jsx";
+import {FallingLines} from "react-loader-spinner";

 const router = createBrowserRouter([
    {
@ -113,6 +114,43 @@ function Root() {
                theme="light"
                transition: Flip
            />
+            <ReAuthMsg/>
+        </div>
+    </>
+}
+
+function ReAuthMsg() {
+    const {is_authenticated} = useAuth()
+    const location = useLocation()
+
+    const notAuthPaths = [
+        /^\/$/s,
+        /^\/affiliation(\/)?$/s,
+        /^\/affiliation\/ok(\/)?$/s,
+        /^\/complete\/auth.*$/s
+    ]
+
+    if (is_authenticated || notAuthPaths.some(r => r.test(location.pathname)))
+        return <></>
+    return <>
+        <div className="overlayBG" style={{position: 'fixed'}}>
+            <div className="overlayContent" onClick={(e) => {
+                e.stopPropagation()
+            }}>
+                <div className="card">
+                    <div className="card-header">
+                        <h5>Session expirée</h5>
+                    </div>
+                    <div className="card-body">
+                        <p className="card-text">Votre session a expirée, veuillez vous reconnecter pour continuer à
+                            utiliser l'application.</p>
+                    </div>
+                    <div className="card-footer">
+                        <button className="btn btn-primary" onClick={() => login()} style={{marginRight: "0.5em"}}>Se reconnecter</button>
+                        <a className="btn btn-secondary" href="/">Accueil</a>
+                    </div>
+                </div>
+            </div>
        </div>
    </>
 }
--- a/src/main/webapp/src/components/MemberCustomFiels.jsx
+++ b/src/main/webapp/src/components/MemberCustomFiels.jsx
@ -88,13 +88,14 @@ export function CountryList({name, text, value, values = undefined, disabled = f
    </div>
 }

-export function TextField({name, text, value, placeholder, type = "text", disabled = false, required = true}) {
-    return <div className="row">
-        <div className="input-group mb-3">
+export function TextField({name, text, value, placeholder, type = "text", disabled = false, required = true, ttip = null}) {
+    return <div className="row mb-3">
+        <div className="input-group">
            <span className="input-group-text" id={name}>{text}</span>
            <input type={type} className="form-control" placeholder={placeholder ? placeholder : text} aria-label={name}
                   name={name} aria-describedby={name} defaultValue={value} disabled={disabled} required={required}/>
        </div>
+        {ttip}
    </div>
 }

--- a/src/main/webapp/src/pages/DemandeAff.jsx
+++ b/src/main/webapp/src/pages/DemandeAff.jsx
@ -20,11 +20,11 @@ function reconstruireAdresse(infos) {
    console.log(infos);
    let adresseReconstruite = "";

-    if(infos.numero_voie === null){
+    if (infos.numero_voie === null) {
        if (infos.complement_adresse) {
            adresseReconstruite += formatAdresse(infos.complement_adresse) + ', ';
        }
-    }else{
+    } else {
        adresseReconstruite += infos.numero_voie + ' ';
    }

@ -46,6 +46,13 @@ function reconstruireAdresse(infos) {
    return adresseReconstruite;
 }

+function getSaisonToAff(currentDate = new Date()) {
+    if (currentDate.getMonth() >= 7) { //aout et plus
+        return currentDate.getFullYear()
+    } else {
+        return currentDate.getFullYear() - 1
+    }
+}

 export function DemandeAff() {
    const {hash} = useLocation();
@ -145,7 +152,7 @@ export function DemandeAff() {
    }

    return <div>
-        <h1>Demande d'affiliation</h1>
+        <h1>Demande d'affiliation {getSaisonToAff() + "-" + (getSaisonToAff() + 1)}</h1>
        <p>L'affiliation est annuelle et valable pour une saison sportive : du 1er septembre au 31 août de l’année
            suivante.</p>
        Pour s’affilier, une association sportive doit réunir les conditions suivantes :
@ -216,7 +223,6 @@ function AssoInfo({initData, needFile}) {
    const [rna, setRna] = useState(initData.rna ? initData.rna : "")
    const [rnaEnable, setRnaEnable] = useState(false)
    const [adresse, setAdresse] = useState(initData.address ? initData.address : "")
-    const [saison, setSaison] = useState(initData.saison ? initData.saison : getSaison())
    const [contact, setContact] = useState(initData.contact ? initData.contact : "")

    const fetchSiret = () => {
@ -245,26 +251,8 @@ function AssoInfo({initData, needFile}) {
                setAdresse(reconstruireAdresse(data2.etablissement_siege))
        })
    }
-
-    const currentSaison = getSaison();
-
    return <>
-        <div className="input-group mb-3">
-            <div className="input-group-text">
-                <input className="form-check-input mt-0" type="radio" value={currentSaison} aria-label={currentSaison + "-" + (currentSaison + 1)}
-                       name={"saison"} checked={saison === currentSaison}
-                       onChange={e => setSaison(Number(e.target.value))}/>
-                {currentSaison + "-" + (currentSaison + 1)}
-            </div>
-            <span className="input-group-text">OU</span>
-            <div className="input-group-text">
-                <input className="form-check-input mt-0" type="radio" value={currentSaison + 1}
-                       aria-label={(currentSaison + 1) + "-" + (currentSaison + 2)}
-                       name={"saison"} checked={saison === currentSaison + 1}
-                       onChange={e => setSaison(Number(e.target.value))}/>
-                {(currentSaison + 1) + "-" + (currentSaison + 2)}
-            </div>
-        </div>
+        <input name="saison" value={getSaisonToAff()} readOnly hidden/>

        <div className="input-group mb-3">
            <span className="input-group-text" id="basic-addon1">Nom de l'association*</span>
--- a/src/main/webapp/src/pages/admin/affiliation/AffiliationReqPage.jsx
+++ b/src/main/webapp/src/pages/admin/affiliation/AffiliationReqPage.jsx
@ -8,7 +8,6 @@ import {RoleList, TextField} from "../../../components/MemberCustomFiels.jsx";
 import {useEffect, useRef, useState} from "react";
 import {FontAwesomeIcon} from "@fortawesome/react-fontawesome";
 import {faFilePdf} from "@fortawesome/free-solid-svg-icons";
-import {ConfirmDialog} from "../../../components/ConfirmDialog.jsx";

 const vite_url = import.meta.env.VITE_URL;

@ -166,7 +165,7 @@ function Content({data, refresh}) {
                <input name="id" value={data.id} readOnly hidden/>
                <div className="card-header">Demande d'affiliation</div>
                <div className="card-body text-center">
-                    {data.club && <h5>Ce club a déjà ete affilier (affiliation n°{data.club_no_aff})</h5>}
+                    {data.club && <h5>Ce club a déjà été affilié (affiliation n°{data.club_no_aff})</h5>}
                    <h4 id="saison">Saison {data.saison}-{data.saison + 1}</h4>

                    <div className="row mb-3">
--- a/src/main/webapp/src/pages/club/club/AffiliationCard.jsx
+++ b/src/main/webapp/src/pages/club/club/AffiliationCard.jsx
@ -139,10 +139,10 @@ function ModalContent2({clubData, data}) {
            }
        }

-        if (list.length !== 3) {
-            toast.error("Il faut sélectionner 3 membres pour renouveler l'affiliation")
-            return
+        while (list.length < 3) {
+            list.push(-1)
        }
+
        apiAxios.get(`/club/renew/${clubData.id}?m1=${list[0]}&m2=${list[1]}&m3=${list[2]}`).then(data => {
            navigate('/affiliation#d' + encodeURI(JSON.stringify(data.data)))
        })
--- a/src/main/webapp/src/pages/club/member/InformationForm.jsx
+++ b/src/main/webapp/src/pages/club/member/InformationForm.jsx
@ -49,7 +49,9 @@ export function InformationForm({data}) {
                <TextField name="lname" text="Nom" value={data.lname}/>
                <TextField name="fname" text="Prénom" value={data.fname}/>
                <TextField name="email" text="Email" value={data.email} placeholder="name@example.com"
-                           type="email"/>
+                           type="email" ttip={<small className="form-text">L'email sert à la création de compte pour se connecter au site et doit être unique. <br/>
+                    Pour les mineurs, l'email des parents peut être utilisé plusieurs fois grâce à la syntaxe suivante : {'email.parent+<caractères alphanumériques>@exemple.com'}.<br/>
+                            Exemples : mail.parent+1@exemple.com, mail.parent+titouan@exemple.com, mail.parent+cedrique@exemple.com</small>}/>
                <OptionField name="genre" text="Genre" value={data.genre}
                             values={{NA: 'N/A', H: 'H', F: 'F'}}/>
                <CountryList name="country" text="Pays" value={data.country}/>