Re: SPIP nu

Le 24 mai 2016 à 08:58, Eric Lupinacci <[email protected]> a écrit :

> Yop,
>
> Le 24 mai 2016 à 08:49, James <[email protected]> a écrit :
>
>> Salut,
>>
>> Un patch (non, 3) donc, mais pas pour un critère {pipeline} parce que
>> finalement, ça me plaisait pas trop.
>>
>> ici, j'introduis un pipeline 'inserer_criteres' qui permet d'injecter des
>> critères avant que le compilateur ne démarre son analyse. Ce qui permet de
>> coder des insertions à l'aide de chaine de caractères (ex: "{id_mots?}"
>> plutôt que de construire des instances de l'objet Critere, parce que c'est
>> cauchemardesque :-)
>>
>> Exemple exhaustif avec les plugins mots et organiseur.
>>
>>
> C'est cool.
> Oui dans mon idée le {pipeline} avait pour but d'être remplacé avant la
> compilation par la chaine souhaitée.
> C'est plus souple comme tu le dis.
> Après avoir un pipeline inserer_critere qui le fait sans critère précis
> dans la boucle why not.
> Le seul inconvénient que j'y vois (déjà discuté avec Rasta) c'est de
> devoir faire le tri des boucles avec et sans injection du critère dans la
> fonction alors qu'avec {pipeline} c'est clair et c'est plus lisible (sans
> ce critère ou ne sait jamais si une boucle va être modifiée ou pas).
>

En fait, le critère {pipeline} pose plusieurs problèmes et le plus
important de tous, c'est qu'il intervient après la compilation de la
boucle. La conséquence, c'est que l'insertion d'un critère passe
nécessairement par l'instanciation d'un objet Critere et d'une structure de
paramètres particulièrement nébuleuse, surtout avec des critères contenant
des champs avec filtres ... (ex: {en_ligne>#VAL{Y-m-d H:i:s}|date{#VAL{-15
minutes}|strtotime}} ...)

Pour permettre à un contributeur d'insérer un critère avec une syntaxe
simple, celle que l'on connait dans les squelettes, il faut insérer le
critère avant compilation de la boucle et là, bah y a pas des masse
d'endroits où le faire sans casser SPIP. En tout cas, c'était pas simple
sans remettre en cause nombres d'implémentations des objets du compilo, du
phraseur, bref ... :-)

J'ai essayé pre_boucle, et aussi de faire un pre_recuperer_fond ... et là,
j'ai eu mal aux cheveux mais je rentre pas dans les détails ;-)


>
> Mais j'ai hate de tester ton patch.
>
> ++
> Eric
>
>
>


-- 
James