Spin off from #591734: Reduce system table queries in the critical path. We already have the information available, saves a query on every page request.

CommentFileSizeAuthor
#4 drupal_591758_system_theme_list_4_d6.patch574 byteshefox
themes.patch989 bytescatch

Comments

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

I am giving this the green light in the hope that stale static caching in list_themes can not cause more problems than we have already given that in D6 system_region_list had no reset and list_themes had so it surely is reset more than s_r_l.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Nice. Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

hefox’s picture

hefox commented
StatusFileSize
new drupal_591758_system_theme_list_4_d6.patch574 bytes

Hm, has needs backport to d6 tag, but wasn't. Is it no longer possible? (I'm a bit unclear on what can't be backported).

Well, anyhow, d6 updated patch, but not going to re-open cause unsure of ^.

hefox’s picture

hefox commented
Version: 7.x-dev » 6.x-dev
Status: Closed (fixed) » Needs review

After noticing that there are performance patches still going in, figured I'd reopen this for d6

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.