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.

CommentFileSizeAuthor
#9 system_element_info_reference-1671498-9.patch483 bytesg3r4
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 72,847 pass(es). View
#2 system_element_info_reference-1671498-2.patch779 bytesJorrit
PASSED: [[SimpleTest]]: [MySQL] 54,771 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

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
PASSED: [[SimpleTest]]: [MySQL] 54,771 pass(es). View

Here you go.

Cottser’s picture

Cottser’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)

Cottser’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

FileSize
483 bytes
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 72,847 pass(es). View

Ok, I think this is it

g3r4’s picture

Status: Needs work » Needs review
Cottser’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.