Re: [SPIP Zone] SPIP 3.3-beta

Re,

Le mer. 11 déc. 2019 à 11:09, Cerdic <[email protected]> a écrit :

> Hello,
>
> à la relecture j’ai corrigé&modifié la fonction objet_lire() comme suit
> (et avant qu’on ne l’utilise intensivement :p)
>  - renommage de l’option ‘informations' en ‘champs’ par coherence avec
> l’option ‘champ_id’
>

Ok, j'avais hésité aussi :p
Et le coup du sql_quote je ne savais pas. A noter.


>  - coherence du retour de la fonction selon si ‘champs' est un array ou
> une string et si l’enregistrement existe ou pas pour pouvoir distinguer les
> différents cas
>

Oui bien vu, ça permet de connaitre l'erreur si il y en a une. J'avais omis
le cas aucun champ valide


>  - bugfix sur le cache statique : il y avait une collision si on
> demandait la meme valeur_id mais avec 2 champ_id différents (et différents
> de la clé primaire)
>

Ah oui bien vu ! Le cas de merde je l'avais bien loupé en voulant
simplifier l'indexation du tableau.


>  - ajout d'une option force pour forcer la relecture en base
>

Ah ben je venais de l'enlever :p

Bon va falloir que je revois mes fonctions xxxx_lire() qui trainent dans
mes plugins car ces bugs peuvent y être. Le mieux serait d'utiliser cette
fonction.
Ne serait-il pas judicieux de la backporter pour SPIP 3 afin de pouvoir
justement l'utiliser sur des plugins SPIP 3 sans forcément faire une
nouvelle branche juste pour 3.3. Ca faciliterait son déploiement et c'est à
priori sans douleur ni risque non ?