Gene,

Sorry for all the hassle. In the Perl class I took, I got the impression that it didn't matter what type of brackets you used, as long as they matched left & right. Also, why do you use == in the second line? Why not just another eq? Another question; I was under the impression that if the code failed, nothing happened. In my case it seems that the code was wrong, but I was getting a notification no matter what. That doesn't make sense.

Kenn

Gene LeDuc wrote:
Kenn, you're using curly brackets to enclose the condition phrase. In this case you really don't need to enclose it at all, but if you do then you need to use parentheses.

This should work:
return 1 if $trans->Type eq "CustomField" && $trans->Field == get_custom_id("Approval-Status") && $trans->NewValue eq "Reviewing Request";

This too:
return 1 if ($trans->Type eq "CustomField" && $trans->Field == get_custom_id("Approval-Status") && $trans->NewValue eq "Reviewing Request");

Also:
return ($trans->Type eq "CustomField" && $trans->Field == get_custom_id("Approval-Status") && $trans->NewValue eq "Reviewing Request");

All assuming that the CF name and Value are as stated and that you have the get_custom_id sub at the end of your code.

At 04:30 PM 5/17/2007, Kenneth Crocker wrote:
return 1 if {$trans->Type eq "CustomField" &&
             $trans->Field == get_custom_id('Approval-Status') &&
             $trans->NewValue eq "Reviewing Request"};


_______________________________________________
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