Doc block wrongly points that this hook is hook_pre_render_HOOK() but this is preprocess

CommentFileSizeAuthor
#2 2900247-2.patch526 bytesandypost

Comments

andypost created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs review
StatusFileSize
new 2900247-2.patch526 bytes

Simple fix, this was copy/pasted to https://www.drupal.org/project/zen

thejacer87’s picture

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

rtbc +1. good catch

  • Cottser committed bff6783 on 8.5.x 
    Issue #2900247 by andypost, thejacer87: Hook...

  • Cottser committed a33651c on 8.4.x 
    Issue #2900247 by andypost, thejacer87: Hook...
star-szr’s picture

star-szr
he/him
Primary language English
commented
Title: Hook seven_preprocess_menu_local_tasks is preprocess » Hook seven_preprocess_menu_local_tasks is preprocess, not pre_render
Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed

Oops, dates back all the way to Twig conversion, nice catch indeed.

https://www.drupal.org/node/1898478#comment-9126927

Committed and pushed bff6783e98 to 8.5.x and a33651cbe1 to 8.4.x. Thanks!

Status: Fixed » Closed (fixed)

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