Closed Bug 578278 Opened 14 years ago Closed 14 years ago

Search.pm: Fully generate @fields (the SELECTed columns) inside of an accessor

Categories

(Bugzilla :: Query/Bug List, enhancement)

3.7.2
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.2

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

Although most of the uses of @fields were in determining the extra @supptables to add to the FROM clause, there are a few other places in init() that @fields is used, and we can move it all into an accessor or two.
Attached patch v1Splinter Review
This creates select_columns and _sql_select, which fully generate the SQL required for the SELECT clause inside of methods.
Assignee: query-and-buglist → mkanat
Status: NEW → ASSIGNED
Attachment #457006 - Flags: review+
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Search.pm
Committed revision 7342.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: approval+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: