Pérennité du filtre |plus sur des tableaux #5600

Closed
opened 2 weeks ago by nicod_ · 2 comments
nicod_ commented 2 weeks ago
Owner

Le PHPdoc indique que ce filtre ajoute deux nombres :
c8befb0279/ecrire/inc/filtres.php (L2115)

Mais actuellement rien n'empêche de l'utiliser sur des tableaux.
Exemple, qu'on ne peut pas faire avec |push ni |array_merge (qui écrase les clés) :

#SET{data,#GET{data}|plus{#ARRAY{#ID,#TITRE}}}

Est-ce pérenne, et dans ce cas on modifie le PHPdoc et la doc https://www.spip.net/fr_article4307.html#plus ? (je peux m'en occuper)

Ou bien est ce que c'est amené à être typé (et donc ça ne marchera plus) ?

Evidemment, ça crée une dérogation par rapport à |moins, |mult, etc mais ça ne me semble pas grave tant que c'est assumé (et ça colle à PHP).

Le PHPdoc indique que ce filtre ajoute deux nombres : https://git.spip.net/spip/spip/src/commit/c8befb0279f4699c08378a2c966e7a6c7504470b/ecrire/inc/filtres.php#L2115 Mais actuellement rien n'empêche de l'utiliser sur des tableaux. Exemple, qu'on ne peut pas faire avec `|push` ni `|array_merge` (qui écrase les clés) : ``` #SET{data,#GET{data}|plus{#ARRAY{#ID,#TITRE}}} ``` Est-ce pérenne, et dans ce cas on modifie le PHPdoc et la doc https://www.spip.net/fr_article4307.html#plus ? (je peux m'en occuper) Ou bien est ce que c'est amené à être typé (et donc ça ne marchera plus) ? Evidemment, ça crée une dérogation par rapport à |moins, |mult, etc mais ça ne me semble pas grave tant que c'est assumé (et ça colle à PHP).
Owner

Hello @nicod_ , ça semble un doublon de #5389

J'ai pas de copie de toute la zone sous la main à greper, mais je vois que c'est aussi utilisé pour combiner des tableaux dans saisies.

Plutôt d'avis de laisser cette flexibilité et modifier le phpdoc pour ma part.

Hello @nicod_ , ça semble un doublon de #5389 J'ai pas de copie de toute la zone sous la main à greper, mais je vois que c'est aussi utilisé pour combiner des tableaux dans saisies. Plutôt d'avis de laisser cette flexibilité et modifier le phpdoc pour ma part.
nicod_ commented 1 week ago
Poster
Owner

Ah ok, je suis pas le seul à me poser la question alors.
Bon, on ferme ici.

Ah ok, je suis pas le seul à me poser la question alors. Bon, on ferme ici.
nicod_ closed this issue 1 week ago
b_b added the
doublon
label 1 week ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: spip/spip#5600
Loading…
There is no content yet.