calculer_langues_utilisees() n'interpréte pas correctement $boucle->where #3101

Closed
opened 9 years ago by maieul · 5 comments
maieul commented 9 years ago
Collaborator

Dans ecrire/inc/rubriques, la fonction calculees_langues_utilisees() n'interprète pas le code de $boucle->where avant de le passer à la requete de selection.

Conséquent : une plugin qui a dans la pipeline pre_boucle mettrait un sql_in dans le $boucle->where ferait plante msysql, puisque celui ci se retrouverait avec un sql_in(xxx) comme commande, ce qu'il ignore.

Cf http://zone.spip.org/trac/spip-zone/changeset/78657 (test avant et après le commit pour voir le problème, lié à exclurepip.php l. 18, qui fonctionne normalement pour les vraies boucles en squelettes, puisque que le $boucle->where est correctement interprété avant d'être envoyé à mysql)

Dans ecrire/inc/rubriques, la fonction calculees_langues_utilisees() n'interprète pas le code de $boucle->where avant de le passer à la requete de selection. Conséquent : une plugin qui a dans la pipeline pre_boucle mettrait un sql_in dans le $boucle->where ferait plante msysql, puisque celui ci se retrouverait avec un sql_in(xxx) comme commande, ce qu'il ignore. Cf http://zone.spip.org/trac/spip-zone/changeset/78657 (test avant et après le commit pour voir le problème, lié à exclurepip.php l. 18, qui fonctionne normalement pour les vraies boucles en squelettes, puisque que le $boucle->where est correctement interprété avant d'être envoyé à mysql)
Poster
Collaborator

testé sous SPIP 3.'0.12

testé sous SPIP 3.'0.12
Owner

Assigné à cedric
Version cible mise à 3.1

**Assigné à cedric** **Version cible mise à 3.1**
Owner

Appliqué par commit r21699.
Statut changé à Fermé

Appliqué par commit r21699. **Statut changé à Fermé**
Owner

Attention cepeendant, je pense que tu devras toujours exclure cette boucle de ton pipelin sinon tu vas fausser le comptage des langues...

Attention cepeendant, je pense que tu devras toujours exclure cette boucle de ton pipelin sinon tu vas fausser le comptage des langues...
Owner
There is no content yet.
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.