API page: http://api.drupal.org/api/drupal/sites--default--default.settings.php/7
Describe the problem you have found:
The list of where the settings.php file is searched for appears to have been malformed so it no longer displays correctly.
I know you folks are busy and a little burnt out so I have took the liberty of commenting a more readable list along with additional information that I feel is missing, such as information regarding the sites.php file.
I feel that it would be useful to look at the documentation for this page and update it to make the information more readable.
Before I go I would just like to say I've never came across documentation as extensive as the Drupal docs in my life, so I'm nit picking more than anything, but the main point is you have done a great job so well done.
Thanks,
Matt
Comment | File | Size | Author |
---|---|---|---|
#11 | bullet_list-1262064-11.patch | 1 KB | michaellenahan |
Comments
Comment #1
agentrickardDo you have a patch or new file for review?
Comment #2
Matthew Bonner CreditAttribution: Matthew Bonner commentedNo because this is the documentation on the Drupal website, as previously linked to.
Comment #3
agentrickardAnd that needs to be refactored as a patch.
Can you try that?
Comment #4
TR CreditAttribution: TR commentedHi Matthew,
The documentation you linked to is automatically generated from the Drupal source code. If you look at the source code for default.settings.php you will see the entire text of http://api.drupal.org/api/drupal/sites--default--default.settings.php/7 included in the documentation comments.
The way to get this fixed is to create a patch. See http://drupal.org/patch for more details. Basically, you modify your local copy of default.settings.php and change the text to be how you think it should be. Then create a patch, which shows the differences between your version and the version currently in Drupal core.
If you post the patch here, it can be reviewed for accuracy and wording, and after review it can be added to Drupal. Once that's done, it will show up in the website API documentation.
Additionally, core patches need to be committed to Drupal 8 first, then backported to Drupal 7. So I've moved this to the D8 issue queue.
Comment #5
Matthew Bonner CreditAttribution: Matthew Bonner commentedHi TR
Thanks for that, being new doesn't always help :)
I will do what you said and let you know how I get on.
Thanks,
Matt
Comment #6
agentrickard@matt Sorry I wasn't more clear.
Comment #7
jhodgdonThe problem is that the API module does not support numeric lists at all. So I am not sure how we would fix this so it would display properly on api.drupal.org.
Comment #8
Matthew Bonner CreditAttribution: Matthew Bonner commentedHas this been created as another issue and shall I proceed with a patch or shall I wait until the API module has been updated?
Comment #9
jhodgdonI don't think it's an issue yet in the API module. I also don't think there is another file besides default.settings.php that needs numbered lists, so I don't think it's a very high priority. Maybe the default.settings.php file could be reformatted to use bullet lists (which are supported -- see http://drupal.org/node/1354 section on Lists)?
Comment #10
jhodgdonSomeone just filed this issue -- same problem as here with conf_path():
#1273032: conf_path() has formatting issue with numbered list
Comment #11
michaellenahan CreditAttribution: michaellenahan commentedHere's the patch for the bullet list.
Comment #12
jhodgdonLooks good, thanks! Should apply to 7.x and 8.x.
Comment #13
Dries CreditAttribution: Dries commentedCommitted to 7.x and 8.x. Thanks.