Opening this to backport #1500160: Document the 'pages' key in hook_block_info()
API page: http://api.drupal.org/api/drupal/modules%21block%21block.api.php/functio...
Even though pages is documented as "a list of pages", it should not be an array, but a single string with the paths separated by "\n". This should be documented.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | drupal-7.x-api-doc-1503614-5.patch | 3.92 KB | jmarkel |
| #4 | drupal-7.x-api-doc-1503614-4.patch | 3.9 KB | jmarkel |
| #2 | drupal-7.x-api-doc-1503614-2.patch | 3.9 KB | jmarkel |
| #1 | drupal-7.x-api-doc-1500160-1.patch | 3.72 KB | jmarkel |
Comments
Comment #1
jmarkel commentedHere's the patch backported from #1500160: Document the 'pages' key in hook_block_info()
Ach - should have renamed the patch for this issue. Ah well :-(
Comment #2
jmarkel commentedRe-rolled the patch with additional info for the 'pages' key - for BLOCK_VISIBILITY_PHP
Comment #3
jmarkel commentedComment #4
jmarkel commentedDang - made a typo. Here it is, fixed.
Comment #5
jmarkel commentedUpdated to note that there are three rather than one path separators allowed - '\n', '\r', and '\r\n'.
Comment #6
jhodgdonThis issue should just be closed. What we normally do is wait until the 8.x issue has been resolved, then change *that* issue to 7.x, so that the entire issue history is in one place. So I'm closing this as a duplicate of
#1500160: Document the 'pages' key in hook_block_info()
Also, it saves a lot of time and effort to just work on one version at a time, rather than trying to maintain the 8.x and 7.x patches through all the reviews. Thanks!
Comment #6.0
jhodgdonUpdated issue summary.