Re: [SPIP Zone] SPIP 3.3-beta

Hello,

pour la fonction objet_lire() j’avais soumis mon interrogation sur le fait qu’elle contiennent un appel à pipeline qui permet d’enrichir l’objet, chose qui n’est pas équivalente à un sql_fetsel() en base

C’est peut-être une direction dans laquelle on voudra aller, mais ça me semble pas certain, et lourd de conséquence.
Donc amha on peut l’intégrer mais sans ce pipeline : la fonction est juste un shorthand facilitateur, avec quelques automatismes/sécurités par rapport à un sql_fetsel() mais renvoie le même résultat.

Ça permet de l’introduire, de voir si on en s’en sert, et éventuellement de réflechir à l’étendre, mais c’est un sujet plus gros et plus vaste.

--
Cédric
Le 9 déc. 2019 à 15:19 +0100, Eric Lupinacci <[email protected]>, a écrit :
> Hello,
>
> Je remets une couche juste pour décider si on intègre ces deux sujets dans la 3.3 :
>
> la fonction objet_lire (que j'ai proposé il y a quelques semaines) :
> oui ou non, et pas de problème si c'est non.
>
> Amélioration ou pas du critère id_? :
> à mon avis il faut restreindre ce critère uniquement aux id_xxx et donc laisser faire le critère sans rajouter de possibilité d'y injecter autre chose.
> Modifier le pipeline lister_champs_selection_conditionnelle en exclure_id_conditionnel qui cette fois ne ferait que donner une liste d'id à exclure.
> A priori pas de souci pour ce changement puisque ce pipeline n'est utilisé que par SVP pour justement supprimer id_plugin et id_paquet je crois que la liste.
>
> ++
> Eric
>
>
> > Le sam. 7 déc. 2019 à 15:01, Jean Marie Grall <[email protected]> a écrit :
> > > Salut,
> > >
> > > Le 04/12/2019 à 16:26, Cerdic a écrit :
> > > > le modele logo n’était pas totalement heureux, je viens de le
> > > > corriger, mais quoi qu’il en soit ça ne devrait pas provoquer de bug
> > > > (les filtres image_xx du core fonctionnant très bien), il faut donc
> > > > regarder du cote du plugin responsive
> > >
> > > effectivement, pas mieux avec la correction, à creuser donc du côté de
> > > image_responsive.
> > >
> > >                      jeanmarie
> > >
> > > _______________________________________________
> > > liste: https://listes.rezo.net/mailman/listinfo/spip-dev
> > > doc: http://www.spip.net/
> > > dev: http://trac.rezo.net/trac/spip/
> > > irc://irc.freenode.net/spip