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"};
--
Gene LeDuc, GSEC
Security Analyst
San Diego State University
_______________________________________________
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
