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

_______________________________________________
http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-users

Community help: http://wiki.bestpractical.com
Commercial support: [EMAIL PROTECTED]


Discover RT's hidden secrets with RT Essentials from O'Reilly Media. Buy a copy at http://rtbook.bestpractical.com

Reply via email to