Skip to content

QUAL: corrige avertissements Phan (nullable / variables non déclarées) dans massactions tâches#41

Merged
mapiolca merged 1 commit into2026-03-25-add-mass-actions-for-external-module-tasksfrom
2026-03-25-add-notices-for-php-issues-in-pr
Mar 25, 2026
Merged

QUAL: corrige avertissements Phan (nullable / variables non déclarées) dans massactions tâches#41
mapiolca merged 1 commit into2026-03-25-add-mass-actions-for-external-module-tasksfrom
2026-03-25-add-notices-for-php-issues-in-pr

Conversation

@mapiolca
Copy link
Copy Markdown
Owner

Motivation

  • Corriger plusieurs alertes de l'analyseur statique Phan liées à des arguments nullable ou des variables potentiellement non déclarées dans le flux de mass actions des tâches (htdocs/projet/tasks.php et htdocs/core/tpl/massactions_pre.tpl.php).

Description

  • Dans htdocs/projet/tasks.php (autour de L1068) on force l'argument 7 passé à load_fiche_titre() à être une string non-nullable via cast/coalescence: (string) ($massactionbutton ?? '').
  • Dans htdocs/core/tpl/massactions_pre.tpl.php on remplace le isset() ambigu par une vérification d'objet robuste pour $taskstatic : if (!is_object($taskstatic ?? null)) { $taskstatic = null; }.
  • Dans le même template on initialise explicitement $trackid et $modelmail en string via $trackid = (string) ($trackid ?? ''); et $modelmail = (string) ($modelmail ?? ''); pour éviter les variables non déclarées et assurer la compatibilité de type.
  • Les changements sont de nature strictement sûre-typage / initialisation et n'altèrent pas le comportement fonctionnel attendu.

Testing

  • Vérification de syntaxe PHP exécutée avec php -l htdocs/projet/tasks.php et php -l htdocs/core/tpl/massactions_pre.tpl.php, les deux commandes ont retourné "No syntax errors detected".

Codex Task

@mapiolca mapiolca merged commit b949081 into 2026-03-25-add-mass-actions-for-external-module-tasks Mar 25, 2026
4 checks passed
@mapiolca mapiolca deleted the 2026-03-25-add-notices-for-php-issues-in-pr branch March 25, 2026 23:32
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