From 27effd99d37c47da4f1462c63cae3d8548624ca2 Mon Sep 17 00:00:00 2001 From: "lpsolit%gmail.com" <> Date: Sat, 7 Jun 2008 18:54:31 +0000 Subject: Bug 437369: Deleting a bug doesn't remove related data from the bugs_fulltext table - Patch by Ed Goose r/a=LpSolit --- Bugzilla/Bug.pm | 4 ++++ sanitycheck.cgi | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) mode change 100755 => 100644 Bugzilla/Bug.pm diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm old mode 100755 new mode 100644 index 736afbcda..ec603e2b2 --- a/Bugzilla/Bug.pm +++ b/Bugzilla/Bug.pm @@ -770,6 +770,7 @@ sub remove_from_db { # - bug_group_map # - bugs # - bugs_activity + # - bugs_fulltext # - cc # - dependencies # - duplicates @@ -811,6 +812,9 @@ sub remove_from_db { $dbh->bz_commit_transaction(); + # The bugs_fulltext table doesn't support transactions. + $dbh->do("DELETE FROM bugs_fulltext WHERE bug_id = ?", undef, $bug_id); + # Now this bug no longer exists $self->DESTROY; return $self; diff --git a/sanitycheck.cgi b/sanitycheck.cgi index 1332f75ed..237eaab38 100755 --- a/sanitycheck.cgi +++ b/sanitycheck.cgi @@ -272,7 +272,8 @@ if ($cgi->param('remove_invalid_bug_references')) { $dbh->bz_start_transaction(); - foreach my $pair ('attachments/', 'bug_group_map/', 'bugs_activity/', 'cc/', + foreach my $pair ('attachments/', 'bug_group_map/', 'bugs_activity/', + 'bugs_fulltext/', 'cc/', 'dependencies/blocked', 'dependencies/dependson', 'duplicates/dupe', 'duplicates/dupe_of', 'flags/', 'keywords/', 'longdescs/', 'votes/') { @@ -408,6 +409,7 @@ CrossCheck("flagtypes", "id", CrossCheck("bugs", "bug_id", ["bugs_activity", "bug_id"], ["bug_group_map", "bug_id"], + ["bugs_fulltext", "bug_id"], ["attachments", "bug_id"], ["cc", "bug_id"], ["longdescs", "bug_id"], -- cgit v1.2.3-24-g4f1b