Recherche avancée

Sur d’autres sites (8618)

• Anomalie #3332 (Fermé) : Message d’erreur erroné : le nombre d’erreur n’est pas le bon

  10 mai 2015, par cedric -

  Appliqué par commit r22128.

• Anomalie #4717 : Erreurs nombre d’argument des filtres

  10 avril 2021, par jluc -

  La PR récupère l’erreur PHP, évite le crash blank et délivre le message d’erreur PHP dans la jolie box d’erreur SPIP. Le pire est évité, et donc il faut intégrer cette PR qui fait du bien.

  Mais l’internaute SPIPien se retrouve avec un message d’erreur en anglais et portant sur le code compilé PHP. Ce code compilé, même le commun des devs PHP ne le connait pas, car à part XRay qui le rend accessible et les core-développeurs qui débuguent le code d’une nouvelle structure SPIP, personne ne va scruter et n’a connaissance du code php généré par le compilateur.

  Pour faire mieux, il y a la piste de la déclaration des arités des filtres. $arite_des_filtres = [[’implode’, 2, 2], [’affdate’, 1, 2]...]. Cette déclaration peut probablement être utilisée au moment de la compilation d’un appel de filtre lorsque le compilateur convertit la structure SPIP #XXX|filtre... en appel PHP.
  Ça permet de détecter une erreur d’argumentcount AVANT l’appel PHP et de fournir un joli message d’erreur SPIP qui fait référence au source SPIP.

  Dans ce cas, la détection d’erreur peut se faire à la compilation.

  Mais il se peut que d’autres erreurs PHP doivent être gérées, avec la strictisation croissante de PHP. Il faudra donc de nouvelles déclarations permettant la détection préventive des erreurs.

  Peut être sur d’autres structures de données que les filtres aussi ?

  Il faut donc développer la capacité d’introspection de SPIP.

  Mais parfois l’erreur n’est pas détectable au moment de la compilation. J’ai évoqué plus haut l’erreur relative au typage des arguments. S’il se confirme que le pb se pose aussi, et vu qu’on peut passer à un filtre un argument calculé dynamiquement, c’est pas lors de la compilation que ça pourra être détecté, mais lors de l’exécution du code compilé.

  Il faut alors prendre chaque erreur PHP possible et voir si/comment le compilateur peut la prévenir.

  Une alternative serait de "traduire" cette erreur PHP de manière à aiguiller le webmestre SPIP.

  Par exemple le message d’erreur de https://www.mail-archive.com/spip@rezo.net/msg81110.html , qui fait référence au source compilé, peut être traduit, par reconnaissance (regexp), analyse du message d’erreur, et calcul du message d’erreur SPIP en « Erreur : dans votre source SPIP, il y un appel au filtre |implode qui n’a pas le bon nombre d’arguments. Ça se passe dans la boucle ’BOUCLE_contenu_article’ : veuillez examiner le code et corriger. » Là en plus, ce serait bien de pouvoir associer le nom du fichier source à la boucle dont on connait le nom. Une capacité d’introspection déjà présente ? ou à développer ?

• Anomalie #4717 : Erreurs nombre d’argument des filtres

  8 avril 2021

  Refs :

  - https://www.php.net/manual/fr/migration71.incompatible.php
  - https://www.php.net/manual/fr/function.implode.php
  - https://www.spip.net/fr_article5672.html

  Au aucun moment il n’est indiqué que le filtre |implode de SPIP puisse être appelé sans second paramètre, donc jusque là pas de problème : la personne a mal écrit son squelette.

  Ceci étant dit, il semble que PHP analyse le problème même sans exécuter la fonction, et que ce Fatal pourrait être catch là https://git.spip.net/spip/spip/src/branch/master/ecrire/public/composer.php#L106 en plus, et retourner à la place une erreur_squelettes(), comme ça le fait pour le ParseError.