Re: [Glpi-dev] Gestion mail

[baaZ]

DOMBRE Julien a écrit :
> Salut,
>
> voila ca a été plus rapide que je pensais.
>
> Voila le fichier de diff pour la gestion email.
> Je peux vous envoyer les fichiers modifiés.
> Ca marche nickel.
> Je n'ai juste pas encore traduit les chaines de textes.
>
> En attente de vos remarques.

> <    function test_type()
> <    {
> <            if (!is_a($this->job,"Job"))
> <                    $this->job=NULL;
> <            if (!is_a($this->user,"User"))
> <                    $this->user=NULL;    
> <    }


Je me suis permis de remplacer ça par ce qui suit car is_a() n'est 
implémentée qu'a partir de PHP 4.2 ou superieur, ce qui risque de poser 
quelques problemes de compatibilité avec certaines vieilles versions :

        function test_type()
        {
                if (!(get_class($this->job)=="Job"))
                        $this->job=NULL;
                if (!(get_class($this->user)=="User"))
                        $this->user=NULL;    
        }

NB : rien n'est commit sur le CVS encore...

Voila qui lance le débat pour savoir si nous partons du principe que les 
versions de PHP des utilisateurs de glpi sont à jour, ou si nous 
utilisons une version "restreinte" de l'API pour que tout le monde soit 
compatible...

Qu'en pensez vous ?

Pour ma part je serais pour essayer tant que cela reste possible 
d'utiliser les fonctions implémentées dans PHP4 ou PHP4.1 choix qui 
n'est pas arbitraire mais qui correspond à la version du package PHP4 
inclu dans une GNU/LINUX Debian dans sa version stable.

Un feed back des autres personnes inscrites ici serait le bienvenu.

--
Bazile