Skip to content

refactor: retire l'usage de easter_date dans EverblockTools#965

Merged
CySSoO merged 1 commit intomasterfrom
cyssoo/retire-easter_date-from-module
Feb 25, 2026
Merged

refactor: retire l'usage de easter_date dans EverblockTools#965
CySSoO merged 1 commit intomasterfrom
cyssoo/retire-easter_date-from-module

Conversation

@CySSoO
Copy link
Contributor

@CySSoO CySSoO commented Feb 25, 2026

Motivation

  • Éliminer la dépendance à la fonction native \easter_date() et obtenir directement une date au format Y-m-d pour calculer les jours mobiles.
  • Rendre le calcul de la date de Pâques indépendant de la valeur timestamp et plus prévisible/portable dans le module.

Description

  • Remplacement de l'appel \easter_date($year) par self::getEasterDate((int) $year) dans getFrenchHolidays().
  • Adaptation des jours fériés mobiles pour partir de la date Y-m-d retournée par getEasterDate() et dériver Ascension/Pentecôte/Lundi de Pentecôte via strtotime (e.g. strtotime($easterDate . ' +39 days')).
  • Ajout de la méthode protected static function getEasterDate(int $year): string qui calcule la date de Pâques selon l'algorithme grégorien (Meeus/Jones/Butcher) et renvoie une chaîne formatée Y-m-d.

Testing

  • php -l src/Service/EverblockTools.php a été exécuté et n'a retourné aucune erreur de syntaxe.
  • Recherche résiduelle de easter_date( dans src/Service/EverblockTools.php a été effectuée avec rg et n'a renvoyé aucun usage trouvé.

Codex Task

@CySSoO CySSoO merged commit ec649db into master Feb 25, 2026
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant