re Bonjour,

J'ai appliqué mon patch sur la 0.83.6 que nous avons actuellement en
test ici, la synchronisation des LDAP se fait sur 1700 comptes environ.

Je vous remonterais toute erreur.

Bon WE

David OLIVIER

On 19/10/2012 15:09, MoYo wrote:
> re Bonjour,
> 
> je viens finalement de faire l'intégration en 0.84 dès aujourd'hui.
> 
> Merci encore.
> 
> Julien Dombre
> 
> 
> Le 19/10/2012 15:03, MoYo a écrit :
>> Bonjour,
>>
>> effectivement maintenant avec la notion d'emails multiples il faut
>> qu'on intègre votre proposition.
>>
>> Je m'en occupe mardi en 0.84. L'application en 0.83 me semble trop
>> dangereuse sans tests à grande échelle.
>>
>> Merci et bon WE
>>
>> Julien Dombre
>>
>>
>>
>> Le 19/10/2012 12:01, David Olivier a écrit :
>>> Bonjour à tous,
>>>
>>> Dans un LDAP l'attribut mail peut contenir plusieurs valeurs, il serait
>>> donc bon de toutes les récupérer si c'est le cas.
>>>
>>> Voici un patch qui fait le travail :
>>>
>>> Index: inc/user.class.php
>>> ===================================================================
>>> --- inc/user.class.php    (revision 19541)
>>> +++ inc/user.class.php    (working copy)
>>> @@ -1179,10 +1179,13 @@
>>>                     case "email2":
>>>                     case "email3":
>>>                     case "email4":
>>> -                     if (!empty($v[0][$e][0])) {
>>> -                        $this->fields["_emails"][] =
>>> addslashes($v[0][$e][0]);
>>> -                     }
>>> -                     break;
>>> +                     if (!empty($v[0][$e])) {
>>> +                        foreach ($v[0][$e] as $km => $m) {
>>> +                           if(!preg_match('/count/',$km)) {
>>> +                              $this->fields["_emails"][] =
>>> addslashes($m);
>>> +                           }
>>> +                        }
>>> +                     }
>>>                     case "language" :
>>>                        $language = Config::getLanguage($v[0][$e][0]);
>>>                        if ($language != '') {
>>> @@ -1470,7 +1473,7 @@
>>>         if ($ID != Session::getLoginUserID() &&
>>> !Session::haveRight("user", "r")) {
>>>            return false;
>>>         }
>>> -
>>> +
>>>         if ($ID > 0) {
>>>            $this->check($ID,'r');
>>>            $caneditpassword = $this->currentUserHaveMoreRightThan($ID);
>>>
>>>
>>> Adadov
>>>
>>> _______________________________________________
>>> Glpi-dev mailing list
>>> Glpi-dev@gna.org
>>> https://mail.gna.org/listinfo/glpi-dev
>>
>>
>> _______________________________________________
>> Glpi-dev mailing list
>> Glpi-dev@gna.org
>> https://mail.gna.org/listinfo/glpi-dev
> 
> 
> _______________________________________________
> Glpi-dev mailing list
> Glpi-dev@gna.org
> https://mail.gna.org/listinfo/glpi-dev


_______________________________________________
Glpi-dev mailing list
Glpi-dev@gna.org
https://mail.gna.org/listinfo/glpi-dev

Reply via email to