[#667 state:resolved] Test $param{mail_transfer} variable in two plac…

…es to suppress uninitialized value warnings on Mail Configuration page in wizard.cgi.
openmelody · Dec 13, 2010 · bcd2058 · bcd2058
1 parent 25a91bf
commit bcd2058
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/MT/App/Wizard.pm b/lib/MT/App/Wizard.pm
@@ -861,12 +861,12 @@ sub optional {
       { id => 'sendmail', name => $app->translate('Sendmail') };
 
     foreach (@$transfer) {
-        if ( $_->{id} eq $param{mail_transfer} ) {
+        if ( $param{mail_transfer} && $_->{id} eq $param{mail_transfer} ) {
             $_->{selected} = 1;
         }
     }
 
-    $param{ 'use_' . $param{mail_transfer} } = 1;
+    $param{ 'use_' . $param{mail_transfer} } = 1 if $param{mail_transfer};
     $param{mail_loop}                        = $transfer;
     $param{config}                           = $app->serialize_config(%param);