Page 1 sur 1
[Corrigé 3.1.9] LDAP mise a jour de la société
Posté : lun. 13 juin 2016 18:03
par olikamm
Bonjour,
un petit bug dans le code, si vous voulez importer la société de l'utilisateur via une synchro LDAP cela ne fonctionnera pas car il y a une erreur dans le code (fichier ldpad.php)
En effet dans le code il lit la table tusers, colonne "company", mais il met a jour la colonne "service" :
Code : Tout sélectionner
if ($find_company!='')
{
$query= "UPDATE tusers SET service='$find_company' WHERE login='$samaccountname'";
$exec = mysql_query($query) or die('Erreur SQL !<br />'.mysql_error());
}
Il faut corriger par :
Code : Tout sélectionner
if ($find_company!='')
{
$query= "UPDATE tusers SET company='$find_company' WHERE login='$samaccountname'";
$exec = mysql_query($query) or die('Erreur SQL !<br />'.mysql_error());
}
Pour info je suis en version 3.0.11 (3.0 patch 11)
Re: BUG - Import LDAP - mise a jour de la société de l'utili
Posté : lun. 13 juin 2016 19:21
par olikamm
Un autre bug pour l'import de l'attribue de société via LDAP :
- si vous avez plusieurs société il va vous créé X fois la société dans les liste car il y a un erreur dans le while, j'en ai profité pour amélioré la comparaison du nom de la société :
Original :
Code : Tout sélectionner
while ($row=mysql_fetch_array($q))
{
if ($company==$row['name']) $find_company=$row['id']; else $find_company='';
}
Corrigé :
Code : Tout sélectionner
while ($row=mysql_fetch_array($q))
{
//if ($company==$row['name']) $find_company=$row['id']; else $find_company='';
if(strcasecmp($company, $row['name']) == 0)
{
$find_company=$row['id'];
break;
}
else
{
$find_company='';
}
}
Re: BUG - Import LDAP - mise a jour de la société de l'utili
Posté : lun. 13 juin 2016 22:22
par Flox
Bonjour,
merci pour votre recherche nous avons corrigé le soucis pour la 3.1.9, n'hésitez pas à passer en dernière version pour profiter dernières corrections.
Cdt
Re: [Corrigé 3.1.9] LDAP mise a jour de la société
Posté : jeu. 21 juil. 2016 21:37
par jerome.stival
Bonjour
J'ai essayé de modifier les lignes cités ci-dessus dans ma version 3.1.7 mais visiblement je n'ai pas su faire cela correctement.
Flox a écrit :Bonjour,
merci pour votre recherche nous avons corrigé le soucis pour la 3.1.9, n'hésitez pas à passer en dernière version pour profiter dernières corrections.
Cdt
La version 3.1.9 ??? Serait il possible d'avoir un patch pour la 3.1.7 car cela est assez pénalisant lorsqu'on a de multiples sociétés comme c'est notre cas
Merci de votre aide à tous
Re: [Corrigé 3.1.9] LDAP mise a jour de la société
Posté : jeu. 21 juil. 2016 23:22
par Flox
Bonjour,
la version 3.1.9 devrai sortir la semaine prochaine en attendant vous pouvez essayer de remplacer le fichier /core/ldap.php, par celui en pj.
cdt
Re: [Corrigé 3.1.9] LDAP mise a jour de la société
Posté : ven. 22 juil. 2016 17:12
par jerome.stival
Bonjour
Effectivement, je viens de faire le test, ça fonctionne beaucoup mieux
J'ai hate de découvrir la version 3.1.9, on peut dès à présent voir le changelog ? Ou ce sera la surprise
Re: [Corrigé 3.1.9] LDAP mise a jour de la société
Posté : lun. 25 juil. 2016 20:12
par Flox
La mises a jour devrai sortir cette semaine.