Fix update_fields fail if not present

coin/members/models.py

@@ -148,7 +148,7 @@ class Member(CoinLdapSyncMixin, AbstractUser):
         # Do not perform LDAP query if no usefull fields to update are specified
         # in update_fields
         # Ex : at login, last_login field is updated by django auth module.
-        if set(['username', 'last_name', 'first_name']).isdisjoint(set(update_fields)):
+        if update_fields and set(['username', 'last_name', 'first_name']).isdisjoint(set(update_fields)):
             return
 
         # Fail if no username specified

coin/mixins.py

@@ -28,7 +28,7 @@ class CoinLdapSyncMixin(object):
         creation = (self.pk == None)
 
         # Récupère les champs mis à jour si cela est précisé
-        update_fields = kwargs['update_fields'] if kwargs['update_fields'] else None
+        update_fields = kwargs['update_fields'] if 'update_fields' in kwargs else None
 
         # Sauvegarde en base de donnée (mais sans commit, cf decorator)
         super(CoinLdapSyncMixin, self).save(*args, **kwargs)