Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 291721

Use !url placeholder in description string in system_token_info

$
0
0

Problem/Motivation

This is a followup to #2339021: Incorrect call to l() in system_token_info to improve this string for translation.

'description' => t('A date in a custom format. See <a href="https://www.php.net/manual/datetime.format.php#refsect1-datetime.format-parameters">the PHP documentation</a> for details.'),

Also, at least in 7.x, calling l() within system_token_info() triggers a theme registry rebuild, which adds 11 seconds to a call to drupal_flush_all_caches() on the site I'm profiling, which is how I found this.

Steps to reproduce

Proposed resolution

Use !url placeholder substitution. See #2339021-11: Incorrect call to l() in system_token_info and #2339021-12: Incorrect call to l() in system_token_info for background.

Remaining tasks

Patch
Review
Commit

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

N/A


Viewing all articles
Browse latest Browse all 291721

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>