Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Issue tags:
Reporter:
Created:
21 Feb 2015 at 14:13 UTC
Updated:
27 Apr 2015 at 11:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
berdirNote: We noticed this because of http://drupal.stackexchange.com/questions/141373/drupal-8-custom-page-te...
Comment #2
jrhodes07 commentedHere's a patch that updates the documentation:
Comment #3
star-szrThanks @ClientGuy :)
The hook is invoked for all modules, period. So this whole paragraph should be updated, something like this might do it:
"This hook allows modules to provide alternative theme function or template name suggestions."
Comment #4
anwar_maxComment #5
anwar_maxAs per #3 changes have been made to the patch.
Comment #6
anwar_maxComment #7
anwar_maxUploaded empty patch. Re-uploading the patch.
Comment #8
berdirThat still doesn't make sense, the part hook_theme() needs to go. As @Cottser said, it is called for all modules. full stop.
Comment #9
joshi.rohit100updated patch.
Comment #10
star-szrMuch closer, thanks!
Arguably "This hook is invoked for all modules" is the default and therefore redundant, but I'm fine either way on that. Might be worth looking around at other *.api.php's as an example.
This part actually should be updated too I think, because it seems to reflect the previous paragraph where only the module implementing hook_theme() would get the hook invocation.
Here's the full couple sentences:
Comment #11
harshil.maradiya commentedUpdated
Comment #12
harshil.maradiya commentedComment #16
star-szr@harshil.maradiya - Patch is corrupt according to git, missing newline at EOF I think.
All my comments from #10 still apply. I don't think we should change the docs for hook_theme_suggestions_HOOK_alter() in this issue. Can we go back to the patch from #9 please and provide interdiffs when making changes to patches? Thanks all! :)
Comment #17
wadmiraal commentedI'll get it.
Comment #18
wadmiraal commentedUpdated documentation. I checked other *.api.php files: they don't mention "this is invoked for all modules" (it is kind of implicit).
Comment #19
wadmiraal commentedComment #20
jhodgdonLooks great to me, thanks!
Comment #21
alexpottCommitted c9a38f1 and pushed to 8.0.x. Thanks!