summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authortimeless%mozdev.org <>2003-01-23 16:43:06 +0100
committertimeless%mozdev.org <>2003-01-23 16:43:06 +0100
commit551081042333edc563ebccc1a10e0b5805b51858 (patch)
tree8db5295891743fe30996ceeace766ea9034f3c91
parent814dd54078a31cab2a51c99cf22a403d3b91e5c2 (diff)
downloadbugzilla-551081042333edc563ebccc1a10e0b5805b51858.tar.gz
bugzilla-551081042333edc563ebccc1a10e0b5805b51858.tar.xz
Bug 190197 AnyEntryGroups() is broken in globals.pl; call from enter_bug.cgi breaks bug entering
patch by jon@vmware.com r=justdave a=justdave
-rw-r--r--globals.pl13
1 files changed, 8 insertions, 5 deletions
diff --git a/globals.pl b/globals.pl
index 5ad624f9a..8162b9d21 100644
--- a/globals.pl
+++ b/globals.pl
@@ -474,12 +474,15 @@ sub AnyEntryGroups {
$query .= " AND product_id = $product_id" if ($product_id);
$query .= " LIMIT 1";
SendSQL($query);
- $::CachedAnyEntryGroups{$product_id} = MoreSQLData();
- FetchSQLData();
- PopGlobalSQLState();
- return $::CachedAnyEntryGroups{$product_id};
+ if (MoreSQLData()) {
+ $::CachedAnyEntryGroups{$product_id} = MoreSQLData();
+ FetchSQLData();
+ PopGlobalSQLState();
+ return $::CachedAnyEntryGroups{$product_id};
+ } else {
+ return undef;
+ }
}
-
#
# This function checks if there are any default groups defined.
# If so, then groups may have to be changed when bugs move from