From f95d29501ddc10e788eac9859cf57bdb59ac29d3 Mon Sep 17 00:00:00 2001 From: Dylan William Hardison Date: Tue, 26 Aug 2014 09:19:04 -0400 Subject: Bug 1056087 - contrib/merge-users.pl fails if there are no duplicate bug_user_last_visit rows r/a=glob --- contrib/merge-users.pl | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'contrib') diff --git a/contrib/merge-users.pl b/contrib/merge-users.pl index 28a378e1c..aeb8156ad 100755 --- a/contrib/merge-users.pl +++ b/contrib/merge-users.pl @@ -171,12 +171,17 @@ my $dupe_ids = $dbh->selectcol_arrayref(" SELECT earlier.id FROM bug_user_last_visit as earlier INNER JOIN bug_user_last_visit as later - ON (earlier.user_id != later.user_id AND earlier.last_visit_ts < later.last_visit_ts + ON (earlier.user_id != later.user_id + AND earlier.last_visit_ts < later.last_visit_ts AND earlier.bug_id = later.bug_id) WHERE (earlier.user_id = ? OR earlier.user_id = ?) AND (later.user_id = ? OR later.user_id = ?)", undef, $old_id, $new_id, $old_id, $new_id); -$dbh->do("DELETE FROM bug_user_last_visit WHERE " . $dbh->sql_in('id', $dupe_ids)); + +if (@$dupe_ids) { + $dbh->do("DELETE FROM bug_user_last_visit WHERE " . + $dbh->sql_in('id', $dupe_ids)); +} # Migrate records from old user to new user. foreach my $table (keys %changes) { -- cgit v1.2.3-24-g4f1b