Re: sql_insertq bug ou pas ?

Hello,

ça me parait plus propre de garder false vs 0 ou n’importe quel entier, à charge pour l’appelant de tester proprement === false, mais il faut mettre à jour le PHPDoc pour que ce soit clair

Le hack consistant à renvoyer -1 est un risque de bug en plus sur des cas où on insère explicitement avec un id=-1 et qui ne devient plus testable (ce qui existe aussi dans la nature)

--
Cédric
Le 8 juil. 2019 à 18:15 +0200, [email protected] <[email protected]>, a écrit :
> Salut
>
> > > Est ce qu'on ne serait pas dans un cas où 0 != false ? De ce que j'en
> > > comprends 0 serait licite comme résultat.
> >
> > Absolument, dans le cas de cette table le retour 0 n'indique pas une erreur d'insertion mais aucun id ce qui est vrai !
> > Ce problème avait été contourné à une époque et puis supprimé sans modifier les explications du PHPDoc ce qui rend l'API pas très compréhensible.
> >
> > > Donc faudrait utiliser === ou !== pour distinguer les 2 cas.
> >
> > C'est ce que j'ai fait mais je trouve pas ça heureux.
> > Mais bon je dois être le seul alors on va pas se faire chier.
>
> Autre option que je vois, c'est directement dans la méthode faire le
> test sur false et retourner -1 , 0, ou plus selon
>
> Km
> _______________________________________________
> 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