Ticket #261 (Fixed)Wed Jan 12 12:17:29 UTC 2011
Forum Recent Posts could list a greater number per page (preferably with user options for specifiying number of items)
| Reported by: | Trevor Johnson (329) | Severity: | Enhancement |
| Part: | Web site: Beast (forum engine; forum pages) | Release: | 3rd public site release |
| Milestone: | 3rd public site release completed | Status | Fixed |
Details by Trevor Johnson (329):
With the forum search being less than ideal (what is the correct syntax for a Boolean search where words are not adjacent?) and the forum possibly getting busier, this suggested enhancement would be a help when searching previous posts.
The current page generated at Recent Posts
seems to list about 26 posts per page. Increasing this to 50, 100 or more would be helpful, if this can be implemented without too much difficulty.
And if users could choose from a selection of numbers of posts to list, that’d be even better (and also please consider a ‘show all’ option, although that may be rather slow).
Changelog:
Modified by Trevor Johnson (329) Sat, January 15 2011 - 00:17:41 GMT
It looks as if line 39 of topics_controller.rb may help. But perhaps this only applies to individual forums, rather than Recent Pages.
<pre>@post_pages, @posts = paginate(:posts, :per_page => 25, :order => ‘posts.created_at’, :include => :user, :conditions => [‘posts.topic_id = ?’, params[:id]])</pre>
Modified by Trevor Johnson (329) Sat, January 15 2011 - 01:09:48 GMT
Of course, I meant line 3 of app/controllers/posts_controller.rb
Modified by Andrew Hodgkinson (6) Wed, March 16 2011 - 19:53:27 GMT
- Milestone changed from Unspecified to Batch Three code release completed
- Release changed from Unspecified to 3rd public site release
Modified by Andrew Hodgkinson (6) Wed, March 16 2011 - 19:54:41 GMT
- Milestone changed from Batch Three code release completed to 3rd public site release completed
Oops!
Modified by Andrew Hodgkinson (6) Fri, August 30 2013 - 06:48:20 GMT
- Status changed from Open to Fixed
“Show all” represents an unbounded load on the database and web server, so I’m not going to expose that, however options for 25, 50 and 100 have proved not too difficult so this has been implemented. See Changeset #442. Thanks for the suggestion.
Modified by Trevor Johnson (329) Fri, August 30 2013 - 08:14:54 GMT
Great stuff! Thanks very much for taking the time to implement it :-)
