Function drupal_pre_render_html is deprecated and no longer used, so let's remove it.

Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Task: simple code clean-up.
Issue priority Not critical: doesn't inhibits the functionality or main purpose of the project.
Prioritized changes Prioritized: because it removes unused functionality.
CommentFileSizeAuthor
remove_drupal_pre_render_html.patch574 bytesjeroent

Comments

heilop’s picture

heilop commented
Issue tags: +Novice, +LatinAmerica2015
develcuy’s picture

develcuy commented
Issue summary: View changes
develcuy’s picture

develcuy commented
Issue tags: +API clean-up
miteshmap’s picture

miteshmap
Primary language Gujarati
commented

Patch tested, Applied correctly.

develcuy’s picture

develcuy commented
Status: Needs review » Reviewed & tested by the community

Function declared, no calls, no references in API docs.

$ grep -nR drupal_pre_render_html .
./core/includes/theme.inc:1277:function drupal_pre_render_html(array $element) {

  • webchick committed c3425bd on 8.0.x 
    Issue #2421681 by JeroenT, develCuy, developermitesh: Remove...
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

I don't see a change record about this, but I think that is fine. It would be highly unusual for a custom/contrib module to call the arbitrary pre-render callback of some element in core.

Removing deprecated functions is explicitly allowed in beta, so committed and pushed to 8.0.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.