[rt-users] Trouble with notification scrip

[Stephen Turner]

At Friday 5/18/2007 01:19 PM, Kenneth Crocker wrote:
>Stephan,
>
>
>         A question; why do you have ($trans->NewValue
>          && $trans->NewValue eq "Reviewing Request")
>
>         in the code? it looks like $trans->NewValue is in there 
> twice. I don't understand why it is not just
>
>         if ($trans->Field == $cf->id &&
>             $trans->NewValue eq "Reviewing Request" )
>             return 1;
>
>         I'm learning here, so please be patient. Thanks.
>
>Kenn
>LBNL
>

Hi Kenn,

>No problem - you're probably right - that first bit ( 
>$trans->NewValue )  may be superfluous. I was checking for null 
>before doing the comparison, but I don't think Perl would object if 
>that null check weren't there.

Steve