aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--Bugzilla/Bug.pm15
-rw-r--r--Bugzilla/DB/Schema/Mysql.pm2
2 files changed, 5 insertions, 12 deletions
diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index ebf00edf3..d4e022336 100644
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -838,13 +838,10 @@ sub create {
Bugzilla::Hook::process('bug_end_of_create',
{bug => $bug, timestamp => $timestamp,});
- $dbh->bz_commit_transaction();
-
- # Because MySQL doesn't support transactions on the fulltext table,
- # we do this after we've committed the transaction. That way we're
- # sure we're inserting a good Bug ID.
$bug->_sync_fulltext(new_bug => 1);
+ $dbh->bz_commit_transaction();
+
return $bug;
}
@@ -1211,17 +1208,13 @@ sub update {
delete $user->{bugs_ignored} if $bug_ignored_changed;
}
- $dbh->bz_commit_transaction();
-
- # The only problem with this here is that update() is often called
- # in the middle of a transaction, and if that transaction is rolled
- # back, this change will *not* be rolled back. As we expect rollbacks
- # to be extremely rare, that is OK for us.
$self->_sync_fulltext(
update_short_desc => $changes->{short_desc},
update_comments => $self->{added_comments} || $self->{comment_isprivate}
);
+ $dbh->bz_commit_transaction();
+
# Remove obsolete internal variables.
delete $self->{'_old_assigned_to'};
delete $self->{'_old_qa_contact'};
diff --git a/Bugzilla/DB/Schema/Mysql.pm b/Bugzilla/DB/Schema/Mysql.pm
index fe2191486..b5bebad30 100644
--- a/Bugzilla/DB/Schema/Mysql.pm
+++ b/Bugzilla/DB/Schema/Mysql.pm
@@ -85,7 +85,7 @@ use constant REVERSE_MAPPING => {
# as in their db-specific version, so no reverse mapping is needed.
};
-use constant MYISAM_TABLES => qw(bugs_fulltext);
+use constant MYISAM_TABLES => qw();
#------------------------------------------------------------------------------
sub _initialize {