Parent Topic Limit

[jonkristian]

I have over 50 topics!
Please remove numberposts => 50 on line 397 in bbp-metaboxes.php

[jonkristian]

Removes numberposts from topics dropdown.

[johnjamesjacoby]

(In [3271]) Bump the numberposts limit in bbp_reply_metabox to 250, but allow the args to be filtered so large forums have a way to adjust the query. Fixes #1512.

[MZAWeb]

Any chance we can also add 'order' => 'DESC' to the $args, so as the forum gets bigger, we get the last 250 topics in there instead of the first 250? When editing a reply, I think it's more probable to edit a newer one.

[gautamgupta]

It should actually be something like the hyperlink dialog box imo, which offers a search option.

[MZAWeb]

One easy solution would be checking if the $selected post is not part of the $posts dataset, and add it. 1512.diff has a rough implementation of this.

But I agree that ajaxify this would be cool. I'm wondering how many people actually updates the topic for a reply. Seems a very specific scenario to me.

What if instead of showing the combo we show the title of the parent topic and a link "Change", and when you click that link it makes an ajax call for the combo, with numberposts=-1. So we keep the UI snappy for everyone, but load the whole set for the specific case you need it. I can have a go to implement it, but I'd love to have some feedback before starting coding.

[MZAWeb]

Replying to MZAWeb:

One easy solution would be checking if the $selected post is not part of the $posts dataset, and add it. 1512.diff has a rough implementation of this.

This works better than the wp_list_pluck in the if in my patch

!wp_list_filter( $posts, array( 'ID' => intval( $r['selected'] ) ) )

[MZAWeb]

Related to #1901

[MZAWeb]

This got fixed in #1901