bullet_train-api 1.21.0 → 1.22.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2f289dcbcb8505fd1666319c76ec7877a1d0ad2d06bf2a39f6d3bf467fda70f
-  data.tar.gz: e8b75f5eb1e1f085381b9451a1cd364eac114eb07b0aa14ec73c0d05524abfe7
+  metadata.gz: 3b7a3614e89a9ae0ddffe302d628205c02131faf10a20d98895d58b82001a2d8
+  data.tar.gz: bfc27b83f7888559aad0dd0bf2153bbc402829012e1960d0a5c9d6fcc662900a
 SHA512:
-  metadata.gz: 4d84cffdc55fe290e69ba543049e04d39a1cc5a99ed1909a572e36d7b4836d52d2e892f86640788833c0bf9262abb978f2860e59461542dbf4649b98031f4075
-  data.tar.gz: fa085c07d334c9541772184d7308673990321b7b10e9c47347e528af00c110a7a274ed1cec24d89ef28190a5141f007eaedc180c913cbe639aad88fd3c6e63ad
+  metadata.gz: 1141c3b197a201cbe7fdc5a7a571f5935e4308f23aa46822850a5a93388a7e27cdbec0812753584d5c5188a8e0be96ccb4b9f3f7b5d1eecc784874f3cd6bf779
+  data.tar.gz: 6983da4808837519ac2332fa6ffd2f53755b044b5162de6ee22e664f16f167d2ad4398688a1193e5f0600f8a7c595aec9d193b623a26aae2d53d4481f4f35279

data/app/controllers/concerns/api/controllers/base.rb

@@ -1,6 +1,3 @@
-require "pagy_cursor/pagy/extras/cursor"
-require "pagy_cursor/pagy/extras/uuid_cursor"
-
 module Api::Controllers::Base
   extend ActiveSupport::Concern
 
@@ -18,6 +15,9 @@ module Api::Controllers::Base
     before_action :set_default_response_format
     after_action :set_pagination_headers
 
+    before_action :apply_filters, only: [:index]
+    before_action :apply_pagination, only: [:index]
+
     def modify_url_params(url, new_params)
       uri = URI.parse(url)
       query = Rack::Utils.parse_query(uri.query)
@@ -30,18 +30,24 @@ module Api::Controllers::Base
 
     def set_pagination_headers
       return unless @pagy
-
-      if @pagy.has_more?
-        if (collection = instance_variable_get(collection_variable))
-          next_cursor = collection.last.id
-          link_header = response.headers["Link"]
-          link_value = "<#{modify_url_params(request.url, after: next_cursor)}>; rel=\"next\""
-          response.headers["Link"] = link_header ? "#{link_header}, #{link_value}" : link_value
-          response.headers["Pagination-Next"] = next_cursor
-        end
+      if collection_has_more?
+        link_header = response.headers["Link"]
+        link_value = "<#{modify_url_params(request.url, after: last_id_in_collection)}>; rel=\"next\""
+        response.headers["Link"] = link_header ? "#{link_header}, #{link_value}" : link_value
+        response.headers["Pagination-Next"] = last_id_in_collection
       end
     end
 
+    def collection_has_more?
+      return false unless last_id_in_collection
+      remaining_collection = collection.limit(nil).order(id: :asc).where("id > ?", last_id_in_collection)
+      remaining_collection.any?
+    end
+
+    def last_id_in_collection
+      @last_id_in_collection ||= collection&.last&.id
+    end
+
     rescue_from CanCan::AccessDenied, ActiveRecord::RecordNotFound do |exception|
       render json: {error: "Not found"}, status: :not_found
     end
@@ -49,8 +55,6 @@ module Api::Controllers::Base
     rescue_from NotAuthenticatedError do |exception|
       render json: {error: "Invalid token or no user signed in"}, status: :unauthorized
     end
-
-    before_action :apply_pagination, only: [:index]
   end
 
   def permitted_fields
@@ -97,10 +101,32 @@ module Api::Controllers::Base
     @collection_variable ||= "@#{self.class.name.split("::").last.gsub("Controller", "").underscore}"
   end
 
+  def collection
+    @collection ||= instance_variable_get(collection_variable)
+  end
+
+  def collection=(new_collection)
+    @collection = new_collection
+    instance_variable_set collection_variable, new_collection
+  end
+
+  def apply_filters
+    # An empty method that descendant controllers can override
+    # A possible implementation might look like:
+    #
+    # self.collection = collection.where(status: params[:filter_status]) if params[:filter_status]
+    #
+    # Keep in mind that for adding filters to auto-generated documentation, you
+    # will need to override index.yaml.erb or _paths.yaml.erbs.
+  end
+
   def apply_pagination
-    collection = instance_variable_get collection_variable
-    @pagy, collection = pagy_cursor collection, after: params[:after], order: {id: :asc}
-    instance_variable_set collection_variable, collection
+    pagination_collection = collection.order(id: :asc)
+    if params[:after]
+      pagination_collection = pagination_collection.where("id > ?", params[:after])
+    end
+    @pagy, pagination_collection = pagy(pagination_collection)
+    self.collection = pagination_collection
   end
 
   def set_default_response_format

data/lib/bullet_train/api/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module Api
-    VERSION = "1.21.0"
+    VERSION = "1.22.0"
   end
 end

data/lib/bullet_train/api.rb

@@ -7,7 +7,6 @@ require "bullet_train/platform/connection_workflow"
 
 # require "wine_bouncer"
 require "pagy"
-require "pagy_cursor"
 require "doorkeeper"
 require "scaffolding"
 require "scaffolding/block_manipulator"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-api
 version: !ruby/object:Gem::Version
-  version: 1.21.0
+  version: 1.22.0
 platform: ruby
 authors:
 - Andrew Culver
@@ -93,20 +93,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '8'
-- !ruby/object:Gem::Dependency
-  name: pagy_cursor
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: doorkeeper
   requirement: !ruby/object:Gem::Requirement