[fix] Régression v3.0.8 — Invalid call dans Distributions / ContractAdmin#154
Merged
Conversation
…signature Le commit 049915f avait ajouté le commentaire et migré les appels Cron.hx mais avait omis de supprimer le ?year:Bool de la signature elle-même. View.hDate reste donc une fonction Neko à 2 arguments — les templates l'appelant avec 1 argument produisent toujours "Invalid call". Ce commit supprime le paramètre optionnel pour de bon. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
InterAMAP44
approved these changes
Jun 28, 2026
Mandrak-Kimigo
added a commit
that referenced
this pull request
Jun 28, 2026
) * fix: View.hDate — retirer effectivement le paramètre optionnel de la signature (#154) Le commit 049915f avait ajouté le commentaire et migré les appels Cron.hx mais avait omis de supprimer le ?year:Bool de la signature elle-même. View.hDate reste donc une fonction Neko à 2 arguments — les templates l'appelant avec 1 argument produisent toujours "Invalid call". Ce commit supprime le paramètre optionnel pour de bon. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Mandrak-Kimigo
added a commit
that referenced
this pull request
Jul 12, 2026
* [mantis-327] Notifications : ajouter l'année dans l'objet des emails Les objets des mails de notification (distribution, besoin de volontaires, instructions bénévoles, abandon de rôle) n'affichaient pas l'année, ce qui rendait les dates ambiguës sur plusieurs saisons. Formatting.hDate() supportait déjà un paramètre optionnel `year`. Ce commit propage ce paramètre dans View.hDate() et passe `true` aux 5 appels concernant les objets de mails. * [mantis-340] Féminisation des métiers dans la liste des producteurs Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> * add fix-* workflow * fix: View.hDate — supprimer le paramètre optionnel ?year incompatible Templo Templo ne supporte pas les paramètres optionnels dans les appels de fonctions (cf. commentaire BaseView : "templo doesnt manage optional params in functions"). Le commit bdbbd8d avait ajouté ?year:Bool=false à View.hDate, ce qui compilait la fonction Neko en 2 arguments. Les templates l'appelant avec 1 argument déclenchaient une "Invalid call" sur les pages Distributions et ContractAdmin/selectDistrib. Solution : View.hDate revient à 1 argument ; les appels avec year=true (Cron.hx, VolunteerService.hx) passent directement par Formatting.hDate. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: corriger l'erreur "Invalid operation (*)" dans orders.mtt Le template contractadmin/orders.mtt calculait `o.quantity * o.productQt` pour l'input de pesée (bloc multiWeight + variablePrice) sans vérifier la nullité de `productQt` (champ SNull<SFloat> dans Product). Quand un produit a multiWeight=true mais qt=null, Neko lançait "Invalid operation (*)". On introduit une variable locale `effectiveQt` qui vaut `o.productQt` si non-null, sinon 1, aligné sur le comportement de OrderService ligne 872. L'opérateur ternaire n'étant pas supporté par le compilateur Templo, on utilise deux instructions ::set:: / ::if::. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * ci: simplify GHCR publish workflow to amd64-only Remove multi-arch build attempt (linux/arm64) and fallback logic. Drop setup-qemu-action, replace the two build steps with a single unconditional linux/amd64 build. * fix: View.hDate — retirer effectivement le paramètre optionnel de la signature (#154) Le commit 049915f avait ajouté le commentaire et migré les appels Cron.hx mais avait omis de supprimer le ?year:Bool de la signature elle-même. View.hDate reste donc une fonction Neko à 2 arguments — les templates l'appelant avec 1 argument produisent toujours "Invalid call". Ce commit supprime le paramètre optionnel pour de bon. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> * [mantis-330/331] Refonte de la désinscription de permanence (#156) * [mantis-330] Remplacer la page de désinscription de permanence par un endpoint REST La page /distribution/unsubscribeFromRole (formulaire avec textarea obligatoire) est remplacée par un endpoint REST POST /api/distributions/unsubscribeFromRole qui effectue la désinscription sans exiger de motif. L'URL legacy redirige désormais vers /distribution/volunteersCalendar. La désinscription est désormais gérée côté React dans le NeoModule volunteersCalendar. * [mantis-331] Ne pas envoyer de mail de désinscription si le rappel des rôles vacants n'a pas encore été envoyé Avant : removeUserFromRole envoyait toujours un mail (aux coordinateurs avant la date d'alerte, à tous les membres après). Après : le mail n'est envoyé que si la date d'alerte est dépassée (c.-à-d. que le cron "rôles vacants" a déjà été déclenché). Avant cette date, le cron prochain couvrira la vacance — un double envoi est inutile. Le paramètre `reason` est supprimé de removeUserFromRole (mantis-330). Le template volunteerUnsuscribed.mtt est mis à jour : la mention de la raison est retirée. * [mantis-331] Corriger la traduction manquante dans le mail de désinscription La chaîne "has left the role" n'avait pas de traduction française suite au changement de libellé (suppression de la mention de raison). Mise à jour de texts_fr.po et allTexts.pot avec le nouveau msgid et sa traduction : "s'est désisté du rôle". * pas d'envoie au souscripteur si la commande est à 0 à la souscription * Camap v3.0.9 --------- Co-authored-by: fbiguet <143797468+fbiguet@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com> Co-authored-by: Renaud Bardet <renaud.bardet@lonestone.io>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
La v3.0.8 a introduit une régression sur plusieurs pages (Distributions,
ContractAdmin/selectDistrib, ContractAdmin/distributions) avec l'erreur
Invalid calldéclenchée par des appels à::hDate(d.date)::dansles templates Templo.
Cause
Le commit bdbbd8d a ajouté
?year:Bool = falseàView.hDate. Nekocompile cette fonction à 2 arguments ; Templo l'appelle avec 1 → crash.
(cf.
BaseView.hx: "templo doesnt manage optional params in functions")Commits inclus
049915f— commentaire + migration Cron.hx / VolunteerService.hxd8300af— fixInvalid operation (*)dans orders.mtt?yeardans la signature deView.hDateTest
Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com