Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
API page: http://api.drupal.org/api/drupal/modules%21system%21system.api.php/funct...
The definition contains a reference to the user/%user_uid_optional path, which is no longer used since D7, unlike previous versions of core.
Comment | File | Size | Author |
---|---|---|---|
#4 | hook_menu_docs_use_drupal8-1512116-4.patch | 725 bytes | mjonesdinero |
#4 | hook_menu_docs_use_drupal7-1512116-4.patch | 707 bytes | mjonesdinero |
Comments
Comment #1
jhodgdonOK, then we'll have to come up with a better example there. Thanks!
Comment #2
fgmTracker may be the simplest current use case for this loader.
Comment #3
jhodgdonSo... What needs to be done on this issue is that in the hook_menu() documentation where it says:
it needs to say tracker/%user_uid_optional instead of user/%user_uid_optional.
This seems like a good Novice project.
Comment #4
mjonesdinero CreditAttribution: mjonesdinero commentedHi attach are patches for d8 and d7, already tested the patch and it change the user to tracker
Comment #6
mjonesdinero CreditAttribution: mjonesdinero commentedI think the patch for drupal 7 here failed because the boot have thought it will be apply in d8
Comment #7
jhodgdon#6 - correct. Please read:
http://drupal.org/node/1319154#multiple-versions
(Quite often the D8 patch will simply apply to D7 as well, by ignoring the core/ directory on the files. If so, I usually commit the same patch to both, so it's often a waste of time to even bother making a d8 and d7 patch.)
Anyway, the patch for D8 looks good and I'll get it committed shortly. Thanks!
Comment #8
jhodgdonCommitted to 8.x and 7.x. Thanks for the patch!