Minor Edit to Interface/Email.pm

(I don’t have the original so I can’t do a diff, but basically the
loop variable should (always) be my’ed to protect it from being
altered. There was a problem with the MailFrom plugin setting $_ and
then corrupting the loop varialbe!)

----CUT—

Since this needs loading, no matter what

foreach my $Class (@RT::MailPlugins) {
my ($Code, $NewAuthStat);
if ( ref($Class) eq “CODE” ) {
$Code = $Class;
} else {
$Class = “RT::Interface::email::” . $Class
unless $Class =~ /^RT::Interface::email::/;

You don’t say what version you’re running (probably 3.6.something)

It would be helpful if you sent a patch (you can download a fresh copy
of
3.6 from http://download.bestpractical.com ) and an explanation
of what bug you were seeing and how to reproduce it to
rt-bugs@bestpractical.com

This would let us get a fix in the svn tree for future releases

thanks

-kevinOn Aug 7, 2008, at 1:35 PM, Erik Aronesty wrote:

(I don’t have the original so I can’t do a diff, but basically the
loop variable should (always) be my’ed to protect it from being
altered. There was a problem with the MailFrom plugin setting $_ and
then corrupting the loop varialbe!)

----CUT—

Since this needs loading, no matter what

foreach my $Class (@RT::MailPlugins) {
my ($Code, $NewAuthStat);
if ( ref($Class) eq “CODE” ) {
$Code = $Class;
} else {
$Class = “RT::Interface::email::” . $Class
unless $Class =~ /^RT::Interface::email::/;


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

Community help: http://wiki.bestpractical.com
Commercial support: sales@bestpractical.com

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