ldap_connection: http://codereview.appspot.com/1882044
ldap_authentication:  http://codereview.appspot.com/1742053
please see the comment on ldap_authentication in appspot:

res.py:48: connection = connection_obj.browse(connection_ids[0])
i'm unsure about the change to remove the "0" next to cursor. it works
only without that "0".

my appologies, i'm off for today for a wedding ( not mine ;-) )

regards

tobias


tested against openldap

On 29.07.2010 17:17, Cédric Krier wrote:
On 28/07/10 23:04 +0200, Cédric Krier wrote:

Hi,

I'm asking your help to speed up the migration to the new transaction design.
Here is my proposition to not have duplicate work.
People who wants to help can post (with reply to this email) on which module
he will work before starting. So by posting it he engages to provide a patch
for review in 3 days (normally it should take 3-4 hours) after that it will be
considered as abandoned.
Modules that will not require modification must be also posted here by saying
no need to change.

By the way, I benefit of this big change to fix the order of the import to
follow PEP 8:

     1. standard library imports
     2. related third party imports
     3. local application/library specific imports




--
[email protected] mailing list

Reply via email to