Added person['enabled'] check to the notify_owners() function. This will
authorStephen Soltesz <soltesz@cs.princeton.edu>
Mon, 19 Nov 2007 15:40:41 +0000 (15:40 +0000)
committerStephen Soltesz <soltesz@cs.princeton.edu>
Mon, 19 Nov 2007 15:40:41 +0000 (15:40 +0000)
prevent the bug where disabled persons with the tech-contact role are notified of boot notifications.

PLC/Boot.py

index cd3a738..221fbb2 100644 (file)
@@ -40,8 +40,8 @@ def notify_owners(method, node, message_id,
 
         persons = Persons(method.api, site['person_ids'])
         for person in persons:
-            if include_pis and 'pi' in person['roles'] or \
-               include_techs and 'tech' in person['roles']:
+            if (include_pis and 'pi' in person['roles'] and person['enabled']) or \
+               (include_techs and 'tech' in person['roles'] and person['enabled']) :
                 To.append(("%s %s" % (person['first_name'], person['last_name']), person['email']))
 
     # Send email