From f539967a22c33422334ca23a78e15e07572bd74f Mon Sep 17 00:00:00 2001
From: PCoder <purple.coder@yahoo.co.uk>
Date: Fri, 6 Mar 2020 00:01:27 +0530
Subject: [PATCH] Set in_ldap even when resetting dummy credential

---
 membership/models.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/membership/models.py b/membership/models.py
index 62c17671..703b4800 100644
--- a/membership/models.py
+++ b/membership/models.py
@@ -233,8 +233,6 @@ class CustomUser(AbstractBaseUser, PermissionsMixin):
                 ldap_manager.create_user(self.username, password=password,
                                          firstname=first_name, lastname=last_name,
                                          email=self.email)
-                self.in_ldap = True
-                self.save()
             else:
                 # User exists already in LDAP, but with a dummy credential
                 # We are here implies that the user has successfully
@@ -244,6 +242,8 @@ class CustomUser(AbstractBaseUser, PermissionsMixin):
                 # was set to a dummy value while migrating users from Django to
                 # LDAP
                 ldap_manager.change_password(self.username, password)
+            self.in_ldap = True
+            self.save()
 
     def __str__(self):  # __unicode__ on Python 2
         return self.email