From 18563ff065fbd7f1a41538c3367bb7e8d9670f72 Mon Sep 17 00:00:00 2001 From: Seth Call Date: Tue, 24 Mar 2015 14:59:20 -0500 Subject: [PATCH] * VRFS-2890 fix infinite scroll as well as bug with limit vs per_page style scrolling --- ruby/lib/jam_ruby/models/jam_track.rb | 10 +++++++++- web/app/assets/javascripts/jamtrack.js | 2 +- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/ruby/lib/jam_ruby/models/jam_track.rb b/ruby/lib/jam_ruby/models/jam_track.rb index 49dfcd4b5..1301c03bf 100644 --- a/ruby/lib/jam_ruby/models/jam_track.rb +++ b/ruby/lib/jam_ruby/models/jam_track.rb @@ -69,8 +69,16 @@ module JamRuby page = options[:page].to_i per_page = options[:per_page].to_i + if per_page == 0 + # try and see if limit was specified + limit = options[:limit] + limit ||= 20 + limit = limit.to_i + else + limit = per_page + end + start = (page -1 )* per_page - limit = per_page else limit = options[:limit] limit ||= 20 diff --git a/web/app/assets/javascripts/jamtrack.js b/web/app/assets/javascripts/jamtrack.js index 57371b79e..99bab2960 100644 --- a/web/app/assets/javascripts/jamtrack.js +++ b/web/app/assets/javascripts/jamtrack.js @@ -63,7 +63,7 @@ } function defaultQuery() { - var query = { limit:LIMIT, page:currentPage}; + var query = { per_page:LIMIT, page:currentPage + 1}; if(next) { query.since = next;