Bug 301020: Remove useless locked tables - Patch by Frédéric Buclin <LpSolit@gmail...

diff --git a/editcomponents.cgi b/editcomponents.cgi
index 601873e83a7323fe3cb4064a3f72328e89cae602..30caaf5f3a62edfaf283158c076e5a7753b59525 100755 (executable)
--- a/editcomponents.cgi
+++ b/editcomponents.cgi
@@ -359,8 +359,7 @@ if ($action eq 'update') {
     my $default_assignee_id   = login_to_id($default_assignee);
     my $default_qa_contact_id = login_to_id($default_qa_contact) || undef;
 
-    $dbh->bz_lock_tables('components WRITE', 'products READ',
-                         'profiles READ');
+    $dbh->bz_lock_tables('components WRITE', 'profiles READ');
 
     if ($comp_name ne $component_old->name) {
 

diff --git a/editgroups.cgi b/editgroups.cgi
index d28d95f19f9b57a9d48e310c942b714ab78ee6d9..9c1280d5cdeceb4a99faba4a93bb17752a9c6e94 100755 (executable)
--- a/editgroups.cgi
+++ b/editgroups.cgi
@@ -565,8 +565,7 @@ sub doGroupChanges {
 
     $dbh->bz_lock_tables('groups WRITE', 'group_group_map WRITE',
                          'bug_group_map WRITE', 'user_group_map WRITE',
-                         'group_control_map READ', 'bugs READ', 'profiles READ',
-                         'namedqueries READ', 'whine_queries READ');
+                         'group_control_map READ', 'bugs READ', 'profiles READ');
 
     # Check that the given group ID and regular expression are valid.
     # If tests are successful, trimmed values are returned by CheckGroup*.

diff --git a/editproducts.cgi b/editproducts.cgi
index 30b3d32a6ee3fafc7f90cf15d74effed95d277fe..794caf158e3ac8f8a225b858ad560068c5cbe64b 100755 (executable)
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -806,12 +806,7 @@ if ($action eq 'update') {
                        {votestoconfirm => $stored_votestoconfirm});
     }
 
-    $dbh->bz_lock_tables('products WRITE',
-                         'versions READ',
-                         'groups WRITE',
-                         'group_control_map WRITE',
-                         'profiles WRITE',
-                         'milestones READ');
+    $dbh->bz_lock_tables('products WRITE', 'milestones READ');
 
     my $testproduct = 
         new Bugzilla::Product({name => $product_name});

diff --git a/editusers.cgi b/editusers.cgi
index 6e56a384780f84e5e0e8e8108beb5e9e8015000b..639e5ee1c02752fe60295817ba2f1e5bf441ef9a 100755 (executable)
--- a/editusers.cgi
+++ b/editusers.cgi
@@ -199,13 +199,8 @@ if ($action eq 'search') {
     my $disabledtext = trim($cgi->param('disabledtext') || '');
 
     # Lock tables during the check+creation session.
-    $dbh->bz_lock_tables('profiles WRITE',
-                         'profiles_activity WRITE',
-                         'groups READ',
-                         'user_group_map WRITE',
-                         'email_setting WRITE',
-                         'namedqueries READ',
-                         'whine_queries READ',
+    $dbh->bz_lock_tables('profiles WRITE', 'email_setting WRITE',
+                         'user_group_map WRITE', 'groups READ',
                          'tokens READ');
 
     # Validity checks
@@ -249,13 +244,10 @@ if ($action eq 'search') {
     $dbh->bz_lock_tables('profiles WRITE',
                          'profiles_activity WRITE',
                          'fielddefs READ',
-                         'namedqueries READ',
-                         'whine_queries READ',
                          'tokens WRITE',
                          'logincookies WRITE',
                          'groups READ',
                          'user_group_map WRITE',
-                         'user_group_map AS ugm READ',
                          'group_group_map READ',
                          'group_group_map AS ggm READ');
  
@@ -528,10 +520,8 @@ if ($action eq 'search') {
                          'profiles_activity WRITE',
                          'email_setting WRITE',
                          'profile_setting WRITE',
-                         'groups READ',
                          'bug_group_map READ',
                          'user_group_map WRITE',
-                         'group_group_map READ',
                          'flags WRITE',
                          'flagtypes READ',
                          'cc WRITE',

diff --git a/editversions.cgi b/editversions.cgi
index dc6992ba8e9146b84860de61fe01e3c6cad24a31..dc03bfed98097979b00c470eaaf5232d62d945d1 100755 (executable)
--- a/editversions.cgi
+++ b/editversions.cgi
@@ -255,9 +255,7 @@ if ($action eq 'update') {
     # Note that the order of this tests is important. If you change
     # them, be sure to test for WHERE='$version' or WHERE='$versionold'
 
-    $dbh->bz_lock_tables('bugs WRITE',
-                         'versions WRITE',
-                         'products READ');
+    $dbh->bz_lock_tables('bugs WRITE', 'versions WRITE');
 
     if ($version_name ne $version_old->name) {
         

diff --git a/process_bug.cgi b/process_bug.cgi
index f19763c237d79ebd6d6c02ffb6a4f0f167ae7ab7..c1f3b9b9048475fa45c1a5ee39a906f25682b0e1 100755 (executable)
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -1471,11 +1471,10 @@ foreach my $id (@idlist) {
     $bug_changed = 0;
     my $write = "WRITE";        # Might want to make a param to control
                                 # whether we do LOW_PRIORITY ...
-    $dbh->bz_lock_tables("bugs $write", "bugs_activity $write",
-            "cc $write", "cc AS selectVisible_cc $write",
+    $dbh->bz_lock_tables("bugs $write", "bugs_activity $write", "cc $write",
             "profiles READ", "dependencies $write", "votes $write",
             "products READ", "components READ",
-            "keywords $write", "longdescs $write", "fielddefs $write",
+            "keywords $write", "longdescs $write", "fielddefs READ",
             "bug_group_map $write", "flags $write", "duplicates $write",
             "user_group_map READ", "group_group_map READ", "flagtypes READ",
             "flaginclusions AS i READ", "flagexclusions AS e READ",

diff --git a/votes.cgi b/votes.cgi
index b7a53eca8e018a0dbcc5633bc66c5a22e71da5f4..e1bfb36164f210c496eb47f29f50041e257d79c1 100755 (executable)
--- a/votes.cgi
+++ b/votes.cgi
@@ -126,8 +126,7 @@ sub show_user {
 
     $dbh->bz_lock_tables('bugs READ', 'products READ', 'votes WRITE',
              'cc READ', 'bug_group_map READ', 'user_group_map READ',
-             'group_group_map READ',
-             'cc AS selectVisible_cc READ', 'groups READ');
+             'group_group_map READ', 'groups READ');
 
     if ($canedit && $bug_id) {
         # Make sure there is an entry for this bug