Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This is berdir's patch, please credit him not me.
Comment | File | Size | Author |
---|---|---|---|
timezone_split.patch | 18.17 KB | chx | |
This is berdir's patch, please credit him not me.
Comment | File | Size | Author |
---|---|---|---|
timezone_split.patch | 18.17 KB | chx | |
Comments
Comment #1
catchTaking 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).
Comment #2
sunI do not understand this change. Can someone explain why this has been done?
Comment #3
catchIt'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.