additional attributes from ldap: mail, preferredLanguage for initial creation

[udono, 2010-04-28 20:20:22]

par kleine sachen, soweit gut.

http://codereview.appspot.com/962047/diff/1/2
File res.py (right):

http://codereview.appspot.com/962047/diff/1/2#newcode139
res.py:139: 'preferredLanguage'], context=context)
Leerzeichenvor 'mail'

http://codereview.appspot.com/962047/diff/1/2#newcode151
res.py:151: lang_obj = self.pool.get('ir.lang')
Objekte werden üblicherweise am Anfang einer Methode instanziert. Ich würde
Zeile 151 hinter Zeile 124 einfügen.

http://codereview.appspot.com/962047/diff/1/2#newcode154
res.py:154: [login])[0]), ], context=context)
Versteh ich nicht. wieso ist [login] der default Wert wenn 'preferredLanguage'
nicht existiert?

D.get(k[,d]) -> D[k] if k in D, else d.  d defaults to None.

http://codereview.appspot.com/962047/diff/1/2#newcode158
res.py:158: pass
else pass kannst du weglassen.

http://codereview.appspot.com/962047/diff/1/2#newcode160
res.py:160: user_pref['email'] = attrs.get('mail', [login])[0]
Versteh ich nicht. wieso ist [login] der default Wert wenn 'mail' nicht
existiert?

D.get(k[,d]) -> D[k] if k in D, else d.  d defaults to None.

[paepke, 2010-04-29 06:26:03]

http://codereview.appspot.com/962047/diff/1/2
File res.py (right):

http://codereview.appspot.com/962047/diff/1/2#newcode151
res.py:151: lang_obj = self.pool.get('ir.lang')
On 2010/04/28 20:20:22, udono wrote:
> Objekte werden üblicherweise am Anfang einer Methode instanziert. Ich würde
> Zeile 151 hinter Zeile 124 einfügen.

ist das instanzieren hier lazy? möchte nicht unnötige arbeit provozieren. hatte
es auch erst oben.

http://codereview.appspot.com/962047/diff/1/2#newcode154
res.py:154: [login])[0]), ], context=context)
On 2010/04/28 20:20:22, udono wrote:
> Versteh ich nicht. wieso ist [login] der default Wert wenn 'preferredLanguage'
> nicht existiert?
> 
> D.get(k[,d]) -> D[k] if k in D, else d.  d defaults to None.

stimmt. macht keinen sinn.

[paepke, 2010-04-29 06:46:08]

http://codereview.appspot.com/962047/diff/5001/6001
File res.py (left):

http://codereview.appspot.com/962047/diff/5001/6001#oldcode147
res.py:147: 'name': attrs.get(str(connection.auth_uid), [login])[0],
wieso die auth_uid? ich bin der meinung das 'cn' hier viel mehr sinn macht als
default. Active Directory hat das sinnvoll bestückt als beispiel. und der
fallback auf login machts ja sicher.

[udono, 2013-03-27 12:59:42]

can be closed?