The docblock of template_preprocess_html() refers to the hook system_elements(), which has been renamed to system_element_info() since Drupal 7. The error exists in both Drupal 7 and 8.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mjonesdinero’s picture

could you elaborate the issue here? on what specific files it is located?so anyone can make a patch on it

Jorrit’s picture

Status: Active » Needs review
FileSize
779 bytes

Here you go.

star-szr’s picture

star-szr’s picture

Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Novice, +Twig, +Documentation

This is still relevant but needs a reroll, just search core for system_elements() and replace with system_element_info().

g3r4’s picture

Assigned: Unassigned » g3r4

Ok I'll work on a re-roll for this one

g3r4’s picture

Assigned: g3r4 » Unassigned
Status: Needs work » Needs review

Well I've searched core for system_elements() and the only place where that exists, is in core/includes/theme.inc, which has already been changed in patch #2, I believe this is ready to be marked as needs review. (marking it now)

star-szr’s picture

Status: Needs review » Needs work

@g3r4 - the patch from #2 no longer applies to 8.x HEAD because it was created ~2 years ago, we need a new patch please :)

g3r4’s picture

Assigned: Unassigned » g3r4

Ok great, will work on that !

Thanks for the heads up @Cottser

g3r4’s picture

Ok, I think this is it

g3r4’s picture

Status: Needs work » Needs review
star-szr’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, thanks!

catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

  • catch committed ba16dfb on 8.x
    Issue #1671498 by g3r4, Jorrit: Fixed Docs of template_preprocess_html...

Status: Fixed » Closed (fixed)

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