Closed Bug 270263 Opened 21 years ago Closed 21 years ago

'Votes' should not be a column choice if param(usevotes) is not true

Categories

(Bugzilla :: Query/Bug List, defect)

2.18
defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: shane.h.w.travis, Assigned: shane.h.w.travis)

Details

Attachments

(3 files)

When in colchange.cgi, users are offered 'Votes' as a potential column even if their site has 'usevotes' turned off. This is a bug, and should not be happening; instead, it should be treated in the same way that bug aliases, time tracking, QA contact, etc. are treated - you can choose to display them if your site uses the feature. Patches forthcoming.
Small change, easily made. Fortunately, 'Votes' was already at the end of the @masterlist, so there will be *no* visual difference for the end-user.
Requesting the same flags for this one as for 267494, since this is exactly the same sort of issue.
Status: NEW → ASSIGNED
Flags: blocking2.20?
Flags: blocking2.16.8?
Flags: approval?
Flags: approval2.18?
Flags: approval2.16?
Attachment #166163 - Flags: review?(justdave)
Attachment #166164 - Flags: review?(justdave)
218+tip patch is actually only against 2.18rc3; tip has a change right where I was making mine that conflicts. Use this one instead for tip. (Other one still works for 2.18) D'oh!
Attachment #166163 - Flags: review?(justdave) → review?(jouni)
Attachment #166164 - Flags: review?(justdave) → review?(jouni)
Attachment #166269 - Flags: review?(jouni)
Attachment #166163 - Attachment description: Patch against 2.18rc3 and tip → Patch against 2.18rc3
Flags: blocking2.20?
Flags: blocking2.20-
Flags: blocking2.18-
Flags: blocking2.16.8?
Flags: blocking2.16.8-
Whiteboard: not blocking releases, but will accept for 2.16 and 2.18 branches
Target Milestone: --- → Bugzilla 2.16
Attachment #166269 - Flags: review?(jouni) → review+
Attachment #166163 - Flags: review?(jouni) → review+
Comment on attachment 166164 [details] [diff] [review] Patch against 2.16.7 r=jouni for the third time. I only actually tested the tip patch, but visually inspected and patched/runtested the 2.16/2.18 ones.
Attachment #166164 - Flags: review?(jouni) → review+
Flags: approval?
Flags: approval2.18?
Flags: approval2.18+
Flags: approval2.16?
Flags: approval2.16+
Flags: approval+
Checking in colchange.cgi; /cvsroot/mozilla/webtools/bugzilla/colchange.cgi,v <-- colchange.cgi new revision: 1.44; previous revision: 1.43 done Checking in colchange.cgi; /cvsroot/mozilla/webtools/bugzilla/colchange.cgi,v <-- colchange.cgi new revision: 1.41.2.2; previous revision: 1.41.2.1 done Checking in colchange.cgi; /cvsroot/mozilla/webtools/bugzilla/colchange.cgi,v <-- colchange.cgi new revision: 1.24.2.4; previous revision: 1.24.2.3 done
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Whiteboard: not blocking releases, but will accept for 2.16 and 2.18 branches
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: