Follow-up to #2572929: Document lack of auto-escape in theme functions and add a theme autoescape helper function
Problem/Motivation
#2572929: Document lack of auto-escape in theme functions and add a theme autoescape helper function is adding a theme_escape_and_render() helper function as a BC layer to help theme functions do autoescaping. To avoid coupling the Drupal theme system to Twig, this function is separate from TwigExtension::escapeFilter(). However, since it performs most of the same functionality, it shares a lot of code.
Proposed resolution
Refactor the overlapped parts so these two functions can reuse the same code and keep them in sync.
Remaining tasks
TBD.
User interface changes
None.
API changes
Should be none; internal refactoring with a possible internal-ish API addition.
Data model changes
None.
Comments
Comment #2
dawehnerThanks for filing the issue!
Comment #3
joelpittetComment #4
joelpittetWe've hit RC so let's move this to 8.1.x
Comment #20
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #21
xjmAmending attribution.
Comment #22
smustgrave commented@xjm so still valid?