[Rt-commit] rt branch, 4.2-trunk, updated. rt-4.2.10-145-ga857212

[Alex Vandiver]

The branch, 4.2-trunk has been updated
       via  a8572123b3a561bee465566ad2ead0cff82e636a (commit)
       via  c9887fb7a2af7b6fdfa3c1a7cf3cd7b31cc2704c (commit)
      from  5ff834f66c9d8b98c94c05807bff63c74090af51 (commit)

Summary of changes:
 lib/RT/Action/Notify.pm | 1 +
 1 file changed, 1 insertion(+)

- Log -----------------------------------------------------------------
commit c9887fb7a2af7b6fdfa3c1a7cf3cd7b31cc2704c
Author: Christian Loos <cloos at netcologne.de>
Date:   Thu Dec 4 16:20:55 2014 +0100

    Don't notify disabled owners
    
    All other notification targets are groups, not individuals -- and
    calling ->MemberEmailAddresses implicitly filters for non-disabled
    users.  The Owner, however, gets no such implicit handling.
    
    Explicitly check to see if the Owner is disabled before adding their
    address to notify.

diff --git a/lib/RT/Action/Notify.pm b/lib/RT/Action/Notify.pm
index d38fd26..483af32 100644
--- a/lib/RT/Action/Notify.pm
+++ b/lib/RT/Action/Notify.pm
@@ -107,6 +107,7 @@ sub SetRecipients {
     if (   $arg =~ /\bOwner\b/
         && $ticket->OwnerObj->id != RT->Nobody->id
         && $ticket->OwnerObj->EmailAddress
+        && not $ticket->OwnerObj->Disabled
     ) {
         # If we're not sending to Ccs or requestors,
         # then the Owner can be the To.

commit a8572123b3a561bee465566ad2ead0cff82e636a
Merge: 5ff834f c9887fb
Author: Alex Vandiver <alexmv at bestpractical.com>
Date:   Thu Mar 5 19:10:55 2015 -0500

    Merge branch '4.2/notify-disabled-owner' into 4.2-trunk


-----------------------------------------------------------------------