Posted by fgm on April 2, 2012 at 10:36am
4 followers
| Project: | Drupal core |
| Version: | 8.x-dev |
| Component: | documentation |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
| Issue tags: | needs backport to D7, Novice |
Issue Summary
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.
Comments
#1
OK, then we'll have to come up with a better example there. Thanks!
#2
Tracker may be the simplest current use case for this loader.
#3
So... 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.
#4
Hi attach are patches for d8 and d7, already tested the patch and it change the user to tracker
#5
The last submitted patch, hook_menu_docs_use_drupal7-1512116-4.patch, failed testing.
#6
I think the patch for drupal 7 here failed because the boot have thought it will be apply in d8
#7
#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!
#8
Committed to 8.x and 7.x. Thanks for the patch!
#9
Automatically closed -- issue fixed for 2 weeks with no activity.