Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
26 Sep 2012 at 14:09 UTC
Updated:
1 Jul 2015 at 15:23 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jamix commentedAttaching the patch.
Comment #2
jhodgdonThanks! We need to change the documentation of the whole function though, not just the return value. For instance, the first line says:
Returns the ancestors (and relevant placeholders) for any given path.
and that is no longer correct. The function documentation in general has some grammar issues and doesn't make a lot of sense to me... maybe it could be rewritten better?
Also, we have to patch Drupal 8.x first, then backport the patch to 7.x.
Comment #3
malcomio commentedmenu_get_ancestors doesn't exist in D8 any more
Comment #4
sivaji_ganesh_jojodae commentedFixed return value and example. Propose as how the whole documentation could be changed,
Comment #5
jhodgdonThanks! This does not address comment #2 fully though.
Comment #6
jhodgdonAlso it is not necessary to paste the final result into a comment. Just upload the patch file next time. Thanks!