closes #559

justingit · Apr 19, 2016 · 4271b68 · 4271b68
1 parent 60f7a51
commit 4271b68
Showing 1 changed file with 16 additions and 16 deletions.
diff --git a/dada/DADA/Mail/Send.pm b/dada/DADA/Mail/Send.pm
@@ -874,24 +874,24 @@ sub _massage_fields_for_amazon_ses {
 sub mail_sending_options_test {
 
     my $self = shift;
-
     my $report = [];
 
-	my ($n_p_t_status, $n_p_t_msg);
-	try {
-		($n_p_t_status, $n_p_t_msg) 
-			= $self->net_ping_test(
-				$self->{ls}->param('smtp_server'), 
-				$self->{ls}->param('smtp_port')
-		);
-	} catch { 
-		warn $_;
-	};
-
-	push(@$report, { 
-    	line    => '',
-        message => $n_p_t_msg,
-	});
+	if($self->{ls}->param('sending_method') eq 'smtp' ) {
+		my ($n_p_t_status, $n_p_t_msg);
+		try {
+			($n_p_t_status, $n_p_t_msg) 
+				= $self->net_ping_test(
+					$self->{ls}->param('smtp_server'), 
+					$self->{ls}->param('smtp_port')
+			);
+		} catch { 
+			warn $_;
+		};
+		push(@$report, { 
+	    	line    => '',
+	        message => $n_p_t_msg,
+		});
+	}
 
     require DADA::Security::Password;