Re: [Spip-zone-commit] r114271 - in _core_/plugins/mots

Le 3 mars 2019 à 22:07 +0100, Matthieu Marcillaud <[email protected]>, a écrit :
> Le 03/03/2019 à 09:35, Matthieu Marcillaud a écrit :
>
> > Ce que tu proposes là me parait aussi intéressant et plus générique,
> > quoi que plus difficile à coder. J’essaierai d’y jeter un œil tout de même.
>
> Oui, donc j’ai proposé un patch là https://core.spip.net/issues/4300
> pour ton idée de critère selection_conditionnelle.
>
> Tel que proposé actuellement, (ARTICLES){selection_conditionnelle}...
> va ajouter sur la boucle :
>
> - tous les {id_xx?} présents sur spip_articles (id_article, id_rubrique,
> id_secteur, id_trad)

Ah oui ceux-là aussi il faut les mettre, je n’y avais pas pensé
>
> - tous les {id_xx?} correspondants à des clés primaires des objets
> éditoriaux éditables (id_auteur, id_mot, id_groupe, id_message,
> id_syndic, id_breve, id_document) avec les plugins dist par défaut.

Du coup sur ça en y reflechissant je pense qu’il faudrait ne prendre que les id_x qui remplissent une des conditions suivantes
* id_x est un champ de la table sur laquelle la boucle porte (articles ici) -> mais du coup c’est pris en compte avec ta condition précédente
* la table x contient un champ id_article (ici, à apadapter en fonction de la boucle)
* il existe une table x_lien

Ce qui réduirait quand même déjà les combinatoires sans aucun sens ( {id_breve?} sur une table article n’a pas de sens)

Ensuite on peut dire que cette liste de clé automatique pour la table (articles) passe dans un pipeline, ce qui permet éventuellement aux plugins d’ajouter des cas particuliers qui sortent de ces critères là, ou au contraire de retirer un id_x qui n’a pas de sens

Et enfin on ajoute tous les {id_x?} en critère sur la boucle

>
> Ça fait quand même beaucoup de conditions théoriques et de calculs de
> jointures potentielles, même si elles ne sont finalement pas utilisées.
>
> En tout cas ça répond à la problématique pour
> prive/objets/liste/{objet}.html et son usage il me semble.

En effet là je pense qu’on peut déjà avoir une version qui marche et qui remplace notre {id_mot?} bancal sans duplication de code, à affiner selon les cas problématiques que l’on rencontrera ensuite

Merci !

Cédric