From: Don Armstrong <don@donarmstrong.com>
Date: Thu, 10 Aug 2017 00:41:20 +0000 (-0700)
Subject: skip empty correspondents
X-Git-Url: https://git.donarmstrong.com/?p=debbugs.git;a=commitdiff_plain;h=e0e2e093dbab85fafa41c4549483aa744fc68014

skip empty correspondents
---

diff --git a/Debbugs/DB/Load.pm b/Debbugs/DB/Load.pm
index d301983..1af6c65 100644
--- a/Debbugs/DB/Load.pm
+++ b/Debbugs/DB/Load.pm
@@ -404,9 +404,11 @@ sub load_bug_log {
 	    my @cors;
 	    for my $type (keys %corr) {
 		for my $addr (@{$corr{$type}}) {
+                    my $cor = $s->resultset('Correspondent')->
+                        get_correspondent_id($addr);
+                    next unless defined $cor;
 		    push @cors,
-			{correspondent => $s->resultset('Correspondent')->
-			 get_correspondent_id($addr),
+			{correspondent => $cor,
 			 correspondent_type => $type,
 			};
 		}
@@ -414,7 +416,8 @@ sub load_bug_log {
 	    $m->update();
 	    $s->txn_do(sub {
 			   $m->message_correspondents()->delete();
-			   $m->add_to_message_correspondents(@cors);
+			   $m->add_to_message_correspondents(@cors) if
+                               @cors;
 		       }
 		      );
 	}