Ne pas génerer un bloc HTML de stats sur le hit, quand on génère autre chose que du HTML #4784

Open
opened 1 year ago by rastapopoulos · 1 comments
Owner

Le plugin génère en permanence un petit bloc HTML de stats :

<div class="stats_hit" onclick="this.style.display = 'none';">1.9 Mo | 49 requetes | 0.48 s</div>

Sauf qu'il le génère… même si on est en train de renvoyer autre chose que du HTML ! Ce qui fait que ça ajoute ce code à la fin d'un CSV, ou d'un JSON de GIS, etc. Et donc ça pète la fonctionnalité.

Le plugin génère en permanence un petit bloc HTML de stats : ``` html <div class="stats_hit" onclick="this.style.display = 'none';">1.9 Mo | 49 requetes | 0.48 s</div> ``` Sauf qu'il le génère… même si on est en train de renvoyer autre chose que du HTML ! Ce qui fait que ça ajoute ce code à la fin d'un CSV, ou d'un JSON de GIS, etc. Et donc ça pète la fonctionnalité.
Poster
Owner

@b_b pointe cette ligne :
https://git.spip.net/spip/dev/src/branch/master/dev_options.php#L18

Et effectivement c'est beaucoup trop basique, ya des centaines d'autres cas où faut pas le générer.

Vu qu'on est à la toute fin du hit avec le shutdown, on doit pouvoir tester où en est le dernier Content-Type déclaré à PHP non ? Et ne faire le travail QUE si c'est du HTML ou si ya rien d'indiqué (donc HTML par défaut).

@b_b pointe cette ligne : https://git.spip.net/spip/dev/src/branch/master/dev_options.php#L18 Et effectivement c'est beaucoup trop basique, ya des centaines d'autres cas où faut pas le générer. Vu qu'on est à la toute fin du hit avec le shutdown, on doit pouvoir tester où en est le dernier Content-Type déclaré à PHP non ? Et ne faire le travail QUE si c'est du HTML ou si ya rien d'indiqué (donc HTML par défaut).
b_b added the
bug
label 3 months ago
b_b added this to the spip-4.1 milestone 3 months ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.