[Freeipa-devel] [PATCH] Catch errors.EmptyModlist in b_ldap

[Jakub Hrozek]

I think we shouldn't let an EmptyModlist exception propagate all the way
up to ldap.update(). The method is usually used in fronted plugins, like
classes that derive from crud.Mod, where modifying an record to the same
attributes can be done pretty easily, so all the crud.Mod plugins would
have to catch that exception anyway.

The attached patch modifies ldap.update to return None if no
modification was done.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Return-None-instead-of-EmptyModlist-in-ldap.update.patch
Type: text/x-patch
Size: 1091 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090213/0cb72607/attachment.bin>