X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/ca8e1bf18822e77acda9437c9980d5359894a502..5cbd4038edb32b0304bd766e70fc680ea447b52b:/lib/classic_pagination/pagination_helper.rb diff --git a/lib/classic_pagination/pagination_helper.rb b/lib/classic_pagination/pagination_helper.rb index 069d77566..cb7ef9dfe 100644 --- a/lib/classic_pagination/pagination_helper.rb +++ b/lib/classic_pagination/pagination_helper.rb @@ -27,7 +27,7 @@ module ActionView # (i.e. Older Pages: 1 2 3 4) # :suffix:: suffix for pagination links # (i.e. 1 2 3 4 <- Older Pages) - # :window_size:: the number of pages to show around + # :window_size:: the number of pages to show around # the current page (defaults to 2) # :always_show_anchors:: whether or not the first and last # pages should always be shown @@ -48,18 +48,18 @@ module ActionView # # => 1 2 3 ... 10 # # pagination_links(@person_pages, :always_show_anchors => false) - # # => 1 2 3 + # # => 1 2 3 # # pagination_links(@person_pages, :window_size => 1) # # => 1 2 ... 10 # # pagination_links(@person_pages, :params => { :viewer => "flash" }) - # # => 1 2 3 ... + # # => 1 2 3 ... # # 10 - def pagination_links(paginator, options={}, html_options={}) + def pagination_links(paginator, options = {}, html_options = {}) name = options[:name] || DEFAULT_OPTIONS[:name] params = (options[:params] || DEFAULT_OPTIONS[:params]).clone - + prefix = options[:prefix] || '' suffix = options[:suffix] || '' @@ -71,9 +71,9 @@ module ActionView # Iterate through the pages of a given +paginator+, invoking a # block for each page number that needs to be rendered as a link. - # + # # ==== Options - # :window_size:: the number of pages to show around + # :window_size:: the number of pages to show around # the current page (defaults to +2+) # :always_show_anchors:: whether or not the first and last # pages should always be shown @@ -98,19 +98,19 @@ module ActionView current_page = paginator.current_page window_pages = current_page.window(options[:window_size]).pages return if window_pages.length <= 1 unless link_to_current_page - + first, last = paginator.first, paginator.last - + html = '' html << prefix if prefix - if always_show_anchors and not (wp_first = window_pages[0]).first? + if always_show_anchors && !(wp_first = window_pages[0]).first? html << yield(first.number) html << ' ... ' if wp_first.number - first.number > 1 html << ' ' end - + window_pages.each do |page| if current_page == page && !link_to_current_page html << page.number.to_s @@ -119,8 +119,8 @@ module ActionView end html << ' ' end - - if always_show_anchors and not (wp_last = window_pages[-1]).last? + + if always_show_anchors && !(wp_last = window_pages[-1]).last? html << ' ... ' if last.number - wp_last.number > 1 html << yield(last.number) end @@ -129,7 +129,6 @@ module ActionView html end - end # PaginationHelper end # Helpers end # ActionView