[rt-users] NotifyActor ignored for Notify Owner

[Jon Daley]

 	Thanks! this fix works perfectly (on 3.4.2)  I just ran into this 
yesterday, and hadn't gotten around to debugging it yet.


On Thu, 16 Jun 2005, Dan Fulbright wrote:
> OK, I figured this one out on my own, based on something I found on the 
> rt-devel list. My e-mail address has a + (plus sign) in it. That messed up 
> the regular expressions in lib/RT/Action/Notify.pm:
>
>    if ($RT::NotifyActor) {
>        @{ $self->{'To'} }  = @To;
>        @{ $self->{'Cc'} }  = @Cc;
>        @{ $self->{'Bcc'} } = @Bcc;
>    }
>    else {
>        @{ $self->{'To'} }  = grep ( !/^$creator$/, @To );
>        @{ $self->{'Cc'} }  = grep ( !/^$creator$/, @Cc );
>        @{ $self->{'Bcc'} } = grep ( !/^$creator$/, @Bcc );
>    }
>
> I fixed this by changing the quoting $creator in the regexps with \Q:
>
>        @{ $self->{'To'} }  = grep ( !/^\Q$creator\E$/, @To );
>        @{ $self->{'Cc'} }  = grep ( !/^\Q$creator\E$/, @Cc );
>        @{ $self->{'Bcc'} } = grep ( !/^\Q$creator\E$/, @Bcc );