Closed
Bug 770438
Opened 13 years ago
Closed 13 years ago
Allow displaying votes column in buglists
Categories
(Bugzilla :: Extensions, enhancement)
Bugzilla
Extensions
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: tpr, Assigned: tpr)
Details
Attachments
(1 file)
|
472 bytes,
patch
|
Wurblzap
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20100101 Firefox/11.0
Build ID: 20120313130609
Steps to reproduce:
Wanted to sort search results by votes count.
Actual results:
Votes column is not available in Edit columns at all.
Expected results:
User should be allowed to see the column and sort by it. The attached patch makes this possible.
Comment 1•13 years ago
|
||
Comment on attachment 638616 [details] [diff] [review]
Patch implementing buglist_columns for voting extension
Putting this patch is reviewers' radar.
Attachment #638616 -
Flags: review?
Updated•13 years ago
|
Severity: normal → enhancement
Summary: [PATCH] Allow displaying votes column in bug listings → Allow displaying votes column in buglists
Comment 2•13 years ago
|
||
Comment on attachment 638616 [details] [diff] [review]
Patch implementing buglist_columns for voting extension
Nice :)
r=Wurblzap
Attachment #638616 -
Flags: review? → review+
Updated•13 years ago
|
Assignee: extensions → tpr
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: approval?
Target Milestone: --- → Bugzilla 4.4
Comment 3•13 years ago
|
||
You tested the patch, isn't it? And clicking on this column header to sort the buglist works correctly?
Flags: approval? → approval+
Comment 4•13 years ago
|
||
I'm sorry, but this patch is not needed. I just tested with a clean 4.3 DB and an upgraded 4.3 DB, and in both cases is the "Votes" column already available and data is displayed correctly.
Marc: was this patch really needed to see the Votes column in your testing?
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Flags: approval+ → approval-
Resolution: --- → WORKSFORME
Target Milestone: Bugzilla 4.4 → ---
Comment 5•13 years ago
|
||
I, for one, didn't test.
The patch conforms to the example Extension; field-descs-end.none.tmpl doesn't cut it, does it?
tpr: please reopen if this is still not working for you on a fresh install.
You need to log in
before you can comment on or make changes to this bug.
Description
•