Search.pm: Fully generate the ORDER BY clause inside of an accessor

RESOLVED FIXED in Bugzilla 4.2

Status

()

Bugzilla
Query/Bug List
--
enhancement
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: Max Kanat-Alexander)

Tracking

3.7.2
Bugzilla 4.2
Bug Flags:
approval +

Details

Attachments

(1 attachment)

v1
8.35 KB, patch
Max Kanat-Alexander
: review+
Details | Diff | Splinter Review
(Assignee)

Description

8 years ago
There's a lot of code spread around Search.pm to generate the ORDER BY clause. It should all be an accessor instead.
(Assignee)

Comment 1

8 years ago
Created attachment 457010 [details] [diff] [review]
v1

This creates _sql_order_by and adjusts various other accessors to Do The Right Thing in relation to generation of the ORDER BY clause.
Assignee: query-and-buglist → mkanat
Status: NEW → ASSIGNED
Attachment #457010 - Flags: review+
(Assignee)

Comment 2

8 years ago
Checked in as revision 7343 on trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Flags: approval+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.