cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use the rails paginator to paginate the trace list, and abstract some

Browse source 
common template code into a partial.
This commit is contained in:
Tom Hughes 2007-08-08 23:58:37 +00:00
parent 96eea9da7f
commit 07ad4a36ea
5 changed files with 26 additions and 53 deletions
Download patch file Download diff file Expand all files Collapse all files

18
app/controllers/trace_controller.rb
View file

@ -9,7 +9,6 @@ class TraceController < ApplicationController
def list (target_user = nil, paging_action = 'list')
@title = 'public GPS traces'
@title += " tagged with #{params[:tag]}" if params[:tag]
@traces_per_page = 20
page_index = params[:page] ? params[:page].to_i - 1 : 0 # nice 1-based page -> 0-based page index
# from display name, pick up user id if one user's traces only
@ -52,18 +51,9 @@ class TraceController < ApplicationController
end
opt[:conditions] = conditions
opt[:per_page] = 20
# count traces using all options except limit
@max_trace = Trace.count(opt)
@max_page = Integer((@max_trace + 1) / @traces_per_page)
# last step before fetch - add paging options
opt[:limit] = @traces_per_page
if page_index > 0
opt[:offset] = @traces_per_page * page_index
end
@traces = Trace.find(:all , opt)
@trace_pages, @traces = paginate(:traces, opt)
# put together SET of tags across traces, for related links
tagset = Hash.new
@ -79,8 +69,8 @@ class TraceController < ApplicationController
# final helper vars for view
@display_name = display_name
@all_tags = tagset.values
@paging_action = paging_action # the action that paging requests should route back to, e.g. 'list' or 'mine'
@page = page_index + 1 # nice 1-based external page numbers
## @paging_action = paging_action # the action that paging requests should route back to, e.g. 'list' or 'mine'
## @page = page_index + 1 # nice 1-based external page numbers
end
def mine