From bd370f377168cc55ff33240a16d574bf7982b5a5 Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Mon, 8 Oct 2007 04:24:46 +0000
Subject: Bug 398707: whine.pl should use transactions for database interaction
 - Patch by Emmanuel Seyman <eseyman@linagora.com> r/a=mkanat

---
 whine.pl | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

(limited to 'whine.pl')

diff --git a/whine.pl b/whine.pl
index 82bfc933b..a809025c0 100755
--- a/whine.pl
+++ b/whine.pl
@@ -206,12 +206,7 @@ sub get_next_event {
     # Loop until there's something to return
     until (scalar keys %{$event}) {
 
-        $dbh->bz_lock_tables('whine_schedules WRITE',
-                             'whine_events READ',
-                             'profiles READ',
-                             'groups READ',
-                             'group_group_map READ',
-                             'user_group_map READ');
+        $dbh->bz_start_transaction();
 
         # Get the event ID for the first pending schedule
         $sth_next_scheduled_event->execute;
@@ -275,7 +270,7 @@ sub get_next_event {
             reset_timer($sid);
         }
 
-        $dbh->bz_unlock_tables();
+        $dbh->bz_commit_transaction();
 
         # Only set $event if the user is allowed to do whining
         if ($owner->in_group('bz_canusewhines')) {
-- 
cgit v1.2.3-24-g4f1b