Refix bug 80289. Don't print header for groups if there aren't any groups to display (was printing a header without groups in rare instances).

Patch by Joe Robins <jmrobins@tgix.com>
r= jake@acutex.net

1 files changed, 6 insertions, 4 deletions

diff --git a/enter_bug.cgi b/enter_bug.cgi
index c78628a6c..c1f4e61b1 100755
--- a/enter_bug.cgi
+++ b/enter_bug.cgi
@@ -413,16 +413,18 @@ if ($::usergroupset ne '0') {
             "  AND isbuggroup != 0 AND isactive = 1 ORDER BY description");
     # We only print out a header bit for this section if there are any
     # results.
-    if(MoreSQLData()) {
-      print "<br><b>Only users in the selected groups can view this bug:</b><br>\n";
-      print "<font size=\"-1\">(Leave all boxes unchecked to make this a public bug.)</font><br><br>\n";
-    }
+    my $groupFound = 0;
     while (MoreSQLData()) {
         my ($bit, $prodname, $description) = (FetchSQLData());
         # Don't want to include product groups other than this product.
         unless(($prodname eq $product) || (!defined($::proddesc{$prodname}))) {
             next;
         }
+        if(!$groupFound) {
+          print "<br><b>Only users in the selected groups can view this bug:</b><br>\n";
+          print "<font size=\"-1\">(Leave all boxes unchecked to make this a public bug.)</font><br><br>\n";
+          $groupFound = 1;
+        }
         # Rather than waste time with another Param check and another database
         # access, $group_bit will only have a non-zero value if we're using
         # bug groups and have  one for this product, so I'll check on that