rotulus 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6bc659ce794517af485f9c65dc18467b4278c21e8813e612a0b4dcdabde0b10f
-  data.tar.gz: 2dc66e54844499214cc07ae49d8ca0ad33586ba28a11d4e2a3a4366d4633525a
+  metadata.gz: 2c74625ece0961a25549797386fa66abb146564211b4dc1640abd92ac4c0e2a0
+  data.tar.gz: 567562c553bbe914cdf5740e4262bd76b59ef04493658a68c69bd420e3109283
 SHA512:
-  metadata.gz: f60f971213d39e4baa5d6fd53227971df6e09fb87f1824487e17a2972e80803db29e2b5894cc7b0be659d73ad3bf959a5944c5ed0235b754f19a80afe0cd69e1
-  data.tar.gz: f0b072563fe10c626772f2879d9533263b393452022a5e4547ea6e5413a5bfab92ef629dd3aa787328b3bda14ae89ae04e1d5c345073fe3ae2e2afc2092dae86
+  metadata.gz: 9e1380f553f3a0756b36b21f2a6193ee7031fe29bc8b3478ec349f6c35d2b9292ed5afd295b6b5250b2dc4498e1d86b66d13d639ea5e3d36db78fc8868fb4e9f
+  data.tar.gz: 625359b428c792c30a2a2ee2b6b702a2c7270e495111c284c82a1d1433eddf5d0ac485d6d5ae961128006f161c4cb01b8fa76f7ab3bd14d63cb761306a9392f6

data/CHANGELOG.md

@@ -1,7 +1,8 @@
 ## 0.2.0
-
-Initial release.
+- Initial release.
 
 ## 0.2.1
+- Drop unnecessary ORDER BY columns following a non-nullable and distinct column.
 
-Drop unnecessary ORDER BY columns following a non-nullable and distinct column.
+## 0.2.2
+- Raise error when there is no non-nullable and distinct column in the configured order definition.

data/README.md

@@ -295,18 +295,19 @@ page = Rotulus::Page.new(items, order: order_by, limit: 2)
 
 | Class | Description |
 | ----------- | ----------- |
-| `Rotulus::InvalidCursor` | Cursor token received is invalid e.g., unrecognized token, token data has been tampered/updated, base ActiveRecord relation filter/sorting/limit is no longer consistent to the token/ |
+| `Rotulus::InvalidCursor` | Cursor token received is invalid e.g., unrecognized token, token data has been tampered/updated, base ActiveRecord relation filter/sorting/limit is no longer consistent to the token. |
 | `Rotulus::Expired` | Cursor token received has expired based on the configured `token_expires_in` |
 | `Rotulus::InvalidLimit` | Limit set to Rotulus::Page is not valid. e.g., exceeds the configured limit. see `config.page_max_limit` |
 | `Rotulus::CursorError` | Generic error for cursor related validations |
-| `Rotulus:: InvalidColumnError` | Column provided in the :order param is can't be recognized. |
+| `Rotulus::InvalidColumnError` | Column provided in the :order param can't be found. |
+| `Rotulus::MissingTiebreakerError` | There is no non-nullable and distinct column in the configured order definition. |
 | `Rotulus::ConfigurationError` | Generic error for missing/invalid configurations. |
 
 ## How it works
 Cursor-based pagination uses a reference point/record to fetch the previous or next set of records. This gem takes care of the SQL query and cursor generation needed for the pagination. To ensure that the pagination results are stable, it requires that:
 
 * Records are sorted (`ORDER BY`).
-* In case multiple records with the same column value(s) exists in the result, a unique column is needed as tie-breaker. Usually, the table PK suffices for this but for complex queries(e.g. with table joins and with nullable columns, etc.), combining and using multiple columns that would uniquely identify the row in the result is needed.
+* In case multiple records with the same column value(s) exists in the result, a unique non-nullable column is needed as tie-breaker. Usually, the table PK suffices for this but for complex queries(e.g. with table joins and with nullable columns, etc.), combining and using multiple columns that would uniquely identify the row in the result is needed.
 * Columns used in `ORDER BY` would need to be indexed as they will be used in filtering.
 
 

data/lib/rotulus/order.rb

@@ -11,6 +11,9 @@ module Rotulus
       @definition = {}
 
       build_column_definitions
+
+      return if has_tiebreaker?
+      raise Rotulus::MissingTiebreakerError.new('A non-nullable and distinct column is required.')
     end
 
     # Returns an array of the ordered columns
@@ -137,6 +140,12 @@ module Rotulus
       )
     end
 
+    def has_tiebreaker?
+      last_column = columns_for_order.last
+
+      last_column.distinct? && !last_column.nullable?
+    end
+
     def primary_key_ordered?
       !definition["#{ar_table}.#{ar_model_primary_key}"].nil?
     end

data/lib/rotulus/version.rb

@@ -1,3 +1,3 @@
 module Rotulus
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 end

data/lib/rotulus.rb

@@ -21,7 +21,8 @@ module Rotulus
   class InvalidCursorDirection < CursorError; end
   class InvalidLimit < BaseError; end
   class ConfigurationError < BaseError; end
-  class InvalidColumnError < BaseError; end
+  class MissingTiebreakerError < ConfigurationError; end
+  class InvalidColumnError < ConfigurationError; end
 
   def self.db
     @db ||= case ActiveRecord::Base.connection.adapter_name.downcase

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rotulus
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Uy Jayson B