Hello,

Je rebondis sur ce commit à propos de l'utilisation de l'API SQL.
La correction est l'ajout d'une quote manquante (en vert foncé autour de
$extension" :
$enbase =
sql_fetsel('id_zip,fichiers,date_modif','spip_zippeur',"`nom`='$nom' and
`extension`='$extension'");

Je trouve que cette écriture est bien compliquée (d'ailleurs on a oublié
une quote) en utilisant 3 types de quotes différentes et donc pas hyper
maintenable surtout quand on a la possibilité d'utiliser les features de
l'API pour simplifier l'écriture de la façon suivante:

$select = array('id_zip', 'fichiers', 'date_modif');
$where = array("nom=" . sql_quote($nom), "extension=" .
sql_quote($extension));
$enbase = sql_fetsel($select, 'spip_zippeur', $where);

Ne devrions nous pas encourager ce type d'écriture plus maintenable à mon
avis et faire les modifications au fil des commits ?


++
Eric


Le ven. 6 sept. 2019 à 20:44, <spip-zone-com...@rezo.net> a écrit :

> Author: mai...@maieul.net
> Date: 2019-09-06 18:43:47 +0000 (Fri, 06 Sep 2019)
> New Revision: 117771
>
> Modified:
>    _plugins_/zippeur/trunk/paquet.xml
>    _plugins_/zippeur/trunk/zippeur_fonctions.php
> Log:
> quote manquante, merci Pierre K
>
>
> Details: https://zone.spip.org/trac/spip-zone/changeset/117771
>
> _______________________________________________
> spip-zone-com...@rezo.net -
> https://listes.rezo.net/mailman/listinfo/spip-zone-commit
>
----
spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone

Répondre à