[sysadmin-bin] Remove silly if which is already captured by the for loop around it



commit 24e94f44ac2b063f39fbdabcdb6fd09c0cb36c8a
Author: Patrick Uiterwijk <patrick puiterwijk org>
Date:   Thu Oct 26 11:39:38 2017 +0200

    Remove silly if which is already captured by the for loop around it
    
    Signed-off-by: Patrick Uiterwijk <patrick puiterwijk org>

 membership/foundation-operations.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/membership/foundation-operations.py b/membership/foundation-operations.py
index b9ba5fb..3051b76 100755
--- a/membership/foundation-operations.py
+++ b/membership/foundation-operations.py
@@ -89,7 +89,7 @@ def remove_expired_memberships_from_foundation():
         mail_attr = ldap_fields[2]
         common_name_attr = ldap_fields[3]
 
-        if member in foundationmembers and convert_to_unix_time < now - 365 * 2 * 24 * 60 * 60:
+        if convert_to_unix_time < now - 365 * 2 * 24 * 60 * 60:
             print "Removing %s from the foundation LDAP group as the membership expired on %s" % (member, 
last_renewed_on_attr)
             glu.remove_user_from_ldap_group(member, 'foundation')
             send_form_letters(expired_membership_form_letter, mail_attr, common_name_attr, 
last_renewed_on_attr)
@@ -365,4 +365,4 @@ def main():
         generate_membership_list()
 
 if __name__ == "__main__":
-    main()
\ No newline at end of file
+    main()


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]