This is berdir's patch, please credit him not me.

CommentFileSizeAuthor
timezone_split.patch18.17 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch’s picture

Status: Needs review » Fixed

Taking that as an RTBC from chx for berdir's patch, and committed/pushed this to 8.x to unblock #1187726: Add caching for configuration / rework config object loading (Was: Memory usage and i/o from config objects).

sun’s picture

Component: configuration system » system.module
Issue tags: +Configuration system

I do not understand this change. Can someone explain why this has been done?

catch’s picture

It's a stop-gap workaround for #1859110: Circular dependencies and hidden bugs with configuration overrides which would otherwise be a hard blocker for #1187726: Add caching for configuration / rework config object loading (Was: Memory usage and i/o from config objects).

Also there may well be requests where the timezone is needed, but not date formats at all (i.e. page cache hits at the moment), so it doesn't really make sense to load the data formats when you only need the timezone anyway.

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