wp_list_bookmarks doesn't behave as advertised in the documentation, generates invalid XHTML

[m0n5t3r]

According to the ​codex documentation, when an empty title_li parameter is passed to wp_list_bookmarks, the function should dump the bookmarks as a series of <li> tags not wrapped in <ul></ul>, which is a reasonable behaviour; however, this isn't the case in reality;

the attached patch solves the issue, and also addresses another "feature": when outputting a single category, the <li> tag that surrounds it has the id "linkuncat"; when several link categories are dumped separately, this leads to invalid HTML (several tags with the same id)

[m0n5t3r]

patch to solve #3664

[markjaquith]

Ryan's territory. Agree with 2.1.1 milestone, however, if valid.

[Znuff]

Also confirmed in SVN, looks like valid.

[ryan]

(In [4800]) Don't output ul when title_li is empty. Props m0n5t3r. fixes #3664

[ryan]

Committed for 2.1.1 and 2.2.