Skip to content

Include sealer name and seal date in summary PDF (support optional seal columns)#137

Merged
mapiolca merged 4 commits intomainfrom
2026-01-27-add-fk_user_seal-and-date_seal-columns-crqe67
Jan 27, 2026
Merged

Include sealer name and seal date in summary PDF (support optional seal columns)#137
mapiolca merged 4 commits intomainfrom
2026-01-27-add-fk_user_seal-and-date_seal-columns-crqe67

Conversation

@mapiolca
Copy link
Copy Markdown
Owner

Motivation

  • Permettre à la synthèse PDF générée par la massaction d’afficher le nom du scelleur et la date de scellement lorsque les colonnes optionnelles fk_user_seal et date_seal existent, afin d’améliorer l’audit et la lisibilité administrative.
  • Assurer une compatibilité descendante en détectant dynamiquement la présence des colonnes et en n’ajoutant l’information que si disponible.
  • Aucun skill spécial utilisé pour cette modification (pas de skill-creator ni skill-installer).

Description

  • Ajout du paramètre sealedOn et affichage du libellé « scellée le … » dans la cellule de statut du PDF via la fonction tw_pdf_compose_status_cell dans lib/timesheetweek_pdf.lib.php.
  • Extension de la collecte de données dans tw_collect_summary_data (dans lib/timesheetweek_pdf.lib.php) pour détecter la présence de fk_user_seal/date_seal, étendre la requête SQL et remplir les champs sealed_by et sealed_on quand disponibles.
  • Persistation des métadonnées de scellement dans la massaction sceller de timesheetweek_list.php en détectant une seule fois la présence des colonnes et en exécutant un UPDATE complémentaire sans bloquer le scellement si la mise à jour échoue (log en warning).
  • Mise à jour de la méthode TimesheetWeek::seal() dans class/timesheetweek.class.php pour détecter dynamiquement les colonnes optionnelles et écrire fk_user_seal/date_seal lors du scellement, en mettant aussi à jour les propriétés de l’objet.
  • Ajout des colonnes et index correspondants dans le schéma sql/llx_timesheet_week.sql et du script d’évolution sql/update_all.sql pour ajouter fk_user_seal, date_seal et leurs index si manquants.
  • Ajout de la chaîne de traduction pour le libellé de date (TimesheetWeekSummaryStatusSealedOn) dans les fichiers langs/*/timesheetweek.lang, incrémentation de la version du module à 1.7.5 dans core/modules/modTimesheetWeek.class.php et mise à jour de ChangeLog.md et README.md.

Testing

  • Aucun test automatisé exécuté pour ces changements (aucune suite CI/automatisée lancée).

Codex Task

@mapiolca mapiolca merged commit 8bf0b2b into main Jan 27, 2026
1 check passed
@mapiolca mapiolca deleted the 2026-01-27-add-fk_user_seal-and-date_seal-columns-crqe67 branch January 27, 2026 21:53
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