spip
/
spip
32
14
Fork 13
Code Issues 430 Pull Requests 7 Projects Releases Wiki Activity
Labels Milestones
New Issue

Différence entre critères {id_xx} et {id_xx?} avec id_xx un tableau #4014

Closed
opened 5 years ago by marcimat · 2 comments
marcimat commented 5 years ago
Owner

Soit un squelette de test appelé avec ?page=test&id_mot[]=1&id_mot[]=2&id_mot[]=3&id_mot[]=4&id_mot[]=5&id_mot[]=6
et 2 boucles avec pour seule différence le ?

#ID_MOT
#ID_MOT

Dans le premier cas la requête SQL crée un IN avec les différents identifiants.
Dans le second cas la requête SQL générée est en erreur avec un where : (mots.id_mot = (2 1 (3) : 4))
(notons que ça s’arrête au 4è élément du tableau)

Quel comportement devrait-il y avoir ?

Soit un squelette de test appelé avec `?page=test&id_mot[]=1&id_mot[]=2&id_mot[]=3&id_mot[]=4&id_mot[]=5&id_mot[]=6` et 2 boucles avec pour seule différence le `?` <pre> <BOUCLE_1(MOTS){id_mot?}>#ID_MOT</BOUCLE_1> <BOUCLE_2(MOTS){id_mot}>#ID_MOT</BOUCLE_2> </pre> Dans le premier cas la requête SQL crée un IN avec les différents identifiants. Dans le second cas la requête SQL générée est en erreur avec un where : `(mots.id_mot = (2 1 (3) : 4))` (notons que ça s’arrête au 4è élément du tableau) Quel comportement devrait-il y avoir ?
cerdic commented 4 years ago
Owner

J'ai tendance à penser que le {id_mot?} a un comportement dérogatoire acceptable puisqu'il implique qu'on ne connait pas le type de la donnée à la compilation et donc il est normal qu'il s'adapte

Si on modifie {id_mot} de la même façon cela implique donc une indétermination à la compilation qui ralentira toutes les requetes, et surtout ça ouvre ensuite la question de pourquoi {id_mot=#ENV{id_mot}} ne fonctionne pas pareil ?
Statut changé à En cours

J'ai tendance à penser que le `{id_mot?}` a un comportement dérogatoire acceptable puisqu'il implique qu'on ne connait pas le type de la donnée à la compilation et donc il est normal qu'il s'adapte Si on modifie `{id_mot}` de la même façon cela implique donc une indétermination à la compilation qui ralentira toutes les requetes, et surtout ça ouvre ensuite la question de pourquoi `{id_mot=#ENV{id_mot}}` ne fonctionne pas pareil ? **Statut changé à En cours**
cerdic commented 2 years ago
Owner

Statut changé à Fermé

**Statut changé à Fermé**
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
1.8
1.9.1
1.9.2
2.0
2.1
3.0
3.1
3.2
4.0
4.1
4.2
boutons-danger
coquille_doc
debug_ecrire_fichier
dev-sortable
dev/autoloader
dev/hasard_fixe
dev/instituer_ergo
dev/issue_5447_exporter_csv
dev_infos_image
fix/valider_url_distante
fix_issue_5454
fix_modifier_login
issue_4101
issue_4678
issue_4705
issue_4717
issue_4836
issue_4946
issue_5258
issue_5344
issue_5427_bis
master
Labels
Apply labels
Clear labels
accessibilité amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label accessibilité amélioration APIs authentification base de données bug code généré compilo css divers documentation doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide javascript langues LDAP plugin PostgreSQL refusé sécurité traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
3.1
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes