rename renderPlain to something by making renderPlain a wrapper and adding another better named method

Created on 3 October 2015, about 9 years ago
Updated 5 March 2024, 10 months ago

Problem/Motivation

Follow-up to #2575615-81: Introduce HtmlEscapedText and remove SafeMarkup::setMultiple() and SafeMarkup::getAll() and remove the static safeStrings list β†’

\Drupal\Core\Render\RendererInterface::renderPlain
implies in its name that it will return plaintext, but returns html (in an object)

Proposed resolution

  • come up with a better name
  • move the code in \Drupal\Core\Render\Renderer::renderPlain to the new method
  • change renderPlain to be a wrapper for the new method and deprecate renderPlain
  • replace usages of renderPlain to the newly named method
  • make a change record

Remaining tasks

<!-- See https://drupal.org/core-mentoring/novice-tasks for tips on identifying novice tasks. Delete or add "Novice" from the Novice? column in the table below as appropriate. Uncomment tasks as the issue advances. Update the Complete? column to indicate when they are done, and maybe reference the comment number where they were done. -->

User interface changes

API changes

Data model changes

πŸ“Œ Task
Status

Closed: duplicate

Version

11.0 πŸ”₯

Component
BaseΒ  β†’

Last updated 1 day ago

Created by

πŸ‡ΊπŸ‡ΈUnited States yesct

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024