Opened 8 years ago

Last modified 7 years ago

#62 new task

Merge getFooCount and getFooList subs

Reported by: Kris Deugau Owned by:
Priority: minor Milestone: 1.6
Version: Keywords:
Cc:

Description

The ...Count() and ...List() subs are getting increasingly complex, with ever-growing sets of requirements, limits, and hairy SQL.

Rather than hand-duplicate changes in two places, try merging the two, with a "count" flag to pick whether we do "SELECT count(*) ..." or "SELECT fielda,fieldb....".

This is more complex in part, but it will *reduce* overall complexity by removing duplication of complicated conditionals, limits, and SQL fragments.

Change History (2)

comment:1 by Kris Deugau, 8 years ago

(In [733]) /trunk

Add log filtering supporting redirecting to a "base" log set for user,
domain, or revzone, and a free text field to filter the entries. Closes #17.

While we're meddling with the log entry retrieval, start an experiment
to merge getFooCount() and getFooList() subs. See #62.

comment:2 by Kris Deugau, 7 years ago

Milestone: 1.41.6

Bumping to 1.6 since this is more of an ongoing code cleanup thing

Note: See TracTickets for help on using tickets.