Merge lp:~hbrunn/ocb-addons/6.1-lp1376743 into lp:ocb-addons/6.1

Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 6848
Proposed branch: lp:~hbrunn/ocb-addons/6.1-lp1376743
Merge into: lp:ocb-addons/6.1
Diff against target: 12 lines (+1/-1)
1 file modified
users_ldap/users_ldap.py (+1/-1)
To merge this branch: bzr merge lp:~hbrunn/ocb-addons/6.1-lp1376743
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review Approve
Ronald Portier (Therp) code review Approve
Stefan Rijnhart (Opener) Approve
Review via email: mp+236886@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks!

review: Approve
Revision history for this message
Ronald Portier (Therp) (rportier1962) wrote :

LGTM

review: Approve (code review)
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

LGTM

review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'users_ldap/users_ldap.py'
2--- users_ldap/users_ldap.py 2013-09-10 15:12:35 +0000
3+++ users_ldap/users_ldap.py 2014-10-02 14:11:23 +0000
4@@ -181,7 +181,7 @@
5 """
6
7 user_id = False
8- login = tools.ustr(login.lower())
9+ login = tools.ustr(login.lower().strip())
10 cr.execute("SELECT id, active FROM res_users WHERE login=%s", (login,))
11 res = cr.fetchone()
12 if res:

Subscribers

People subscribed via source and target branches