import user from an UTF-8 LDAP #93
Comments
Hello and thanks for the report (and the diff), As you may notice, Libertempo is a french application, so, even if it's a good practice to set a charset in UTF-8 for occidental apps., we never really considered changing defaut french charset ISO-8859-1{5} to UTF-8. I think it's a error and it should be considered as a bug. I really don't know how to proceed. If it's works well for you for now, you won't be able to do any upgrades as long as we don't give a global solution and i am not sure of the impacts. @wouldsmina can you test this patch with yours ldap / cas users ? Note : you forgot a « I » in $sql1=$sql1."u_nom='".addslashes(mb_convert_encoding($tab_new_user['nom'],"UTF-8","SO-8859-15"))."', "; Why did you pop these two preg_replace ? I invite you to check our futures changelogs. I hope i didn't write english mistakes, i'm rusted :-D |
J'ai un souci d'import d'un utilisateur dont le nom comporte une apostrophe (et aucune erreur php). |
C'est lié au preg_replace qui n'accepte pas les apostrophes. |
@, est ce que tu as des accents dans les login, nom ou prenomprenomk ton ldapldap c'est le cas, aurais tu le temps de tester le patch? J'aimerais savoir si il y a des effets de bord. |
Je n'ai pas d'accent dans les logins, ni caractères spéciaux. Et donc le reste du patch n'est pas testable dans mon cas de figure. |
Chez toi aussi vois êtes pas joueur :p |
Je respecte les normes et bonnes pratiques, nuance ;) |
Je le mets ici @Shadok parce qu'il faut le mettre quelques part et ta PR va être atomisée ^^ Après en avoir bien discuté, nous avons un énorme problème d'encodage, c'est un fait, mais pour ce qui est de permettre d'importer les utilisateurs utf8 de LDAP dans LT, c'est la BDD qui a tord. Nous devons donc mettre les champs Je crois savoir qu'il y a aussi un problème sur les mails. Dans ce cas, il est nécessaire de :
Je pense que ça peut faire le job et être les changements suffisants et nécessaires pour corriger le bug. Il y aura beaucoup de vérification à faire mais ça ne devrait pas être éternel. NB : vu qu'il y aura beaucoup de patches, ne pas oublier les migration des versions antérieures. |
Pas de problème, tant que ça marche au final :) |
Au besoin, plus d'infos peut être trouvé sur la meleu du 7.02.17. |
Gné ? |
bien sur que si qu'il a reçu une réponse!!! il y a 5 minutes :D |
|
user_utf8_ldap_import_patch.txt
Hello,
my ldap ( openDJ) export users in UTF8,
libertempo fail to import user with non ascii name ,
because the database is ISO-8859-1 .
i have made a path to fix this in admin_ajout-user.php
basicaly convert user givenname,name in ISO before inserting in base.
it work well for me , here is the patch (attached), may be useful :
The text was updated successfully, but these errors were encountered: