time_range_uniqueness 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 868b4a177acdae3b835b05cb3b7d18741298b99592cdfb429dd38196287dc9f4
-  data.tar.gz: 3b2a570fe65b0a08024bd730e1743a15966d0c9bfd2c674d5533e70014a27d69
+  metadata.gz: 1856dd82ab1e6aaddf8cd3c332ecbe84250218b2fae914426eba695d377c8513
+  data.tar.gz: f3d66c621fc3089986f9452bfcc02dedadd47c6b75fc914cc8b127af58dde021
 SHA512:
-  metadata.gz: 8c01bcca608fc1341a154f3f7b40293f483ca27814fdfac96139edf390eac43b3d791c2b1aa8d9004d17597a91db06bfdd6205030c46d114abdddb2fd441d5ac
-  data.tar.gz: 5b31ad1001851747cb211231329bdab526750558482d152eaa4fbe4217ef7afc637287d14a6235f1d5d6b8dbdee85ce2286f6dd1babf1bdaa37cb01c29f5e8e5
+  metadata.gz: 60e9c6b37fa3f0cc8ae51903006a182970900b9e17b37afcd4a7ff57fb84f58d7d84c7657eb661da6055e81ed07328941b2ee885c2c6e74e3fbccb942ae8528e
+  data.tar.gz: b2a21bd9c93329bf320ebe9f3b36f79912ab41adfd9a9bbef057d98f1bedbf700a5e058dfbb9ac08c01b3721def1cd71589af17850e9971524954cecef12ba83

data/CHANGELOG.md

@@ -1,5 +1,16 @@
 ## [Unreleased]
 
+## [1.0.1] - 2026-05-31
+
+- Fix the overlap validation to treat a `NULL` scope value as never-conflicting, matching
+  the exclusion constraint (`NULL = NULL` is never true in PostgreSQL). Previously the
+  validation reported a false overlap for rows the database would accept.
+- Support models with a composite primary key in the overlap validation. Previously the
+  validation raised `ArgumentError` when excluding the current record, so such models
+  could not be validated or created.
+- Raise `ArgumentError` when a custom `:name` exceeds PostgreSQL's 63-character identifier
+  limit, instead of relying on the database to truncate it silently.
+
 ## [1.0.0] - 2026-05-31
 
 - Require Ruby >= 3.2 and support ActiveRecord 7.1 through 8.x (and pg >= 1.5).

data/lib/time_range_uniqueness/migration_additions.rb

@@ -47,7 +47,7 @@ module TimeRangeUniqueness
     # @option options [Array<Symbol>] :scope (Optional) Columns to scope the uniqueness check.
     # @option options [String] :name (Optional) The name of the constraint.
     def add_time_range_uniqueness(table, options = {})
-      raise ArgumentError, 'You must specify the :with option with the time range column name' unless options[:with]
+      validate_options!(options)
 
       time_range_column = options[:with]
       scope_columns = Array(options[:scope])
@@ -61,6 +61,19 @@ module TimeRangeUniqueness
 
     private
 
+    # Validates the options passed to +add_time_range_uniqueness+.
+    #
+    # @param options [Hash] The options for the constraint.
+    # @raise [ArgumentError] If +:with+ is missing or +:name+ exceeds the identifier limit.
+    def validate_options!(options)
+      raise ArgumentError, 'You must specify the :with option with the time range column name' unless options[:with]
+
+      name = options[:name]
+      return unless name && name.to_s.length > MAX_IDENTIFIER_LENGTH
+
+      raise ArgumentError, "The :name option exceeds the #{MAX_IDENTIFIER_LENGTH}-character identifier limit"
+    end
+
     # Applies the changes for the up migration.
     #
     # @param table [Symbol, String] The name of the table.

data/lib/time_range_uniqueness/model_additions.rb

@@ -70,15 +70,16 @@ module TimeRangeUniqueness
       time_range = record.public_send(time_range_column)
       return false if time_range.nil?
 
+      # A NULL scope value can never satisfy the exclusion constraint's `=` comparison,
+      # so such a record can never conflict at the database level. Mirror that here.
+      return false if scope_columns.any? { |col| record.public_send(col).nil? }
+
       column = record.class.connection.quote_column_name(time_range_column)
       bounds = time_range.exclude_end? ? '[)' : '[]'
 
-      scoped_relation(record, scope_columns).where(
-        "#{column} && tstzrange(?, ?, ?)",
-        time_range.begin,
-        time_range.end,
-        bounds
-      ).exists?
+      scoped_relation(record, scope_columns)
+        .where("#{column} && tstzrange(?, ?, ?)", time_range.begin, time_range.end, bounds)
+        .exists?
     end
 
     # Builds the set of other records to check against, optionally scoped by the given columns.
@@ -88,7 +89,10 @@ module TimeRangeUniqueness
     # @return [ActiveRecord::Relation] All other records, scoped by the given columns.
     def self.scoped_relation(record, scope_columns)
       klass = record.class
-      relation = klass.where.not(klass.primary_key => record.id)
+      # Pair each primary key column with its value so this works for both single
+      # and composite primary keys (record.id is an array for composite keys).
+      excluded = Array(klass.primary_key).zip(Array(record.id)).to_h
+      relation = klass.where.not(excluded)
 
       scope_columns.each do |col|
         relation = relation.where(col => record.public_send(col))

data/lib/time_range_uniqueness/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TimeRangeUniqueness
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: time_range_uniqueness
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - j-boers-13