Changes between Initial Version and Version 3 of Ticket #51678
- Timestamp:
- 10/30/2020 02:05:22 PM (5 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #51678
-
Property
Component
changed from
GeneraltoI18N -
Property
Priority
changed from
normaltohigh -
Property
Version
changed from
totrunk -
Property
Milestone
changed from
Awaiting Reviewto5.6 - Property Keywords needs-testing added
-
Property
Component
changed from
-
Ticket #51678 – Description
initial v3 1 In WooCommerce's internal aut mated performance testing, we are seeing performance degradations in all paths where any plugin is loaded whose translation file is not present. Root cause seems to in #49236, where we added some changes and introduced WP_Textdomain_Registry to store text domains and their language directory paths.1 In WooCommerce's internal automated performance testing, we are seeing performance degradations in all paths where any plugin is loaded whose translation file is not present. Root cause seems to in [https://core.trac.wordpress.org/changeset/49236 changeset 49236], where we added some changes and introduced WP_Textdomain_Registry to store text domains and their language directory paths. 2 2 3 3 Looks like, if translation file is not present for a domain, we hit the [https://core.trac.wordpress.org/browser/trunk/src/wp-includes/l10n.php?rev=49236#L742 is_readable] on every l10n call. While cached internally, it seems like this call is still slow on many hosts and it will be good to add additional caching around it.