From 4984769c242b72daf93900bfd69a177f064ae9f2 Mon Sep 17 00:00:00 2001 From: Stephen Soltesz Date: Mon, 19 Nov 2007 15:40:41 +0000 Subject: [PATCH] Added person['enabled'] check to the notify_owners() function. This will prevent the bug where disabled persons with the tech-contact role are notified of boot notifications. --- PLC/Boot.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PLC/Boot.py b/PLC/Boot.py index cd3a738..221fbb2 100644 --- a/PLC/Boot.py +++ b/PLC/Boot.py @@ -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 -- 2.43.0