Warning lors de la pagination ajax #17

Closed
opened 12 months ago by jeanmarie · 19 comments
Collaborator

J'ai un warning lorsque je pagine en ajax : Warning: Invalid argument supplied for foreach() in C:\www\toto\plugins\auto\tarteaucitron\v1.2.8\tarteaucitron_pipelines.php on line 50

Pour reproduire:

  • afficher une page où il y a une pagination ajax (par ex la liste des articles d'une rubrique)
  • afficher les articles suivants avec la pagination

Le warning s'affiche en bas du bloc ajaxé.

J'ai un warning lorsque je pagine en ajax : ` Warning: Invalid argument supplied for foreach() in C:\www\toto\plugins\auto\tarteaucitron\v1.2.8\tarteaucitron_pipelines.php on line 50` Pour reproduire: * afficher une page où il y a une pagination ajax (par ex la liste des articles d'une rubrique) * afficher les articles suivants avec la pagination Le warning s'affiche en bas du bloc ajaxé.
Collaborator

Bonjour,

Que tu retroune $GLOBALS['contexte'] avant le foreach par exemple ?

Bonjour, Que tu retroune `$GLOBALS['contexte']` avant le foreach par exemple ?
Collaborator

Je confirme le problème.
image

$GLOBALS['contexte'] me retourne NULL.

Je confirme le problème. ![image](/attachments/0f0d504e-d61e-4075-a8a1-0f8f80f4ee70) `$GLOBALS['contexte']` me retourne `NULL`.
Collaborator

Je peux avoir des urls pour voir le problème ?

Je peux avoir des urls pour voir le problème ?
Poster
Collaborator

Sur cette page : https://www.lemanege.com/-Agenda-.html
Mais j'ai désactivé les warnings depuis car le site est en prod.

Sur cette page : https://www.lemanege.com/-Agenda-.html Mais j'ai désactivé les warnings depuis car le site est en prod.
Collaborator

ok et sans la pagination cela fonctionne ?

ok et sans la pagination cela fonctionne ?
Poster
Collaborator

Oui, le warning apparait uniquement au chargement du contenu en ajax et s'affiche en bas du bloc ajaxé.

Testé en SPIP4 / PHP 7.4.20

Oui, le warning apparait uniquement au chargement du contenu en ajax et s'affiche en bas du bloc ajaxé. Testé en SPIP4 / PHP 7.4.20
Collaborator

Perso, j'ai testé sur un site que j'ai en dev. Au chargement de la page, je n'ai pas de warning. Seulement lorsque je vais sur une des pages de la pagination. Mais bizarrement la page n°2 ne m'affiche pas de warning.

Perso, j'ai testé sur un site que j'ai en dev. Au chargement de la page, je n'ai pas de warning. Seulement lorsque je vais sur une des pages de la pagination. Mais bizarrement la page n°2 ne m'affiche pas de warning.
Collaborator

donc on perds le contexte mais je ne sais pas comment le remplacer et j'ai pas de site en 4.0

donc on perds le contexte mais je ne sais pas comment le remplacer et j'ai pas de site en 4.0
Poster
Collaborator

Il n'y a pas un moyen de ne pas relancer le script au rechargement ajax ? (je tente :) )
A priori, quand c'est en ajax, il ne faut pas déclencher l'affichage du bandeau.

Il n'y a pas un moyen de ne pas relancer le script au rechargement ajax ? (je tente :) ) A priori, quand c'est en ajax, il ne faut pas déclencher l'affichage du bandeau.
Collaborator

Ah si. J'ai bien un warning sur la page n°2 (donc sur toutes les pages de la paginaion). C'est juste qu'il était caché sous une DIV. Ta piste est bonne @jeanmarie ? Il faudrait exclure aussi les flux RSS (cf #16).

Ah si. J'ai bien un warning sur la page n°2 (donc sur toutes les pages de la paginaion). C'est juste qu'il était caché sous une DIV. Ta piste est bonne @jeanmarie ? Il faudrait exclure aussi les flux RSS (cf #16).
Poster
Collaborator

Avec if ($GLOBALS['html'] and !test_espace_prive() and tarteaucitron_actif()) { (et non !$GLOBALS['html']) ici https://git.spip.net/spip-contrib-extensions/tarteaucitron/src/branch/master/tarteaucitron_pipelines.php#L49 ça semble résoudre le problème.

Avec `if ($GLOBALS['html'] and !test_espace_prive() and tarteaucitron_actif()) {` (et non `!$GLOBALS['html']`) ici https://git.spip.net/spip-contrib-extensions/tarteaucitron/src/branch/master/tarteaucitron_pipelines.php#L49 ça semble résoudre le problème.
Collaborator

En effet, je n'ai plus de warning :)

En effet, je n'ai plus de warning :)
paidge closed this issue 11 months ago
Collaborator

ok mais est ce que l'on perd des services sur la page en question ?

ok mais est ce que l'on perd des services sur la page en question ?
Poster
Collaborator

ok mais est ce que l'on perd des services sur la page en question ?

Tout semble fonctionner, voir par là : https://www.lemanege.com/-Agenda-.html

> ok mais est ce que l'on perd des services sur la page en question ? Tout semble fonctionner, voir par là : https://www.lemanege.com/-Agenda-.html
Poster
Collaborator

Si c'est ok pour tout le monde, on distribue (up de z et tag v1.2.9) ?

Si c'est ok pour tout le monde, on distribue (up de z et tag v1.2.9) ?
Collaborator

Est-ce que vous pensez que la branche que j'ai avancée (dev/services_video_modeles) peut être mergée avant de distribuer ?

Est-ce que vous pensez que [la branche que j'ai avancée (dev/services_video_modeles)](https://git.spip.net/spip-contrib-extensions/tarteaucitron/src/branch/dev/services_video_modeles) peut être mergée avant de distribuer ?
Poster
Collaborator

@paidge je t'avoue que j'ai suivi de très loin les échanges sur la PR et sur discuter, c'est sans doute mieux de valider cette PR dans ces discussions plutôt qu'ici :)

Et je me dis que c'est une correction de bug (donc up de z) alors que ta PR apporte de nouvelles foncitonnalités (donc up de y), il vaut mieux ne pas les mélanger.

Qu'en dis-tu ?

Les liens de discussion pour le PR :

@paidge je t'avoue que j'ai suivi de très loin les échanges sur la PR et sur discuter, c'est sans doute mieux de valider cette PR dans ces discussions plutôt qu'ici :) Et je me dis que c'est une correction de bug (donc up de z) alors que ta PR apporte de nouvelles foncitonnalités (donc up de y), il vaut mieux ne pas les mélanger. Qu'en dis-tu ? Les liens de discussion pour le PR : * https://discuter.spip.net/t/re-spip-zone-commit-tarteaucitron-services-video-mise-en-place-des-services-pour-le-plugins-oembed-avec/19838/13 * https://git.spip.net/spip-contrib-extensions/tarteaucitron/pulls/10
Collaborator

Oui cela peut se faire sûrement en deux temps.

Oui cela peut se faire sûrement en deux temps.
jeanmarie referenced this issue from a commit 11 months ago
Poster
Collaborator

C'est fait.

C'est fait.
Sign in to join this conversation.
No Milestone
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.