openerp-community-reviewer team mailing list archive

["Holger Brunn \(Therp\)" <hbrunn@xxxxxxxx>]

Holger Brunn (Therp) has proposed merging lp:~hbrunn/ocb-addons/6.1-lp1376743 into lp:ocb-addons/6.1.

Requested reviews:
  OpenERP Community Backports (ocb)
Related bugs:
  Bug #1376743 in OpenERP Community Backports (Addons): "space at the end of user names confuse users_ldap"
  https://bugs.launchpad.net/ocb-addons/+bug/1376743

For more details, see:
https://code.launchpad.net/~hbrunn/ocb-addons/6.1-lp1376743/+merge/236886
-- 
https://code.launchpad.net/~hbrunn/ocb-addons/6.1-lp1376743/+merge/236886
Your team OpenERP Community Backports is requested to review the proposed merge of lp:~hbrunn/ocb-addons/6.1-lp1376743 into lp:ocb-addons/6.1.

=== modified file 'users_ldap/users_ldap.py'
--- users_ldap/users_ldap.py	2013-09-10 15:12:35 +0000
+++ users_ldap/users_ldap.py	2014-10-02 14:09:16 +0000
@@ -181,7 +181,7 @@
         """
         
         user_id = False
-        login = tools.ustr(login.lower())
+        login = tools.ustr(login.lower().strip())
         cr.execute("SELECT id, active FROM res_users WHERE login=%s", (login,))
         res = cr.fetchone()
         if res: