Aller au contenu
Candidater

Journal des modifications

Les évolutions de Cohez.io, de la plus récente à la plus ancienne.

44 versions correspondent à « HelloAsso ».

Versions 1 à 10 sur 44

0.25.11.0 — 2026-09-29

Ajouts

  • Aide en ligne : rubrique Paniers & paiements (#333, 2026-09-29). Six guides pour les trésoriers et dirigeants : comprendre un panier, versements, paiement manuel, paiement en ligne HelloAsso, relances, annuler une inscription. Les guides décrivent le comportement actuel, limites comprises (pas de remboursement par Cohez.io, versement corrigé par rejet, pas de paiement en plusieurs fois HelloAsso). L'accueil de l'aide ajoute l'étape « Choisir les moyens de paiement » avant l'ouverture, et les pages Inscriptions renvoient vers la nouvelle rubrique. Chemins figés par des tests ; captures dans une PR suivante.

0.25.5.3 — 2026-09-27

Corrections

  • Dédoublonnage des personnes : une date de naissance absente d'un côté ne crée plus de doublon (#321, 2026-09-27). À e-mail et prénom égaux (casse et espaces ignorés), PersonIdentityMatcher::whereDedupKey() traite une date de naissance absente — sur la fiche ou dans la saisie — comme compatible : le funnel d'inscription publique et l'import HelloAsso retrouvent une fiche créée sans date (back-office, import) au lieu d'en créer une seconde, et complètent sa date sans jamais écraser une date existante. Deux dates renseignées et distinctes restent deux personnes (fratries, jumeaux). Une fiche sans date qui n'est que payeur ou responsable légal (sans adhésion) n'est pas rapprochée d'une saisie datée, pour ne pas rattacher un enfant homonyme à la fiche de son parent. Quand plusieurs fiches correspondent, celle qui porte déjà un Member est retenue, puis celle dont la date est exactement celle saisie (preferringMemberThenExactBirthdate()). Un nouveau Member créé sur une fiche datée retrouvée sans date saisie tire son statut mineur de la date de la fiche. Le contrôle de doublon d'inscription, le garde-fou d'adhésion active et le préflight d'import (« Réutilisée » au lieu de « Ambiguë », message dédié pour la fiche payeur / responsable légal) suivent la même règle. L'import résout les adhérents d'un panier avant son payeur et ses représentants : un adhérent qui paie pour lui-même retrouve sa fiche sans date au lieu d'en créer une seconde ; le préflight bloque une fiche sans date ni adhésion dont l'identité est payeur ou représentant d'une autre commande du fichier (issue dépendante de l'ordre des commandes), et son message d'ambiguïté cite de préférence la fiche datée. Le garde-fou d'adhésion active et le contrôle « déjà inscrit » ignorent une fiche sans date quand une fiche à la date saisie porte déjà une adhésion : une fille datée n'est plus bloquée par l'inscription de sa mère homonyme sans date sur l'e-mail familial. Les doublons déjà en base ne sont ni fusionnés ni corrigés.

0.25.5.2 — 2026-09-26

Corrections

  • Unicité de l'e-mail adhérent en back-office insensible à la casse et aux espaces stockés (#320, 2026-09-26). EloquentMemberRepository::emailExistsInTenant (et findByEmail) compare désormais LOWER(TRIM(persons.email)) : un e-mail enregistré avec des majuscules ou des espaces (import HelloAsso, données historiques) n'échappe plus au contrôle d'unicité à la création ou à la modification d'un adhérent.

0.25.5.1 — 2026-09-26

Corrections

  • Dédoublonnage des personnes insensible à la casse et aux espaces de l'e-mail et du prénom (#319, 2026-09-26). Le funnel d'inscription publique et l'import HelloAsso reconnaissent désormais une personne déjà en base même si son e-mail ou son prénom diffère par la casse ou par des espaces, via PersonIdentityMatcher (LOWER(TRIM())) : dédoublonnage PersonDeduplicationService, contrôle de doublon d'inscription, garde-fou d'adhésion active, payeur et responsable légal (funnel et ImportPayerResolver), préflight d'import. Quand plusieurs fiches correspondent, la fiche qui porte déjà un Member (ou un Payeur) est retenue, puis la plus ancienne ; le contrôle de doublon et le garde-fou d'adhésion examinent toutes les fiches. Les doublons déjà en base ne sont ni fusionnés ni corrigés.

0.25.0.1 — 2026-09-04

