143547 - "secure bugs" also grey in list (.bz_secure set) even if usebuggroups is on

Status: VERIFIED FIXED

(Bugzilla :: User Interface, defect)

Description

[Robert Kaiser]

css/buglist.css says:

/* Style secure bugs if the installation is not using bug groups.
 * Installations that *are* using bug groups are likely to be using
 * them for almost all bugs, in which case special styling is not
 * informative and generally a nuisance.
 */
.bz_secure { color: black; background-color: lightgrey; }

I'm using bug groups, having usebuggroups set to "On" in parameters.
bug_list.cgi is setting bz_secure though and so all bugs in my bug lists are
always shown with grey background :(

Comment 1

[Dave Miller [:justdave]]

That's on purpose.  Not every product has to be secure, even if you have
usebuggroups on.  If you do have every product secured on your install and would
rather not have the gray background, change your local stylesheet.

Comment 2

[Dave Miller [:justdave]]

I just re-read the comment in the original description here copied/pasted from
the css file....

According to that comment, something's broken.  Either the comment needs to be
fixed for reality or buglist.cgi needs to be fixed to prime the template classes
according to the comment.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: fixes problem

This is a regression caused by a typo.	This patch fixes the problem.

Comment 4

[Gervase Markham [:gerv]]

Comment on attachment 83106 [details] [diff] [review]
patch v1: fixes problem

2xr=gerv. This is obviously right.

Gerv

Comment 5

[Myk Melez [:myk] [@mykmelez]]

Checked in on trunk and branch:

Checking in buglist.cgi;
/cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v  <--  buglist.cgi
new revision: 1.171; previous revision: 1.170
done

Checking in buglist.cgi;
/cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v  <--  buglist.cgi
new revision: 1.169.2.2; previous revision: 1.169.2.1
done

Comment 6

[Robert Kaiser]

I see it working with current branch cvs in my installation. Thanks all!
vefified.