Re: [Dolibarr-dev] Problème sur la foncti on update de societe.class.php

[Laurent Destailleur (Eldy)]

Rodolphe Quiedeville wrote:

> Bonjour,
>
> Le retour de la fonction update de societe.class.php est passé de 0 à 1
> en cas de succès. Il faut faire très attention à ce genre de
> modification au vu des conséquence que cela peut avoir.
>
> Pour rappel le retour correct d'une fonction doit être 0 en cas de
> succès. Beaucoup de fonction retourne encore 1 en cas de succès mais il
> faut au fur et à mesure les modifier vers 0 et surtout pas le contraire.
>
>  
Euh je crois que c'est plutot l'inverse. La plupart des fonctions 
renvoient négatif en cas d'erreur et positif en cas de succès (plus de 
325, sans compter celle qui retourne un autre positif comme le résultat 
d'un comptage ou qui retourne l'id d'un objet crée).
Il y a encore beaucoup de fonctions qui font return 0 (316) mais presque 
toute font return 0 pour signaler une erreur et return 1 en cas de 
succès. Il est donc plus simple de modifier celles la pour modifier le 0 
(qui pour elles veut dire erreur) en négatif (-1, -2 selon erreur) où la 
y a plus d'ambiguité car cela ne change pas le cas succès pour elle.
A mon avis il faut mieux instaurer que succès c'est >= 0 (plutot que = 0 
seul) et que erreur c'est < 0. Cela permet d'etre proche plus proche du 
but (à moindre risque)...

--
Laurent Destailleur.
---------------------------------------------------------------
EMail: address@hidden
Web: http://www.destailleur.fr
IM: IRC=Eldy, Jabber=Eldy

AWStats (Author) : http://awstats.sourceforge.net
Dolibarr (Contributor) : http//www.dolibarr.com
CVSChangeLogBuilder (Author) : http://cvschangelogb.sourceforge.net
AWBot (Author) : http://awbot.sourceforge.net