get_links_list() in wp-includes/links.php ignores $text_after_link settings

[Froosh]

Out of the box, index.php (which calls get_links_list()) does not use any $text_after_link (text between link and description) settings that you specify in link categories (Not even the default <br />).

I have attached a diff to links.php that resolves this minor problem.

[Froosh]

A quick browse of the rest of the code in links.php reveals that get_links_list() is the only function that ignores text_before_link, text_after_link and text_after_all properties.

[dennis]

I'm bumping this because it was addressed in links.php CVS 1.12.4.1 but removed in 1.12.4.2

[ryan]

get_links_list() is purposely hard-coded to use list items. That's why it is get_links_*list*. You can use wp_get_links() if you want to use the DB defaults. That method can be modified to allow listing all link categories rather than just one. Or, we can deprecate all of the link settings junk and clear that cruft out of the links UI.

[Froosh]

Ok, that makes sense.

Should this instead be a request that wp-sidebar use wp_get_links() rather than get_links_list()?

From a user-experience point of view it's very frustrating when changing the settings in Link Categories and seeing zero change in the site.

I would prefer see the link separators stay and be implemented rather than be removed altogether. Not that I'm even using them at the moment... :)