NotifyActor ignored for Notify Owner

RT Users
1

When I “Take” a ticket, I get an e-mail that says, for example:

Transaction: Given to supportguy by supportguy

The problem is, I’m supportguy, so I know I just gave the ticket to
myself. I have this variable unchanged from the default:

Set($NotifyActor, 0);

I know which scrip is doing this (again, a default scrip):

On Owner Change Notify Owner with template Transaction

Is this the desired behavior, or is this a bug? Is there any way I could
achieve the desired effect of notifying the new owner only if someone
else gives them the ticket?

I know that I could just dink around with lib/RT/Action/Notify.pm and
figure this out, but I thought someone might have some quick insight.

I’m using RT 3.4.1 (from the official Debian Sarge package).

Thanks in advance.

–df

2

Dan Fulbright (on rt-users) wrote:

When I “Take” a ticket, I get an e-mail that says, for example:

Transaction: Given to supportguy by supportguy

The problem is, I’m supportguy, so I know I just gave the ticket to
myself. I have this variable unchanged from the default:

Set($NotifyActor, 0);

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 );

I’m cross-posting this to the devel list, so someone can tell me if I
did something I shouldn’t have.

I’m using RT 3.4.1.

–df

3

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 );