Closed Bug 151023 Opened 21 years ago Closed 21 years ago

duplicates.cgi sort by delta sorts wrong direction

Categories

(Bugzilla :: Query/Bug List, defect)

2.17
x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: asa, Assigned: gerv)

References

Details

Attachments

(1 file)

when sorting by the list of duplicates by delta the top of the list is the
fewest changes and the bottom of the list is the most changes. This is backwards
from how it used to be and a lot less useful. See this list for an example
http://bugzilla.mozilla.org/duplicates.cgi?sortby=delta&maxrows=20&changedsince=5

I think that most people will want to see the bugs which have gathered the most
duplicates at the top of the list, not the fewest.
Blocks: 150783
Maybe we should sort by the absolute value of the change?
Attached patch Patch v.1Splinter Review
This should be what's wanted.

Gerv
Comment on attachment 87421 [details] [diff] [review]
Patch v.1

r=bbaetz

We may want to consider sorting by absolute value at some point, but thats not
important now.

What happened to bmo's duplicates.cgi?
Attachment #87421 - Flags: review+
b.m.o's duplicates.cgi got taken offline for performance reasons. As a fun
experiment, read the code and see if you can spot the obvious problem :-) Then
check out bug 151281.

Gerv
Yeah, that was the next bug in my list (I look at bugs chronologically). See my
comments there.
Keywords: patch, review
Comment on attachment 87421 [details] [diff] [review]
Patch v.1

>Index: template/en/default/reports/duplicates-table.html.tmpl
>+              [% ELSE %]
>+                 [%-# Some columns start off reversed %]
>+                [% "&reverse=1" IF column.name.match('delta|count') %]

Zap that one extra space before the comment line unless there is a reason for
it. With that, r=jouni.
Attachment #87421 - Flags: review+
Fixed on trunk and branch.

Checking in template/en/default/reports/duplicates-table.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/reports/duplicates-table.html.tmpl,v
 <--  duplicates-table.html.tmpl
new revision: 1.3; previous revision: 1.2
done

Checking in template/en/default/reports/duplicates-table.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/reports/duplicates-table.html.tmpl,v
 <--  duplicates-table.html.tmpl
new revision: 1.1.2.2; previous revision: 1.1.2.1
done

Gerv
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Whiteboard: [applied to b.m.o]
Target Milestone: --- → Bugzilla 2.16
Whiteboard: [applied to b.m.o]
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.