Ajouts

  • Chaque méthode de paiement affiche désormais des instructions de règlement, identiques sur les trois surfaces qui les rendent (#286, 2026-09-04). PaymentInstructionsResolver remplace BankTransferInstructionsResolver — supprimé sans résidu — et produit un PaymentInstructionsSet de 0 à 2 blocs (acompte / reste à régler) à chaînes déjà rendues ; l'e-mail de confirmation de panier, l'e-mail de relance et l'écran de succès public se contentent d'afficher : trois partials Blade (HTML, texte, markdown) et le composant Vue PaymentInstructions.vue. Auparavant, seul le virement produisait des instructions ; chèque, espèces, carte hors ligne, HelloAsso et moyen personnalisé n'en affichaient aucune.
  • La description d'un moyen de paiement personnalisé survit désormais au submit du formulaire d'inscription publique (#286, 2026-09-04). Elle est portée par le bloc d'instructions rendu sur les trois surfaces au lieu de disparaître une fois l'inscription passée.

Corrections

  • La carte bancaire hors ligne annonçait un règlement en ligne (#286, 2026-09-04). PaymentInstructionsResolver::actionText() dégroupe désormais PaymentMethod::CreditCard (« Règlement par carte bancaire sur place, auprès de l'association. ») de PaymentMethod::HelloAsso (paiement en ligne), auparavant confondues sous la même phrase.
  • L'ordre du chèque se replie sur le nom du tenant (#286, 2026-09-04). tenant_payment_methods ne porte pas encore de champ dédié au destinataire ; PaymentInstructionsResolver::checkPayee() rend tenant->name. Le paramètre $row, inutilisé pour l'instant, est conservé pour y brancher ce champ sans changer les appelants.
  • Les deux canaux de paiement d'une règle d'acompte sont désormais distingués (#286, 2026-09-04). CampaignDepositRule::settlement_method est le canal de l'acompte (« Moyen de règlement de l'acompte » côté admin), tandis que la méthode choisie par l'adhérent — qui déclenche la règle via applicable_payment_methods — reste celle du reste à régler. Quand les deux blocs sont rendus, chacun porte son étiquette de portée (« Pour l'acompte », puis « Pour le solde ») : les deux e-mails alignent les blocs sans séparateur visuel, et un second bloc nu s'y lit comme une seconde consigne d'acompte. L'étiquetage du bloc du reste est décidé après filtrage : resté seul, ce bloc couvre tout le montant réclamé et non un reliquat, donc il ne porte aucune étiquette — comme la ligne « Moyen de paiement : … » qu'affichait l'e-mail auparavant. Le docblock de CampaignDepositRule, qui décrivait settlement_method comme la méthode du solde, est corrigé.
  • L'e-mail de relance annonçait un total de panier que le bloc « Pour l'acompte » contredisait en silence (#286, 2026-09-08). Sous règle d'acompte, la relance affichait le solde entier du panier puis, juste en dessous, un bloc d'instructions étiqueté « Pour l'acompte » ne portant aucun chiffre : le lecteur rattachait le total au canal de l'acompte et pouvait virer l'intégralité sur le mauvais canal. CartPaymentReminderMail calcule désormais le montant réellement exigible — par ligne, l'acompte restant (deposit_amount_due - total_paid) tant qu'il est strictement positif et inférieur au solde, sinon le solde — et le mail ajoute « À régler maintenant (acompte) : … » suivi du reliquat, uniquement quand cette somme est strictement inférieure au total. Garde d'homogénéité : la ligne n'apparaît que si toutes les inscriptions relancées sont en contexte d'acompte. Les blocs d'instructions sont résolus sur la seule ligne représentative ; sur un panier mixte, une somme étiquetée « acompte » ne correspondrait à aucun des canaux affichés, donc le total reste seul et le comportement est inchangé. Correctif du montant réclamé côté relance, hors périmètre initial des instructions de paiement de cette PR.
  • La ligne représentative d'un panier pouvait être une ligne gratuite dont la méthode de paiement avait été effacée, faisant partir la confirmation sans instructions (#286, 2026-09-04). whereIn sur des UUID ne garantit aucun ordre : le panier est désormais trié par member_index, et la ligne représentative choisie par le nouveau helper CartRepresentative::pick() — la première ligne au montant dû strictement positif, ou la première ligne à défaut d'un panier entièrement gratuit.
  • Le mail de confirmation d'un panier réglé (0 €, ou soldé) ne porte plus la ligne « Moyen de paiement : — » (#286, 2026-09-04). Un panier Validated et soldé (hasOutstandingBalance() à false) fait court-circuiter PaymentInstructionsResolver::forCart() en PaymentInstructionsSet::empty() : aucun bloc d'instructions n'est rendu, volontairement — pas de paiement dû, pas d'instructions. Un panier Validated mais encore partiellement payé continue de produire le bloc de son reste à régler. À l'inverse, un panier encore dû dont aucune méthode ne se résout reçoit un bloc générique (« Moyen de paiement : — » et phrase de règlement générique), comme avant : le jeu vide signifie donc « plus rien à régler », jamais « méthode inconnue ». FreeCartSettlementTest > I-1 est aligné sur ce comportement.
  • Un moyen de paiement dépublié pouvait voir son RIB ressortir dans les mails et sur l'écran de succès public (#286, 2026-09-08). Les deux requêtes TenantPaymentMethod de PaymentInstructionsResolver::resolveMethodKey() ne filtraient ni is_enabled ni is_public ; elles passent désormais par le scope public(), le même que le reste de la surface publique. Le canal le plus exposé est settlement_method : il est choisi par l'admin dans CampaignDepositRule::eligiblePaymentMethods(), qui rend tous les enums sauf HelloAsso sans consulter la configuration du tenant — il ne passait donc par aucun filtre de publication ailleurs. Ligne absente ⇒ pas d'échec : le libellé de la méthode reste rendu, seuls les détails (IBAN, BIC…) disparaissent au profit de la phrase générique. Aucun orderBy n'est ajouté : l'index partiel uq_tenant_std_payment garantit au plus une ligne par (tenant, méthode standard).
  • Une panne de résolution retirait toute la section paiement des mails au lieu du seul bloc de détails (#286, 2026-09-08). Le catch (Throwable) de PaymentInstructionsResolver::forCart() rendait PaymentInstructionsSet::empty(), qui signifie « plus rien à régler » pour les trois surfaces — la confirmation et la relance annonçaient donc l'absence de paiement dû sur un panier encore dû. Avant ce résolveur, PublicCartConfirmationMail calculait sa ligne « Moyen de paiement » de son côté et une panne ne coûtait que le bloc RIB. Le catch renvoie désormais le bloc générique tant qu'un solde subsiste ; le jeu vide n'est conservé que si le panier est soldé — ou si la lecture du solde échoue elle aussi, seul cas où il reste l'état sûr. PublicCartConfirmationMail charge en outre payments explicitement : $settledNotice appelle hasOutstandingBalance(), et ne dépendre que du chargement en effet de bord du résolveur rendait cette lecture sensible à l'ordre des appels.

0.24.0.0 — 2026-08-25

Ajouts

  • registrations:import {tenant} {file} --campaign=<uuid> --mapping=<json> [--apply] [--scan] importe en masse des inscriptions historiques depuis un export HelloAsso converti au format CSV pivot. Elle répond au rapatriement de Cholet Water-Polo (environ 700 inscriptions, échéance 2026-09-01). PivotCsvReader lit le fichier ligne à ligne en générateur, jamais chargé entier en mémoire ; TierMapping relie chaque libellé de tarif du fichier à une proposition existante ; ImportPayerResolver détermine le payeur et les représentants légaux des mineurs. ImportPreflight::build() est une fonction pure et en lecture seule qui produit un ImportPlan — par défaut la commande simule et affiche ce plan sans rien écrire ; --apply déclenche l'écriture, après confirmation interactive, par ImportRegistrationsUseCase, seul écrivain, sous verrou. L'écriture est idempotente au niveau panier : rejouer le même fichier après un échec partiel ne duplique rien et ne demande aucun nettoyage manuel. --scan profile un fichier (en-tête, distribution des tarifs) avant de lancer un import, sans exiger --campaign ni --mapping. Un test dédié garantit que la commande n'est jamais automatisée (ni post_build.sh, ni le planificateur) : c'est un outil d'opérateur, pas un job récurrent.
  • Le préflight confronte désormais chaque identité du fichier à la base, en lecture seule, avant d'autoriser la moindre écriture. Chaque adhérent est classé en quatre dispositions par rapport à ce que PersonDeduplicationService::findOrCreate() (INTOUCHÉ) retrouverait réellement : Réutilisée (fiche existante retrouvée à l'identique), Ambiguë — un e-mail est fourni, une fiche du même e-mail et du même prénom existe, mais la correspondance exacte échoue (casse d'e-mail ou date de naissance divergente) —, Aveugle — aucun e-mail dans le fichier alors qu'une fiche candidate existe déjà sur prénom/nom/date de naissance —, ou Créée. Ambiguë et Aveugle sont bloquantes ; un avertissement agrégé annonce « N adhérent(s) du fichier rejoindront une fiche existante, M seront créés. ». Motif : PersonDeduplicationService compare l'e-mail à la casse exacte près mais le prénom insensible à la casse, et ne cherche rien quand l'e-mail est nul — sans cette confrontation, l'import aurait créé des doublons en silence dans ces trois cas précis.
  • Une même proposition visée deux fois par un même adhérent dans le fichier, à des montants différents, fait échouer le préflight — jamais cumulée. attachPropositions() (ImportRegistrationsUseCase) ne conserve que le premier unit_amount vu par proposition et cumule les quantités : deux lignes à des libellés de tarif distincts pointant par erreur vers la même proposition auraient fait disparaître de l'argent en silence, sans qu'aucune contrainte SQL ne le détecte. validatePropositionAmounts() refuse ce cas en amont et détaille les lignes et montants en conflit — les cumuler (quantité × montant) aurait masqué l'incohérence de saisie au lieu de la signaler.
  • Les UUID de proposition passés via --mapping= sont normalisés en minuscules dans TierMapping. Str::isUuid() est insensible à la casse, mais propositions.id est stocké en minuscules (colonne uuid native sous PostgreSQL) : sans cette normalisation, un UUID saisi en majuscules passait la garde de forme puis échouait silencieusement le array_diff() (sensible à la casse) de validatePropositions(), rapporté à tort comme « introuvable ou hors de la campagne ». Le symptôme est le même sur les deux moteurs, seul le chemin diffère : sous PostgreSQL la colonne uuid native normalise la casse, la requête trouve donc la ligne et la rend en minuscules — c'est array_diff() qui la déclare ensuite absente ; sous SQLite, où id est du texte, c'est la requête elle-même qui ne trouve rien. Dans les deux cas l'opérateur lisait « introuvable » sur une proposition valide.

Notes techniques

  • Le cast Eloquent date sans format explicite a failli rendre la garde anti-doublon elle-même aveugle. Il sérialise l'écriture avec l'heure (2016-05-02 00:00:00) même pour une colonne date native : un whereIn() sur des chaînes AAAA-MM-JJ nues ne retrouve donc rien sous SQLite — comparaison de chaînes brute, invisible en CI, PostgreSQL n'ayant pas ce travers. Les requêtes de confrontation de la classification « Aveugle » utilisent whereDate(), la même fonction date(colonne) que PersonDeduplicationService::findOrCreate() applique déjà pour sa propre correspondance exacte — mêmes garanties cross-moteur. Sans ce choix, la garde anti-faux-négatif décrite ci-dessus aurait elle-même été un faux négatif : verte en CI, muette en production.

0.23.0.0 — 2026-08-18

Ajouts

  • memberships:sync {tenant} [--apply] [--anchor=registered_at|season_start] réconcilie les périodes d'adhésion d'un tenant, saison par saison. Elle remplace memberships:backfill, qui ancrait les périodes sur created_at sans jamais rattacher de saison. Par défaut la commande simule : elle n'écrit rien et annonce ce qu'elle ferait. Simulation et application consomment le même plan (MembershipSyncPlan, produit par la fonction pure SyncMemberMembershipsUseCase::plan()) : le rapport ne peut pas diverger de l'écriture, contrairement à un miroir de dry-run maintenu à la main. Un test de parité exécute les deux modes sur le même décor et vérifie que les compteurs annoncés sont identiques et que l'application produit exactement le delta annoncé. La parité vaut aussi sur la branche d'échec : plan() rejoue la garde de non-chevauchement que l'écriture applique, si bien qu'un adhérent en conflit est compté en échec — et non annoncé comme écrit — dans les deux modes. Un second test l'exerce sur une fixture en conflit, avec la même liste de compteurs attendus.
  • Une pré-vérification refuse le tenant entier avant de toucher la moindre ligne. Trois motifs : une saison inversée (start_date > end_date, nommée dans le message) ; deux saisons du tenant qui se chevauchent ; au moins une adhésion non annulée sans end_date — ligne héritée non bornée, correction manuelle ou import. Ce troisième motif nomme les adhérents porteurs (jusqu'à 20) : sans eux le refus est un mur sans issue. Le critère est la borne manquante, pas members.subscription_type — le message ne préjuge donc pas de la cause. Ce refus survit à la suppression du régime à vie (voir plus bas) : son critère a toujours été la borne, jamais le régime, et memberships.end_date NULL reste un état de schéma atteignable en base alors qu'aucun chemin applicatif ne le produit plus. Le moteur ancré sur la saison ne sait ni prolonger ni annuler une adhésion sans fin ; l'opérateur tranche à la main, puis relance. Deux tests l'épinglent — le refus lui-même sur une ligne héritée, et le fait que l'écouteur ne puisse plus l'armer. Les saisons chevauchantes ne sont pas un détail d'hygiène : elles donnent deux season_id cibles pour une même date, que l'index unique partiel accepte et que la contrainte GiST refuse — la divergence SQLite/PostgreSQL pointerait alors dans le mauvais sens et la CI verte mentirait. Un refus n'écrit rien et sort en échec.
  • Rattrapage des adhérents sans inscription validée, par la seule voie déterministe. Quand aucune inscription ne fournit de cible, la commande cherche la saison dont end_date correspond exactement à membership_expires_at et écrit cette saison pleine avec source = Import. Il n'existe pas de repli sur members.status : un adhérent ACTIVE sans date d'expiration est signalé, jamais deviné. Le deviner aurait imposé de retenir la saison contenant le jour courant, c'est-à-dire de faire dépendre le plan de la date d'exécution — deux passages à deux dates auraient fabriqué deux périodes différentes pour le même adhérent, ce qui ruine la garantie « rejouable » sur laquelle repose le mode d'emploi de migration. Et members.status ne prouve pas l'adhésion : aucun écrivain applicatif ne le rétrograde, MemberStatus::EXPIRED n'étant posé que par les seeders de démonstration. Sur la base de développement (sauvegarde de production anonymisée), le rattrapage résout 6 adhérents (3 club-asso, 2 demo-cholet-waterpolo, 1 demo-epgv-44) et en signale 2 sans saison déductible (1 club-asso, 1 les-fontaines-musicales) — attendu, pas une anomalie. Un adhérent sans aucune cible sort avant la réconciliation : ses lignes héritées sont laissées intactes, jamais annulées faute de cible. Un test dédié rejoue la sélection des cibles sous deux horloges distantes de dix mois et exige le même plan.
  • Garantie que memberships:sync n'est jamais automatisée (tests/Feature/Console/MembershipCommandsNotAutomatedTest.php) : ni clevercloud/post_build.sh, ni pre_build.sh, ni cron.sh, ni cron.json, ni routes/console.php, ni bootstrap/app.php ne la mentionnent, et le planificateur résolu ne l'enregistre pas. Chaque cas vérifie d'abord que le fichier existe : un fichier renommé fait échouer le test au lieu de le rendre vide, et l'assertion sur le planificateur est protégée contre la vacuité (la liste des commandes planifiées doit être non vide).

Modifications

  • La période d'adhésion est ancrée sur la saison de la campagne, plus sur des réglages de tenant. CreateMembershipFromRegistrationListener ne consulte plus MembershipSettings ni MembershipPeriodCalculator : il résout la saison via registration.campaign.season et appelle MembershipPeriod::forSeason(seasonStart:, seasonEnd:, anchor: $event->validatedAt()). La règle est celle arrêtée en conception : fin = saison.fin toujours, début = max(saison.début, instant de validation), et repli sur la saison entière si la validation tombe après la fin de saison. La garde MembershipSettings::isConfigured() disparaît : un tenant sans réglage d'adhésion produisait jusqu'ici un écouteur muet, donc des adhérents validés sans aucune période — la saison suffit désormais à décider. Le season_id et created_by: 'listener' sont transmis à RecordMembershipPeriodUseCase, seul écrivain, dont l'idempotence « premier écrivain gagne » est calée sur la saison : une seconde validation dans la même saison n'écrit rien, même à une date postérieure ; une validation dans une autre saison écrit une seconde ligne.
  • RegistrationValidated transporte l'instant de validation. L'événement porte validatedAt et les six sites de dispatch (CancelRegistrationService, SettleFreeCartService, VersementService, RegistrationTransitionController, HelloAssoPaymentConfirmationService, BackfillCancelledAllocationsCommand) le renseignent — un seul CarbonImmutable::now() hissé par opération, si bien que toutes les inscriptions soldées ensemble partagent le même ancrage. C'est l'événement qui fait autorité, pas l'horloge de l'écouteur : un dispatch différé par la file ou rejoué reproduit la période d'origine au lieu d'en fabriquer une décalée.

Retraits

  • Le régime d'adhésion « à vie » est supprimé. Le cas SubscriptionType::LIFETIME disparaît de l'énumération, et avec lui la surcharge de l'écouteur qui remplaçait la période de saison par une période sans fin, le constructeur nommé MembershipPeriod::lifetime(), l'état MemberFactory::lifetime() (aucun appelant) et la traduction « À vie ». Motif : une end_date NULL contredit frontalement l'invariant du moteur — une période vit dans sa saison — et une seule ligne suffisait à faire refuser memberships:sync sur le tenant entier, sans issue automatique. La colonne members.subscription_type est conservée, l'énumération aussi, à un seul cas — non pas comme point d'extension du régime rolling_year, qui appartient à MembershipRenewalType (l'axe du tenant, cf. entrée suivante), mais parce que la colonne reste en base par décision PR3 : elle est castée sur le modèle, portée par l'entité de domaine, le mapper, les DTO et le front. SubscriptionType est l'axe de l'adhérent — quelle formule il a souscrite — et ne calcule aucune borne, sous aucun régime. UpdateMemberRequest valide désormais avec Rule::enum(SubscriptionType::class) au lieu d'une liste Rule::in(...) recopiée à la main, qui aurait continué d'accepter 'lifetime' en silence. Côté front, SubscriptionType (TypeScript) et les types Member / Person sont réduits au même cas unique. MembershipFactory::lifetime() est renommée unbounded() — elle ne modélise plus un régime produit mais un état de schéma, et sert à exercer les gardes qui le refusent.
  • Le régime « année civile » disparaît du code. MembershipPeriodCalculator (calcul fixed_period / rolling_year), le VO MembershipSettings (configuration d'adhésion du tenant) et la commande memberships:backfill sont supprimés avec leurs tests. Plus aucun appelant ne subsistait : la période est ancrée sur la saison depuis l'écouteur réécrit ci-dessus, et memberships:sync remplace le rattrapage. L'enum MembershipRenewalType est conservée comme point d'extension documenté (season seul mode implémenté, rolling_year réservé) — c'est la configuration d'un régime disparu qui part, pas la possibilité d'en ajouter un.
  • Les réglages « type de renouvellement / jour / mois » quittent la page Association. Le bloc disparaît de UpdateAssociationRequest (règles, withValidator() de validation jour/mois, messages), de AssociationController (props Inertia et écriture de settings), de settings/Association/Edit.vue et des clés de traduction (lang/fr/settings.php, resources/js/types/translations.d.ts). Ce n'était pas un simple champ inerte : la garde isset($validated['membership_renewal_type']) était toujours vraie puisque le formulaire postait la valeur, si bien que chaque sauvegarde de la page réécrivait fixed_period + 1/1 dans settings — un réglage menteur que plus rien ne lisait. AssociationController::update() ne touche désormais plus du tout à la colonne settings. AssociationMembershipSettingsTest est réécrit en test de non-régression : les anciens champs postés sont acceptés sans erreur mais non persistés, la page d'édition ne les expose plus, et les autres clés de settings sont préservées.
  • Member::scopeExpired() et Member::scopeExpiringSoon() sont supprimés, ainsi que leurs annotations @method. Ils interrogeaient members.membership_expires_at, colonne vouée à disparaître et déjà sans écrivain depuis 0.22.3.0 : les laisser offrait une réponse plausible mais fausse à « qui est expiré ». Le statut d'adhésion se dérive des lignes memberships. Aucun appelant n'existait.

Notes techniques

  • La propriété validatedAt est nullable avec valeur par défaut, et ce n'est pas cosmétique. SendRegistrationValidatedEmailListener implements ShouldQueue : l'événement est sérialisé en file, et SerializesModels::__unserialize() saute les clés absentes. Un job enfilé avant ce déploiement porte une charge sans la clé ; une propriété typée sans défaut y resterait non initialisée — un état distinct de null dont tout accès lève une Error, qu'un simple ?? ne rattrape pas. Le défaut null rend cet état structurellement impossible et l'accesseur validatedAt() replie sur l'horloge courante. Un test désérialise explicitement une charge amputée de la clé (tests/Feature/Events/RegistrationValidatedTest.php) ; sonde de falsifiabilité passée : retirer le repli fait tomber ce test.
  • L'invariant « RegistrationValidated est dispatché à DB::transactionLevel() === 0 » reste la condition de sûreté de l'écouteur. Son catch (\Throwable) avale l'échec sans rethrow — acceptable seulement parce qu'aucune transaction appelante n'est en cours, sans quoi une écriture d'adhésion ratée laisserait une transaction marquée. MembershipDispatchTransactionLevelTest couvre désormais quatre chemins au lieu de deux (CancelRegistrationService::cancel et HelloAssoPaymentConfirmationService::confirm s'y ajoutent), chacun vérifiant que l'inscription atteint bien Validated et que le niveau de transaction observé au dispatch est le niveau de base.
  • La réconciliation est ancrée sur la saison, jamais sur les bornes. Une ligne existante est annulée si et seulement si son season_id est NULL (ligne héritée) — jamais parce que sa saison est absente des cibles : la seule façon pour une saison d'en sortir est que l'inscription validée qui la portait ait été annulée depuis (la saison, elle, ne peut pas avoir disparu, season_id étant en cascade), et la règle produit est que l'adhésion reste acquise. Une cible est écrite si et seulement si aucune ligne non annulée ne porte déjà cette saison — « premier écrivain gagne, par saison », donc une période déjà posée à la main n'est pas réécrite. Comparer des bornes aurait laissé passer la forme observée sur club-asso : une ligne héritée disjointe de sa cible ne viole aucune contrainte sur aucun moteur, si bien qu'une annulation oubliée laisserait deux lignes actives sans qu'aucune exception ne soit levée. Le test correspondant assère donc toHaveCount(1) sur l'état des lignes, pas l'absence d'exception. La vérification finale, sous verrou, exige que chaque saison cible soit couverte et qu'aucune ligne héritée ne subsiste — une inclusion, pas une égalité d'ensembles : exiger l'égalité transformerait la conservation d'une adhésion acquise en échec par adhérent, et la ligne ne survivrait alors que par rollback. Les lignes conservées hors cible sont comptées et affichées.
  • Une transaction par adhérent, verrou d'abord. SyncMemberMembershipsUseCase::execute() prend lockMember() avant toute lecture, puis annule et écrit dans la même transaction. L'écriture passe par RecordMembershipPeriodUseCase, seul écrivain : sa transaction imbriquée devient un SAVEPOINT, ce dont dépend la traduction d'un 23P01 sans empoisonner la transaction englobante. Un catch (Throwable) par adhérent, placé hors de la transaction, compte l'échec, le journalise et poursuit la campagne ; l'adhérent en échec est défait en entier — ni annulation, ni écriture partielle — et la commande sort en code 1.
  • Le regroupement des inscriptions par saison est fait en PHP, jamais en SQL. L'ancre d'une saison est le min(registered_at) des inscriptions validées de cette saison, départagé par id. Un MIN(uuid) en SQL ne fonctionne que sur PostgreSQL : la CI, sur SQLite, ne l'aurait jamais exercé. Un test vérifie que le résultat ne dépend pas de l'ordre d'insertion.
  • SeasonIndex filtre tenant_id explicitement. En console, TenantContext est nul et le scope global de BelongsToTenant est donc inerte : sans filtre explicite, la commande lirait les saisons de tous les tenants. Aucun test n'initialise de contexte — c'est précisément l'absence de contexte qui rend le scope inerte, et tester sous contexte masquerait un filtre manquant. Les saisons sont chargées une seule fois par exécution (pas de N+1 sur plusieurs milliers d'adhérents).
  • Le moteur ne filtre jamais sur seasons.status, et un test le fixe. Une saison draft est une saison à venir, pas une saison invalide : c'est l'état normal d'une campagne de rentrée ouverte avant le 1er septembre — la campagne réelle de lezards-animes a recueilli ses inscriptions validées entre mai et août 2026 sur une saison 2026-09-01 → 2027-08-31 restée draft. Un filtre sur le statut priverait donc d'adhésion toute une campagne. SeasonFactory produisant Draft par défaut, l'invariant était couvert par accident dans toute la suite ; il est désormais nommé et fixé explicitement (CreateMembershipFromRegistrationListenerTest), ce qui le fait survivre à un changement du défaut de la factory. Sonde de falsifiabilité passée : ignorer les saisons draft dans l'écouteur fait tomber ce test.
  • OverlappingMembershipException est journalisée en warning, pas en error, et attrapée avant \Throwable. Un chevauchement est un refus attendu du moteur (23P01 traduit en PR1), pas une panne : il n'écrit rien et ne remonte pas. Toute autre exception passe en error avec registration_id. Une inscription sans membre ou sans saison résolvable sort en error sans écrire. Ce warning étant la seule trace de l'anomalie, son contexte porte member_id et season_id en plus de registration_id : sans eux l'opérateur ne peut ni identifier l'adhérent ni savoir quelle saison est restée sans période, et doit requêter l'inscription à la main pour chaque ligne du journal. Les deux valeurs sont lues sur l'inscription — colonnes NOT NULL — et non sur les variables assignées à l'intérieur du try, qui ne sont pas garanties affectées quand l'exception survient.
  • Retirer un cas d'énumération est une opération de DONNÉES, pas seulement de code. Le cast enum de Member transforme toute valeur 'lifetime' restée en base en ValueError fatale, sur tout chemin de lecture d'un adhérent — fiche, liste, mapper. La migration normalize_lifetime_subscription_type réécrit donc ces lignes en 'annual' avant que le cast ne les rencontre. Sur la copie de production du 2026-08-18 elle est un no-op mesuré (880 adhérents, tous annual, et zéro memberships.end_date NULL) : elle couvre la fenêtre entre cette mesure et le déploiement, pendant laquelle UpdateMemberRequest acceptait encore 'lifetime'. Elle passe par DB::table() et non Eloquent — Member porte le scope global BelongsToTenant, qui hors contexte tenant ne toucherait aucune ligne, alors qu'une migration doit balayer tous les tenants. Son down() est volontairement vide : le régime d'origine des lignes réécrites n'est pas conservé, et restaurer un cas que l'énumération ne connaît plus casserait la lecture — ici c'est le rollback qui serait destructeur, pas la montée.
  • L'invariant « toute période écrite par l'application est bornée » remplace l'ancien test du membre à vie, sans devenir une tautologie. Épingler l'absence d'un cas d'énumération n'aurait rien prouvé et serait tombé à l'arrivée du régime rolling_year ; ce qui est fixé est la propriété qui doit lui survivre — la borne haute vaut la fin de saison, quelle que soit l'ancre. Le test de bout en bout écouteur → commande est retourné dans le même mouvement : il vérifie désormais que l'écouteur ne peut plus armer la pré-vérification. Sonde de falsifiabilité passée : réintroduire une période sans fin dans l'écouteur fait tomber 13 tests, dont ces deux gardes.

0.22.4.0 — 2026-08-18

Modifications

  • Les saisons du jeu de démonstration sont ancrées sur l'horloge, plus sur des millésimes figés. SeasonFactory produisait une saison tirée au hasard entre 1900 et 2090 : environ deux tirages sur trois tombaient sur une saison déjà close, et les cinq seeders de démonstration codaient en dur 2024-2025 / 2025-2026. Une inscription datée hors des bornes de sa saison fabrique, une fois le moteur d'adhésion branché, une période début = max(saison.début, validation) postérieure à fin = saison.fin — donc inversée. La fabrique expose désormais window() / windowFor() et les états explicites past() / current() / future(), et les seeders dérivent chaque borne, nom de campagne et date d'expiration de seasonWindow(-1) / seasonWindow(0). Sans ce préalable, la non-déterminisme se serait manifesté en CI aléatoire une fois l'écouteur d'adhésion actif partout.
  • createRegistration() exige une date d'inscription et un statut. Les deux paramètres passent en position 5 et 6 du helper partagé HasMemberSeedHelpers, familyPosition recule en 7 : tout appel resté à l'ancienne signature échoue bruyamment au lieu de glisser silencieusement une position de famille dans la date. assertRegisteredWithinSeason() refuse en outre toute inscription hors bornes — une borne codée en dur qui a vieilli casse le seed au lieu d'écrire une période inversée.
  • Les cinq seeders de démonstration suppriment puis recréent leur tenant. DemoLigue56Seeder, EpgvSeeder et WaterPoloSeeder conservaient à la place une branche « tenant existant → données préservées » qui ne rejouait qu'une fraction du jeu de données ; ils sont désormais rejouables comme AssoCultureSeeder et ClubAssoSeeder, et DemoResetSeeder les appelle tous les cinq.
  • Les settings d'un tenant sont écrits comme objet JSON. TenantFactory et les cinq seeders posaient [], qui sérialise en tableau JSON et casse toute requête settings->>'clé' ou jsonb_typeof. La valeur de démonstration est {"hello_asso_sandbox": true} : un jeu fabriqué ne doit jamais viser HelloAsso en production.
  • L'horodatage technique created_at d'une inscription suit sa date d'inscription. HasMemberSeedHelpers::createRegistration() alignait registered_at sur la saison mais laissait created_at à now() — or c'est created_at que lit memberships:backfill pour ancrer la période (BackfillMembershipsCommand, périodes « ancrées au passé »). Une inscription de saison révolue rejouait donc l'inversion max(début de saison, aujourd'hui) > fin de saison sans qu'aucune assertion portant sur registered_at ne bronche : sonde de falsifiabilité, 7 tests sur 8 tombent si l'alignement est retiré.

Ajouts

  • Garde de production sur les seeders de démonstration. demoSeedIsAllowed() refuse le cycle supprime/recrée quand app()->isProduction(). Les tenants demo-ligue-56, demo-epgv-44 et demo-cholet-waterpolo existent en production et portent des données de démonstration commerciale : rejouer leur seeder les détruirait. clevercloud/post_build.sh n'appelle que PermissionSeeder et RoleSeeder, mais un db:seed --class=… lancé à la main aurait suffi.
  • Quatre fichiers de tests de non-régression (tests/Feature/Database/) : déterminisme de SeasonFactory (la saison par défaut contient toujours le jour courant, sur 50 itérations et par décalage explicite), idempotence des cinq seeders (chacun rejoué deux fois, un seul tenant, settings objet JSON), et absence d'inscription hors bornes de saison — dont trois cas sous horloge déplacée (2026-09-15, 2027-03-01, 2028-08-30), sans quoi l'assertion serait tautologique tant que l'horloge réelle coïncide avec les millésimes autrefois codés en dur. S'y ajoutent le déterminisme de MembershipFactory (période par défaut figée, réutilisation d'une saison au libellé libre, shifted(), et deux refus d'adhésion en double dont un qui isole memberships_unique_season en faisant varier start_date à saison constante — sans quoi memberships_no_overlap (tenant_id, member_id, start_date) suffirait à faire passer le test) et la survie du cycle supprime/recrée en présence d'une adhésion.

Corrections

  • Le test de chevauchement partiel du listener ne dépend plus de l'horloge. CreateMembershipFromRegistrationListenerTest posait sa saison antérieure avec Season::factory()->past(). past() dérive son nom de start_date via startYearOf(), qui bascule de millésime le 1er septembre : à partir du 2026-09-01, past() produit « 2025-2026 » — le nom exact de la saison que validatedRegistration() crée sur le même tenant avec les bornes 2025-09-15 → 2026-06-30. La CI serait donc passée du vert au rouge sans qu'aucun commit ne soit poussé, sur une violation unique(tenant_id, name) dont la cause serait la date d'exécution. Les bornes de la saison antérieure sont désormais absolues (2024-09-15 → 2025-06-30) : le nom « 2024-2025 » est constant. La saison n'existe dans ce décor que pour porter un season_id — ses bornes ne sont lues par aucune assertion.
  • MembershipFactory ne tire plus sa période au hasard. Depuis que season_id est déduit de start_date, la période par défaut est devenue porteuse : un tirage sur 366 jours ne franchit qu'une seule fois la frontière du 1er septembre, si bien que deux memberships par défaut pour un même membre tombaient sur la même saison la plupart des jours de l'année — violant l'index unique partiel memberships_unique_season à un taux qui variait avec la date d'exécution, et virant au vert deux jours par an. La période par défaut est désormais la fenêtre de la saison courante : la collision est systématique et lisible, et un test qui a besoin d'une seconde saison écrit ->shifted(-1) (miroir de SeasonFactory::shifted()) ou ->period(...). La résolution de saison cherche en outre la saison qui contient la période et non celle qui porte le nom calculé — chercher par nom manquait une saison aux bonnes bornes mais au libellé libre et en fabriquait une seconde, identique.
  • Quatre décors de test ne créent plus deux saisons chevauchantes pour un même tenant. Rendre SeasonFactory déterministe donne à toutes les saisons par défaut d'un tenant des bornes identiques : RegistrationControllerTest, RegistrationExportTest (deux cas) et SeasonControllerTest produisaient un chevauchement que seasons_no_overlap (EXCLUDE USING gist) refuse en production, et que SQLite laisse passer sans un mot. Les saisons secondaires reçoivent des fenêtres explicitement disjointes (past() / current() / SeasonFactory::window(-1)). Ce n'est pas cosmétique : la pré-vérification de memberships:sync traitera des saisons chevauchantes comme un échec, et un décor chevauchant produit deux season_id distincts que l'index unique accepte alors que la contrainte GiST rejette (23P01 en production).
  • EpgvSeeder ne perd plus ses activités. addCoursesDemoFeatures() (cours Gym Douce et Pilates, campagne « Inscriptions Activités Vitafédé », 12 inscrits) n'était atteignable que par la branche « tenant existant » : le passage au cycle supprime/recrée l'aurait rendue morte, et le tenant serait reparti sans aucun cours, en silence. L'appel est câblé dans le chemin de création.

Notes techniques

  • memberships.season_id est en noActionOnDelete(), et le re-seed le supporte réellement. Le raisonnement d'origine — « NO ACTION est différé en fin d'instruction, donc les cascades sœurs tenants -> seasons et tenants -> memberships se résolvent » — est une propriété de PostgreSQL : SQLite vérifie NO ACTION immédiatement, et la divergence irait ici dans le sens inverse de l'habituel (rouge en CI, vert en production). L'idempotence prouvée jusqu'ici ne portait sur rien, memberships restant vide pendant le test faute de membership_renewal_type dans les settings de démonstration : l'écouteur d'adhésion y est inerte. Le test pose désormais explicitement une adhésion ancrée sur une saison du tenant avant de rejouer le seeder — la branche delete-recreate est exercée, et elle passe sur les deux moteurs.
  • Code mort retiré. addNewFeatures() et refreshDemoMeetings() disparaissent des trois seeders convertis : elles n'existaient que pour la branche « tenant existant ». createDemoMeetings() et addCoursesDemoFeatures() sont conservées et appelées depuis run().

0.22.1.0 — 2026-07-31

Ajouts

  • Codes de réduction par campagne. Un adhérent saisit un code dans le formulaire public (ou un agent en back-office) et obtient une remise en pourcentage (1-100 %) sur son panier. Chaque code est configuré par campagne : ciblage optionnel par rubrique, quota d'utilisations, date d'expiration (inclusive). Techniquement, un code est une PricingRule dont la colonne code est non-null — même table, même moteur de calcul que les remises automatiques ; Campaign::pricingRules() filtre désormais whereNull('code') pour que les dix appelants existants ne voient jamais un code. CRUD back-office sur la carte campagne (créer, modifier, archiver, supprimer), saisie côté funnel avec badge de confirmation et remise reflétée dans le total affiché. Doc : Docs/Features/Promo-Codes.md.
  • Consommation par panier, pas par compteur. La table promo_code_redemptions enregistre une ligne par panier ; deux index uniques partiels portent les règles métier (un seul code par panier, une seule consommation active par panier). Le quota est gardé deux fois — pré-filtre dans CartPricingRules::resolveUsableCode() et garde post-verrou dans RedeemPromoCodeService — et la ligne est libérée à l'abandon du panier. La remise survit aux mutations de cours : MutateRegistrationCoursesService et CartMutationService recalculent avec CartPricingRules::forCart(), pas avec la relation filtrée.
  • Validation immédiate d'un panier ramené à 0 €. Un panier intégralement couvert par un code ne passe plus par HelloAsso : il est validé sur-le-champ, le moyen de paiement est réinitialisé et le mail de confirmation part sans ligne de reste à payer. Le code de réduction appliqué est tracé dans le snapshot de facture et affiché sur le détail panier.

Sécurité

  • Oracle d'existence de codes fermé. L'autorisation vivait dans le contrôleur, alors que Laravel résout rules() et withValidator() avant lui : n'importe quel utilisateur authentifié du tenant pouvait distinguer « code inexistant » de « code existant mais interdit » via les messages de validation. L'autorisation est remontée dans StorePromoCodeRequest::authorize(). Côté public, la réponse d'application d'un code est uniformisée et un seau anti-énumération partagé limite les tentatives.
  • Fuite de quota fermée. Le décompte pouvait être contourné en concurrence ; la consommation se fait maintenant sous verrou, dans une transaction gardée, et le prédicat est testé des deux côtés (code utilisable / code épuisé).

Modifications

  • La suite de tests ne peut plus atteindre PostgreSQL. La connexion pgsql est dé-configurée pour les tests et tests/Unit/TestDatabaseIsolationTest.php monte la garde — il vérifie aussi qu'aucune trace d'un flag d'opt-in ne revient dans le dépôt. Motif : un tel flag avait vidé la base de développement. Les agrégats sensibles à la divergence SQLite↔PostgreSQL se vérifient désormais à la main sur la base de dev, jamais via la suite.

0.19.0.0 — 2026-07-03

Ajouts

  • Lien de paiement en ligne pour panier offline impayé : depuis le détail panier, un bouton du panneau de relance génère un lien de paiement HelloAsso pour un panier initialement réglé hors ligne (chèque, virement, espèces) qui reste impayé — sans obliger l'adhérent à repasser par tout le formulaire. Guardé par token public valide, solde panier > 0 relu sous verrou, et HelloAsso configuré pour le tenant (le bouton est masqué sinon).
  • Verrou de checkout HelloAsso anti double-clic/double-onglet : Cache::lock par panier (30s, attente 5s) sérialise les checkouts concurrents sur le même jeu d'inscriptions ; le solde est relu sous verrou juste avant chaque appel HelloAsso, avec dégradation gracieuse (HelloAssoAlreadySettledException) si un règlement concurrent a déjà tout soldé. Les appels HTTP HelloAsso sont bornés à 10s pour ne jamais dépasser la durée du verrou.

Corrections

  • Le montant réclamé par la relance et le checkout HelloAsso se base désormais sur le solde du panier (VersementService::cartOutstandingCents), plus sur la ventilation par inscription — élimine les relances erronées après une re-ventilation des paiements à l'annulation.
  • Élimination de plusieurs N+1 (registration.payments, registration.balance) dans HelloAssoRelanceService et le flux de retry HelloAsso.
  • Exclusion des paiements HelloAsso fantômes de la relance panier : payableCartPayments() (et canSendManualRelance(), createOnlinePaymentLink()) retournent vide dès que le panier n'a plus de solde dû au niveau versement (VersementService::cartOutstandingCents), même si un Payment Pending/Rejected résiduel survit sur une inscription réglée hors ligne entre-temps ; garde sur destinataire absent ; resynchronisation de Payment.amount au moment du checkout.