Re: API Objet - Nouvelle fonction de lecture

Hello,

moi je suis sceptique sur le fait d’introduire des données dans la lecture via un/des pipelines, ce qui veut dire que de facto on a plus d’équivalence entre un sql_select et l’utilisation de l’API de lecture.

La fonction peut avoir de l’intérêt en tant que shorthand, pour fluidifier le code et/ou bénéficier du static (mais il faudrait voir si cela apporte vraiment un gain de perf, car de son côté mysql cache aussi les requêtes, c’est à dire que le même select va beaucoup plus vite la seconde fois), mais si elle doit devenir un passage obligé pour la lecture des données (pour cause de pipeline et d’injection externe de contenu hors SQL), ça va être contreproductif :

on va se retrouver à faire des gros SQL select avec des conditions compliquées en SQL pour récupérer les id, puis à appeler la fonction de lecture pour chaque id, et ça va ramer très fort à la fin.
De plus, cela veut dire aussi qu’on récupérerait via l’api de lecture des données qui ne correspondent pas à celles qu’on a avec une boucle.

Je pense donc que la fonction ne doit pas permettre de modifier/alterer/compléter ce qui vient de la base et n’être donc qu’un simple raccourci de sql_xx pour qui veut s’en servir.

--
Cédric
Le 10 sept. 2019 à 2:42 PM +0200, Bruno Bergot <[email protected]>, a écrit :
> Hop,
>
> Le 08/09/2019 à 13:22, Eric Lupinacci a écrit :
> >
> > Bon je ne sais pas si l'une ou l'autre est plus prioritaire mais je ne suis
> > pas sur que ce soit un gros de les proposer dans la prochaine version.
> > Je vais continuer à proposer les trois fonctions manquantes et on verra
> > alors.
> >
>
> Gogogo, fais péter une PR sur le trunk, tu auras certainement plus de
> retours dans les commentaires de celle-ci qu'ici ;)
>
> /me mode commentaire pas super utile mais qui au moins commente ^^
>
> ++
> b_b
> _______________________________________________
> 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