diff options
author | jocuri%softhome.net <> | 2003-10-19 17:46:39 +0200 |
---|---|---|
committer | jocuri%softhome.net <> | 2003-10-19 17:46:39 +0200 |
commit | 491f40fc7d1d671db0aea71ff09a088a89333304 (patch) | |
tree | 0ec91d717089128c53b06431dd929c4702e490c6 /process_bug.cgi | |
parent | 1fb505eb4677ff52a7e03e75996812d687df8ff5 (diff) | |
download | bugzilla-491f40fc7d1d671db0aea71ff09a088a89333304.tar.gz bugzilla-491f40fc7d1d671db0aea71ff09a088a89333304.tar.xz |
Bug 218977: "Table 'namedqueries' was not locked with LOCK TABLES" on ThrowUserError('product_edit_denied'); r=bbaetz; a=justdave.
Diffstat (limited to 'process_bug.cgi')
-rwxr-xr-x | process_bug.cgi | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/process_bug.cgi b/process_bug.cgi index e504fd7ea..e9b665c3a 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -1143,7 +1143,7 @@ foreach my $id (@idlist) { $oldhash{'product'} = get_product_name($oldhash{'product_id'}); if (!CanEditProductId($oldhash{'product_id'})) { ThrowUserError("product_edit_denied", - { product => $oldhash{'product'} }); + { product => $oldhash{'product'} }, "abort"); } if (defined $::FORM{'product'} @@ -1151,7 +1151,7 @@ foreach my $id (@idlist) { && $::FORM{'product'} ne $oldhash{'product'} && !CanEnterProduct($::FORM{'product'})) { ThrowUserError("entry_access_denied", - { product => $::FORM{'product'} }); + { product => $::FORM{'product'} }, "abort"); } if ($requiremilestone) { my $value = $::FORM{'target_milestone'}; @@ -1161,7 +1161,6 @@ foreach my $id (@idlist) { SendSQL("SELECT defaultmilestone FROM products WHERE name = " . SqlQuote($oldhash{'product'})); if ($value eq FetchOneColumn()) { - SendSQL("UNLOCK TABLES"); ThrowUserError("milestone_required", { bug_id => $id }, "abort"); |