For the example list of path options generated by conf_path(), the line breaks are a bit out:

1. $confdir/8080.www.example.com.mysite.test 2. $confdir/www.example.com.mysite.test 3. $confdir/example.com.mysite.test 4. $confdir/com.mysite.test

5. $confdir/8080.www.example.com.mysite 6. $confdir/www.example.com.mysite 7. $confdir/example.com.mysite 8. $confdir/com.mysite

9. $confdir/8080.www.example.com 10. $confdir/www.example.com 11. $confdir/example.com 12. $confdir/com

13. $confdir/default

Instead of:

1. $confdir/8080.www.example.com.mysite.test
2. $confdir/www.example.com.mysite.test
3. $confdir/example.com.mysite.test
4. $confdir/com.mysite.test
5. $confdir/8080.www.example.com.mysite
6. $confdir/www.example.com.mysite
7. $confdir/example.com.mysite
8. $confdir/com.mysite
9. $confdir/8080.www.example.com
10. $confdir/www.example.com
11. $confdir/example.com
12. $confdir/com
13. $confdir/default

CommentFileSizeAuthor
#2 bullet_list-1273032-2.patch1.4 KBmichaellenahan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Title: Bad formatting of conf_path() at http://api.drupal.org/api/drupal/includes--bootstrap.inc/function/conf_path » conf_path() has formatting issue with numbered list
Version: 7.8 » 8.x-dev
Issue tags: +Needs backport to D7

http://api.drupal.org/api/drupal/includes--bootstrap.inc/function/conf_path

Just as a note, this is also an issue in default.settings.php:
#1262064: default.settings.php has formatting issue with numbered list
Changing title so it's more obvious these two issues are the same.

As I suggested there, I think the best fix is to make this a bullet list, which the API module supports and formats correctly, instead of using a numbered list, which isn't supported at this time. Getting changes into the API module is not usually speedy (mostly because no one works on them).

michaellenahan’s picture

Patch attached. I'm still quite new to this - I started on codesprint day at drupalcon London - so feedback appreciated.

michaellenahan’s picture

Status: Active » Needs review
jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Looks great to me! Thanks for making the patch. Should apply to 8.x/7.x.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks.

Automatically closed -- issue fixed for 2 weeks with no activity.