Opened 3 years ago

Last modified 2 years ago

#62 new task

Merge getFooCount and getFooList subs

Reported by: kdeugau Owned by:
Priority: minor Milestone: 1.6
Version: Keywords:


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 Changed 3 years ago by kdeugau

(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 Changed 2 years ago by kdeugau

  • Milestone changed from 1.4 to 1.6

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

Note: See TracTickets for help on using tickets.