Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
21 Sep 2012 at 07:38 UTC
Updated:
4 Jan 2014 at 02:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jhodgdonThanks! The page in question is:
http://api.drupal.org/api/drupal/modules!aggregator!aggregator.api.php/f...
I think we should probably remove the admin path from the documentation entirely -- our standard in general is not to include specific paths in documentation (for instance, in page callbacks from hook_menu()) because they change too much and the docs don't get updated (as in this case).
Comment #2
ivan zugec commentedHere's a first pass at cleaning up the API documentation.
Comment #3
jhodgdonTHANK YOU, this is perfect. I'll get it committed shortly.
Comment #4
jhodgdonassigning to self for commit so webchick doesn't get overwhelmed by the RTBC queue. :)
Comment #5
jhodgdonCommitted to 8.x. Needs to be ported to 7.x now. Thanks!
Comment #6
jhodgdonComment #7
ivan zugec commentedAwesome. Here's a patch for Drupal 7.
Comment #8
jhodgdonThere are some spacing and wrapping problems in the last block of this patch:
(two spaces before "within", and "configuration" will fit on the previous line. Other than that, looks good, thanks!
Comment #9
ivan zugec commentedFixed patch as requested.
Comment #10
ivan zugec commentedComment #11
jhodgdonLooks ready to commit when the test bot comes back green, thanks!
Comment #12
jhodgdonCommitted to 7.x. Thanks!