perf(calendar_event): calcul des occurrences limité à la plage demandée

[AntTBD]

Description

calculOccurrence() itérait depuis la 1ère occurrence de l'évènement, quelle que soit la plage demandée : des milliers de tours de boucle inutiles à chaque rendu de widget pour un agenda ancien.

La boucle démarre maintenant directement au début de la plage, en avançant d'un multiple entier de la fréquence de répétition — la grille des occurrences reste donc inchangée. Limité aux unités de durée fixe (minutes, heures, jours, semaines), l'arithmétique PHP sur les mois n'étant pas associative en fin de mois.

Deux correctifs adjacents, dans des commits séparés :

• getNationalDay() recevait un argument de trop et l'année courante au lieu de celle de l'occurrence : les options "jours fériés" étaient fausses au-delà de l'année suivante. Résultat mis en cache par année au passage.
• Plafond d'itérations sur la boucle + isset() manquants (warnings PHP 8).

Comment tester. Sur beta, exporter toutes les occurrences et le temps de calcul :

mkdir -p /var/www/html/tmp && sudo -u www-data php -d memory_limit=512M -r 'require_once "/var/www/html/core/php/core.inc.php"; $s=(new DateTime("-6 months"))->format("Y-m-d H:i:s"); $f=(new DateTime("+6 months"))->format("Y-m-d H:i:s"); $o=""; $n=0; $b=microtime(true); foreach(calendar_event::all() as $e) foreach($e->calculOccurrence($s,$f) as $c){ $o.=$e->getId()."|".$c["start"]."|".$c["end"]."\n"; $n++; } $ms=(microtime(true)-$b)*1000; file_put_contents("/var/www/html/tmp/occ_avant.txt", $o); printf("AVANT : %d occurrences en %.1f ms\n", $n, $ms);'

Puis avec cette PR appliquée, la même commande vers occ_apres.txt :

mkdir -p /var/www/html/tmp && sudo -u www-data php -d memory_limit=512M -r 'require_once "/var/www/html/core/php/core.inc.php"; $s=(new DateTime("-6 months"))->format("Y-m-d H:i:s"); $f=(new DateTime("+6 months"))->format("Y-m-d H:i:s"); $o=""; $n=0; $b=microtime(true); foreach(calendar_event::all() as $e) foreach($e->calculOccurrence($s,$f) as $c){ $o.=$e->getId()."|".$c["start"]."|".$c["end"]."\n"; $n++; } $ms=(microtime(true)-$b)*1000; file_put_contents("/var/www/html/tmp/occ_apres.txt", $o); printf("APRES : %d occurrences en %.1f ms\n", $n, $ms);'

Comparer :

diff /var/www/html/tmp/occ_avant.txt /var/www/html/tmp/occ_apres.txt && echo "IDENTIQUE"

IDENTIQUE attendu, hors évènements "jours fériés" au-delà de l'année suivante (2ème commit). Le gain est d'autant plus net que la plage est courte : sur ±7 jours, celle du rendu de widget, remplacer -6 months/+6 months par -7 days/+7 days.

Suggested changelog entry

EN: Occurrences are now calculated only over the requested date range
FR: Les occurrences sont désormais calculées uniquement sur la plage de dates demandée

Related issues/external references

Fait suite à #53, réécrite sur la version actuelle de beta.

Types of changes

• Bug fix (non-breaking change which fixes)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
  • This change is only breaking for integrators, not for external standards or end-users.
• Documentation improvement

PR checklist

• I have checked there is no other PR open for the same change.
• I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
• I grant the project the right to include and distribute the code under the GNU.
• I have added tests to cover my changes.
• I have verified that the code complies with the projects coding standards.
• [Required for new sniffs] I have added MD documentation for the sniff.