Un warning PHP en moins styliser_par_z.php #3354

Closed
opened 8 years ago by teddy.spip · 4 comments

Pour éviter des warnings PHP "Undefined index". Voici un patch en SPIP 3.0 /ecrire/public/styliser_par_z.php

Pour éviter des warnings PHP "Undefined index". Voici un patch en SPIP 3.0 /ecrire/public/styliser_par_z.php
b_b commented 8 years ago
Owner

Hop, avant d'intégrer, peux-tu me dire quel environnement utiliser pour reproduire "le bug" ?

Hop, avant d'intégrer, peux-tu me dire quel environnement utiliser pour reproduire "le bug" ?
Poster

Alors... Je me demandes en fait si le soucis n'est pas ailleurs. Le warning apparait si tu utilises zcore qui forcément utilise styliser_par_z.php pour ses besoins.

Il faudrait donc voir selon 2 pistes :

  • c'est zcore qui ne fait pas correctement les choses, ne donnerait pas les bonnes infos ;
  • c'est ecrire/public/styliser_par_z.php qui ne prend pas tous les cas de figure alors qu'il est "sensé le faire".
Alors... Je me demandes en fait si le soucis n'est pas ailleurs. Le warning apparait si tu utilises zcore qui forcément utilise styliser_par_z.php pour ses besoins. Il faudrait donc voir selon 2 pistes : - c'est zcore qui ne fait pas correctement les choses, ne donnerait pas les bonnes infos ; - c'est ecrire/public/styliser_par_z.php qui ne prend pas tous les cas de figure alors qu'il est "sensé le faire".
b_b commented 8 years ago
Owner

Si cela se produit lors de la présence du plugin zcore, je dirais bien que c'est au plugin en question de faire que l'erreur ne soit pas produite.

Si cela se produit lors de la présence du plugin zcore, je dirais bien que c'est au plugin en question de faire que l'erreur ne soit pas produite.
Owner

Le patch n'est pas exact, car sur le dernier test il ne faut pas se contenter dans !isset().
Ça a déjà été intégré en 3.1, on ne backporte pas
Version cible mise à 3.1
Statut changé à Fermé

Le patch n'est pas exact, car sur le dernier test il ne faut pas se contenter dans !isset(). Ça a déjà été intégré en 3.1, on ne backporte pas **Version cible mise à 3.1** **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.