Re: [SPIP Zone] [Spip-zone-commit] r117771 - _plugins_/zippeur/trunk

2019-09-08 Par sujet Eric Lupinacci
Ok.

++
Eric


Le dim. 8 sept. 2019 à 11:12, Maïeul Rouquette  a écrit :

> Tu as parfaitement raison sur le fond.
> Simplement j'éais un peu pressé par le temps.
> Le dimanche 08 septembre 2019 à 11:09 +0200, Eric Lupinacci a écrit :
> > 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
> >
> >
>
>

spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] [Spip-zone-commit] r117771 - _plugins_/zippeur/trunk

2019-09-08 Par sujet Maïeul Rouquette
Tu as parfaitement raison sur le fond. 
Simplement j'éais un peu pressé par le temps. 
Le dimanche 08 septembre 2019 à 11:09 +0200, Eric Lupinacci a écrit :
> 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
> 
> 


spip-zone@rezo.net - https://listes.rezo.net/mailman/listinfo/spip-zone


Re: [SPIP Zone] [Spip-zone-commit] r117771 - _plugins_/zippeur/trunk

2019-09-08 Par sujet Eric Lupinacci
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,  a écrit :

> Author: mai...@maieul.net
> Date: 2019-09-06 18:43:47 + (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