I am not sure if this is an issue but I noticed that the path mentioned is incorrect.
Currently it is: admin/config/services/aggregator
But it should be admin/config/services/aggregator/settings
All the path mentioned in the aggregator.api.php point to admin/config/services/aggregator so I am not sure if this is an issue or not.
If it is please let me know and I can create a patch.
Comment | File | Size | Author |
---|---|---|---|
#9 | aggregator_api_doc_cleanup-1791090-9-d7.patch | 4.01 KB | Ivan Zugec |
#7 | aggregator_api_doc_cleanup-1791090-7-d7.patch | 4.01 KB | Ivan Zugec |
#2 | aggregator_api_doc_cleanup-1791090-2.patch | 3.14 KB | Ivan Zugec |
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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: Ivan Zugec commentedFixed patch as requested.
Comment #10
Ivan Zugec CreditAttribution: Ivan Zugec commentedComment #11
jhodgdonLooks ready to commit when the test bot comes back green, thanks!
Comment #12
jhodgdonCommitted to 7.x. Thanks!