Skip to content

fix_divers

Maïeul a demandé de fusionner gh-0536ee7e/208/unknown/refs/pull/208/head vers master

Hop @rastapopoulos ceci est une série de fix de micro warning en PHP 8 suite au message https://contrib.spip.net/Manuel-de-redaction-du-site#comment511155

Cela étant le plus étonnant est d'avoir eu ces erreurs. C'est parce que

  1. Manuel de rédaction du site utilise les saisies en déclaration PHP automagique
  2. Mais n'utilise pas YAML
  3. Or saisies_generer_html() utilise les informations du YAML pour supprimer du contexte certaines valeurs potentiellement en conflit avec les options, cf https://git.spip.net/spip-contrib-extensions/saisies/src/branch/master/inc/saisies_afficher.php#L171

et là cela me laisse un peu dubitatif ; je trouve étrange / problématique que la fonction qui genere le code HTML d'une saisies puisse potentiellement ne pas renvoyer le même contexte selon que le plugin YAML soit présent et/ou qu'il y ait un descripteur de la saisie en .YAML. Cela pose 2 problèmes :

  • pb de consistance / cohérence / reproductibilité
  • le décodage des .yaml c'est pas ce qu'il y a de plus rapide (même si bon, j'avais mis un static pour améliorer les perfs).

Un avis ?

Rapports de requête de fusion

Chargement en cours