Remove drupal_pre_render_html [#2421681]

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.

Comment	File	Size	Author
	remove_drupal_pre_render_html.patch	574 bytes	JeroenT

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

heilop CreditAttribution: heilop commented 12 February 2015 at 19:56

Issue tags:

+Novice, +LatinAmerica2015

Comment #2

DevElCuy CreditAttribution: DevElCuy commented 13 February 2015 at 17:23

Issue summary:

View changes

Comment #3

DevElCuy CreditAttribution: DevElCuy commented 13 February 2015 at 17:27

Issue tags:

+API clean-up

Comment #4

miteshmap

Gujarati

CreditAttribution: miteshmap commented 13 February 2015 at 17:30

Patch tested, Applied correctly.

Comment #5

DevElCuy CreditAttribution: DevElCuy commented 13 February 2015 at 17:46

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) {

Comment #6

13 February 2015 at 19:46

webchick committed c3425bd on 8.0.x

Issue #2421681 by JeroenT, develCuy, developermitesh: Remove...

Comment #7

webchick

she/they

English

Vancouver 🇨🇦

CreditAttribution: webchick commented 13 February 2015 at 19:46

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!