a5ab25d0ba
Replace this common pattern in controllers:
@tags = Tag.search(search_params).paginate(params[:page], :limit => params[:limit], :search_count => params[:search])
with this:
@tags = Tag.paginated_search(params)
`search_count` is used to skip doing a full page count when we're not
doing a search (on the assumption that the number of results will be
high when not constrained by a search). We didn't do this consistently
though. Refactor to do this in every controller.
40 lines
992 B
Ruby
40 lines
992 B
Ruby
class PoolVersionsController < ApplicationController
|
|
respond_to :html, :xml, :json
|
|
before_action :check_availabililty
|
|
around_action :set_timeout
|
|
|
|
def index
|
|
if params[:search] && params[:search][:pool_id].present?
|
|
@pool = Pool.find(params[:search][:pool_id])
|
|
end
|
|
|
|
@pool_versions = PoolArchive.paginated_search(params)
|
|
respond_with(@pool_versions)
|
|
end
|
|
|
|
def diff
|
|
@pool_version = PoolArchive.find(params[:id])
|
|
|
|
if params[:other_id]
|
|
@other_version = PoolArchive.find(params[:other_id])
|
|
else
|
|
@other_version = @pool_version.previous
|
|
end
|
|
end
|
|
|
|
private
|
|
|
|
def set_timeout
|
|
PoolArchive.connection.execute("SET statement_timeout = #{CurrentUser.user.statement_timeout}")
|
|
yield
|
|
ensure
|
|
PoolArchive.connection.execute("SET statement_timeout = 0")
|
|
end
|
|
|
|
def check_availabililty
|
|
if !PoolArchive.enabled?
|
|
raise NotImplementedError.new("Archive service is not configured. Pool versions are not saved.")
|
|
end
|
|
end
|
|
end
|