The Render API overview:

https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21...

references drupal_render() exclusively, even though this is deprecated in 8.x and to be removed in 9.0.0:

https://api.drupal.org/api/drupal/core%21includes%21common.inc/function/...

To begin to encourage best practice, in good time for 9.0.0, this API documentation should be changed to reference RendererInterface::render() instead.

CommentFileSizeAuthor
#2 render_api_overview-2848336-2.patch2.58 KBmarkdorison

Comments

jp.stacey created an issue. See original summary.

markdorison’s picture

markdorison
he/him
commented
Status: Needs work » Needs review
StatusFileSize
new render_api_overview-2848336-2.patch2.58 KB
lomo’s picture

lomo commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Needs review » Reviewed & tested by the community

Looks like good and needed changes. This is entirely non-disruptive and I've verified that the patch also applies to the 8.3.x-dev branch, where I think developers could find this documentation fix useful well before 8.4.x has a stable release, so I'm changing the version for this fix to 8.3.x-dev for "backporting".

star-szr’s picture

star-szr
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Closed (duplicate)
Related issues: +#2834889: Replace documentation and string references to drupal_render()

When making changes like this, we should scope it to take care of all such needed changes, rather than just in one area. See the scope guidelines here for more information: https://www.drupal.org/core/scope#incomplete

#2834889: Replace documentation and string references to drupal_render() already exists, so closing this as a duplicate. Thanks for your efforts :)