dolibarr-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Dolibarr-dev] Bravo pour le module associations,


From: Franky Van Liedekerke
Subject: Re: [Dolibarr-dev] Bravo pour le module associations,
Date: Tue, 16 Jan 2007 13:10:41 +0100


Yannick,

j'ai vu que tu utilise addslashes, mais c'est dangereux si php le fait déjà (magic_qoutes_gpc, activé par default en php.ini). Alors, je pense que c'est mieux de créer un fonction doli_addslashes avec le code suivante:

function doli_addslashes ($string) {
   return (! get_magic_quotes_gpc ()) ? addslashes ($string) : $string;
}

Franky

--
 


-----Original Message-----
From: Yannick Warnier <address@hidden>
Sent: Tuesday, 16. Jan 2007 12:35 +0100
To: Discussions sur le developpement de Dolibarr <address@hidden>
Subject: Re: [Dolibarr-dev] Bravo pour le module associations,

Le mardi 16 janvier 2007 à 11:11 +0000, Yannick Warnier a écrit :
> Le mardi 16 janvier 2007 à 10:04 +0100, Franky Van Liedekerke a écrit :
> [...]
> > - pour le bug de "date de naissance": c'estle même problème que
> > Yannick a corrigé pour un autre cas: mktime ne peut pas fonctionner
> > avec une date moins de 1970 (1969, ...). Les dates après 1970
> > fonctionnent bien (le bug existe aussi sur linux). Si Yannick n'a pas
> > le temps de corriger, je le fais ce soir aussi.
>
> Je vois que t'as pas réussi à attendre ce soir
>
> Bon, il y a quand même quelques modifs en plus à apporter par rapport à
> tes changements. Je suis en train de regarder ça, donc pas touche, mais
> en gros, la date de cotisation suit la même logique et j'ai également
> remarqué que dans certains cas on a 195436 pour 1954-03-06, ce qui
> forcément ne va pas, donc présumant qu'une opération sur les entiers est
> plus rapide qu'une opération sur les strings, j'ai maintenant:
> $date = ($year*1000)+($month*100)+$day;
> plutôt que
> $date = $year.$month.$day;
>
> Je corrige cela dans quelques minutes, après avoir bien testé le tout.
> Patience.

Voilà qui est fait. En même temps, apparemment, à l'introduction d'un
nouveau membre, la date de cotisation n'était pas enregistrée si la date
de naissance n'était pas remplie... (la vérification des données en
entrée étaient légèrement *décalée*)

Yannick



_______________________________________________
Dolibarr-dev mailing list
address@hidden
http://lists.nongnu.org/mailman/listinfo/dolibarr-dev

reply via email to

[Prev in Thread] Current Thread [Next in Thread]