diff options
author | lpsolit%gmail.com <> | 2007-11-19 03:20:53 +0100 |
---|---|---|
committer | lpsolit%gmail.com <> | 2007-11-19 03:20:53 +0100 |
commit | 9d6205cdb651fe3769ac58f03068dedb68e7689a (patch) | |
tree | 47414a6cf3f484e27a74fe91842ebe08843bb2c5 /Bugzilla/Search/Saved.pm | |
parent | 0dadaed4ffb582ccfbc2356e7715007819d1cea2 (diff) | |
download | bugzilla-9d6205cdb651fe3769ac58f03068dedb68e7689a.tar.gz bugzilla-9d6205cdb651fe3769ac58f03068dedb68e7689a.tar.xz |
Bug 399163: Bugzilla/*.pm should use transactions for database interaction - Patch by Emmanuel Seyman <eseyman@linagora.com> r/a=mkanat
Diffstat (limited to 'Bugzilla/Search/Saved.pm')
-rw-r--r-- | Bugzilla/Search/Saved.pm | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Bugzilla/Search/Saved.pm b/Bugzilla/Search/Saved.pm index bcc72cffe..3484eb3bc 100644 --- a/Bugzilla/Search/Saved.pm +++ b/Bugzilla/Search/Saved.pm @@ -97,13 +97,12 @@ sub create { Bugzilla->login(LOGIN_REQUIRED); my $dbh = Bugzilla->dbh; $class->check_required_create_fields(@_); + $dbh->bz_start_transaction(); my $params = $class->run_create_validators(@_); # Right now you can only create a Saved Search for the current user. $params->{userid} = Bugzilla->user->id; - $dbh->bz_lock_tables('namedqueries WRITE', - 'namedqueries_link_in_footer WRITE'); my $lif = delete $params->{link_in_footer}; my $obj = $class->insert_create_data($params); if ($lif) { @@ -111,7 +110,7 @@ sub create { (user_id, namedquery_id) VALUES (?,?)', undef, $params->{userid}, $obj->id); } - $dbh->bz_unlock_tables(); + $dbh->bz_commit_transaction(); return $obj; } |