diff options
author | lpsolit%gmail.com <> | 2008-06-07 20:54:31 +0200 |
---|---|---|
committer | lpsolit%gmail.com <> | 2008-06-07 20:54:31 +0200 |
commit | 27effd99d37c47da4f1462c63cae3d8548624ca2 (patch) | |
tree | 7dd003f0239db18a5137bfa70b9b5c757167338d | |
parent | 87cdd1c7afb9c11e2d2fdc246f79c3a0e42de0e8 (diff) | |
download | bugzilla-27effd99d37c47da4f1462c63cae3d8548624ca2.tar.gz bugzilla-27effd99d37c47da4f1462c63cae3d8548624ca2.tar.xz |
Bug 437369: Deleting a bug doesn't remove related data from the bugs_fulltext table - Patch by Ed Goose <ed.goose@gmail.com> r/a=LpSolit
-rw-r--r--[-rwxr-xr-x] | Bugzilla/Bug.pm | 4 | ||||
-rwxr-xr-x | sanitycheck.cgi | 4 |
2 files changed, 7 insertions, 1 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm index 736afbcda..ec603e2b2 100755..100644 --- 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"], |