Skip to:
Content

bbPress.org

Opened 14 years ago

Closed 14 years ago

#1365 closed task (blessed) (fixed)

Widgetize forums/topics/replies/latest/popular/etc

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by:
Milestone: 2.0 Priority: high
Severity: normal Version:
Component: Front-end Keywords: close
Cc:

Description

Handlers are needed to wrap around the existing template tags and loop functions to provide widgets for sidebars. This should be fairly straight-forward since the code already exists within the plugin to handle this functionality.

Attachments (1)

widgets-order.diff (9.0 KB) - added by GautamGupta 14 years ago.
Contains topic/reply order, whitespace and local textdomain fixes

Download all attachments as: .zip

Change History (12)

#1 @markmcwilliams
14 years ago

[2653] from Google Code-In

#2 follow-up: @elfin
14 years ago

bbpress\bbp-includes\bbp-widgets.php errors in the bbPress Topics List widget:

Notice: Undefined index: title on line 147

Notice: Undefined index: max_shown on line 148

Notice: Undefined index: show_date on line 149

Notice: Undefined index: pop_check on line 150

Plus the wording:Number of topics back to check reply count to determine popularity. A number less than the maximum number of topics to show disables the check. is complicated beyond belief.

Last edited 14 years ago by elfin (previous) (diff)

#3 @elfin
14 years ago

(not sure if you want to put these into separate tickets)
Reply List lists replies in the wrong order, it's showing the oldest first.

Same for Topics List.

The Forum List is showing days old, might be better to have that off by default

#4 @elfin
14 years ago

The Replies widget is linking to reply/ which doesn't seem to include the first post of the thread - is there any use for this? Would it not be easier to just link to the topic page?

(and once again the replies/ page is messed up in firefox)

Last edited 14 years ago by elfin (previous) (diff)

#5 @johnjamesjacoby
14 years ago

Yep it needs work. Was a first pass and will get iterated on. This ticket is fine. :)

#6 in reply to: ↑ 2 @GautamGupta
14 years ago

Replying to elfin:

bbpress\bbp-includes\bbp-widgets.php errors in the bbPress Topics List widget:

Notice: Undefined index: title on line 147

Notice: Undefined index: max_shown on line 148

Notice: Undefined index: show_date on line 149

Notice: Undefined index: pop_check on line 150

Opened up #1402 with a patch for that.. :)

#7 follow-up: @elfin
14 years ago

Line 190 of bbp-widget.php is

'order'          => 'ASC'

and should be

'order'          => 'DESC'

@GautamGupta
14 years ago

Contains topic/reply order, whitespace and local textdomain fixes

#8 in reply to: ↑ 7 @GautamGupta
14 years ago

  • Component changed from Administration to Front-end
  • Keywords has-ticket tested added; gcin removed

Replying to elfin:

Line 190 of bbp-widget.php is

'order' => 'ASC'

and should be

'order' => 'DESC'

Same thing for topics too. Attached a patch which does thing and a few other things..

#9 @GautamGupta
14 years ago

  • Keywords has-patch added; has-ticket removed

#10 @GautamGupta
14 years ago

  • Keywords close added; has-patch tested removed

[2750] and [2751] should close this..? New tickets for bugs/enhancements.

#11 @GautamGupta
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.