Make WordPress Core

Opened 20 years ago

Closed 20 years ago

#2587 closed defect (bug) (fixed)

New template function: wp_list_bookmarks()

Reported by: ryan Owned by: westi
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: bookmarks lists has-patch
Cc: Focuses:

Description

wp_list_bookmarks() will replace get_links_list() and get_links(). By default, it should create a list of all bookmarks organized by category. We want to avoid the mistake of hard coding h2s and other formatting, so there should be options for specifying any formatting that should come before and after each category heading. wp_list_bookmarks() should also allow creating a big list of uncategorized bookmarks as well as listing bookmarks in a specific category or set of categories. A stub for wp_list_bookmarks() exists in template-functions-bookmarks.php along with some implementation notes. wp_list_bookmarks() should use get_bookmarks() for querying the DB.

Attachments (1)

2587.wp_list_bookmarks.diff (2.7 KB ) - added by westi 20 years ago.
Implement the rest of wp_list_bookmarks

Download all attachments as: .zip

Change History (12)

#1 @MichaelH
20 years ago

Question on this requirement--"as well as listing bookmarks in a specific category or set of categories".

If it's a 'set of categories' does that imply allowing something like 'category=1,4,7,9'? If yes, should get_bookmarks() be changed to handle that?

#2 @ryan
20 years ago

Yes and yes.

#4 @ryan
20 years ago

[3880] first pass

#5 @westi
20 years ago

  • Owner changed from anonymous to westi
  • Status newassigned

Quite a bit of the support code for this seems in good shape.

I'm working on finishing off coding wp_list_bookmarks against the requirements spec.
I'll upload a patch when I've got a bit more finished.

@westi
20 years ago

Implement the rest of wp_list_bookmarks

#6 @westi
20 years ago

  • Keywords has-patch added

Attached patch implements the rest of wp_list_bookmarks I believe

#8 follow-up: @foolswisdom
20 years ago

westi, ryan, is there still more to do for this bug? Or should it be closed?

#9 in reply to: ↑ 8 @westi
20 years ago

Replying to foolswisdom:

westi, ryan, is there still more to do for this bug? Or should it be closed?

I don't think this is complete yet.

I think we now need to move the deprecated functions to deprecated.php and implement them as calls to the new function.

I guess that is why ryan left the ticket open?

#10 @matt
20 years ago

  • Milestone 2.12.2

#11 @ryan
20 years ago

  • Milestone 2.22.1
  • Resolutionfixed
  • Status assignedclosed

Actually, this is done.

Note: See TracTickets for help on using tickets.