Bug 415652

Cause: QN-ReQ

• Evidence: "So we should either have a separate Bugzilla->get_active_custom_fields() which would cache this list of fields, or have Bugzilla->get_fields() cache all fields and do the matching itself instead of letting the DB do it", comment 1

Fix: BQ

• "So we should either have a separate Bugzilla->get_active_custom_fields() which would cache this list of fields, or have Bugzilla->get_fields() cache all fields and do the matching itself instead of letting the DB do it"
• "Implement Bugzilla->get_active_custom_fields and remove the no-longer used Bugzilla->custom_field_names method"

Interesting Finding:

• Developer may not be aware of the optimizations in the database side
  • "In actuality this is not a performance problem at all for most installations, due to the DB's query cache, but yes, it should definitely be handled for architectural reasons"