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)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: shane.h.w.travis, Assigned: shane.h.w.travis)
Details
Attachments
(3 files)
|
775 bytes,
patch
|
jouni
:
review+
|
Details | Diff | Splinter Review |
|
567 bytes,
patch
|
jouni
:
review+
|
Details | Diff | Splinter Review |
|
461 bytes,
patch
|
jouni
:
review+
|
Details | Diff | Splinter Review |
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.
| Assignee | ||
Comment 1•21 years ago
|
||
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.
| Assignee | ||
Comment 2•21 years ago
|
||
| Assignee | ||
Comment 3•21 years ago
|
||
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?
| Assignee | ||
Updated•21 years ago
|
Attachment #166163 -
Flags: review?(justdave)
| Assignee | ||
Updated•21 years ago
|
Attachment #166164 -
Flags: review?(justdave)
| Assignee | ||
Comment 4•21 years ago
|
||
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!
| Assignee | ||
Updated•21 years ago
|
Attachment #166163 -
Flags: review?(justdave) → review?(jouni)
| Assignee | ||
Updated•21 years ago
|
Attachment #166164 -
Flags: review?(justdave) → review?(jouni)
| Assignee | ||
Updated•21 years ago
|
Attachment #166269 -
Flags: review?(jouni)
Updated•21 years ago
|
Attachment #166163 -
Attachment description: Patch against 2.18rc3 and tip → Patch against 2.18rc3
Updated•21 years ago
|
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
Updated•21 years ago
|
Attachment #166269 -
Flags: review?(jouni) → review+
Updated•21 years ago
|
Attachment #166163 -
Flags: review?(jouni) → review+
Comment 5•21 years ago
|
||
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+
Updated•21 years ago
|
Flags: approval?
Flags: approval2.18?
Flags: approval2.18+
Flags: approval2.16?
Flags: approval2.16+
Flags: approval+
Comment 6•21 years ago
|
||
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
Updated•13 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•