active_median 0.3.3 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8fac5b8a4669bedd7b9a7bd104c375df5acf6c9e60bc2c91ec97efb2ebea6969
-  data.tar.gz: afc2609d7983f06b975244e57ab4f6890916fdc3943870974a56d50cca19a832
+  metadata.gz: d5ba23ce76f2234aaba1595b26ce41faeba63c03807b8b101d7e0152d48df133
+  data.tar.gz: 15fb7d99f6991034b9f93bdfa6c7aeb681518cf45ab618bcd11e471ff163eb48
 SHA512:
-  metadata.gz: f3cff2041e5d602bd84f99f99cd75efbe6fba5c78faf4efbd6b3886edb91bf1db7deeca609ded57a859f10e670298f04a225f3f1b69ea8b4df9aecda68661b5e
-  data.tar.gz: d494fddff7e01063c99d549d4669f6efff207fef25f71761329d3c16166bc6a268f02020e12adde251be855ed0c31998f47fb73890689565bb67a0c320875b06
+  metadata.gz: c4078461f5f215b8d5632f2c75caf7f57289af0b979e51fa2cf6f3c8a0dc83c7661bd6677efd5800e81337123681c111a8f908807d66203f5e0d713db9c5be3e
+  data.tar.gz: b96f21e8135e91a4d1e4ec78e128852b364319d7c56af0a15c77d810cf1276ec421ff37b91d1a2d5f66bd7683617d45e5483740f4f6f9c948354d9280d70ff62

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.4.0 (2023-05-25)
+
+- Fixed error with Active Record 7.0.5
+- Raise `ArgumentError` for invalid percentiles
+- Dropped support for Ruby < 3 and Active Record < 6.1
+
 ## 0.3.3 (2021-08-17)
 
 - Fixed null values for SQLite without an extension and MongoDB

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2013-2021 Andrew Kane
+Copyright (c) 2013-2023 Andrew Kane
 
 MIT License
 

data/README.md

@@ -20,7 +20,7 @@ Supports:
 Add this line to your application’s Gemfile:
 
 ```ruby
-gem 'active_median'
+gem "active_median"
 ```
 
 For MySQL, also follow [these instructions](#additional-instructions).
@@ -105,18 +105,6 @@ db.load_extension("percentile.so") # or percentile.dylib
 db.enable_load_extension(0)
 ```
 
-## Upgrading
-
-### 0.3.0
-
-ActiveMedian 0.3.0 protects against unsafe input by default. For non-attribute arguments, use:
-
-```ruby
-Item.median(Arel.sql(known_safe_value))
-```
-
-Also, percentiles are now supported with SQLite. Use the [percentile extension](#sqlite) instead of `extension-functions`.
-
 ## Contributing
 
 Everyone is encouraged to help improve this project. Here are a few ways you can help:
@@ -126,12 +114,29 @@ Everyone is encouraged to help improve this project. Here are a few ways you can
 - Write, clarify, or fix documentation
 - Suggest or add new features
 
-To get started with development and testing:
+To get started with development:
 
 ```sh
 git clone https://github.com/ankane/active_median.git
 cd active_median
-createdb active_median_test
 bundle install
+
+# Postgres
+createdb active_median_test
 bundle exec rake test
+
+# SQLite
+ADAPTER=sqlite3 BUNDLE_GEMFILE=gemfiles/sqlite3.gemfile bundle exec rake test
+
+# MariaDB and MySQL (for MySQL, install the extension first)
+mysqladmin create active_median_test
+ADAPTER=mysql2 BUNDLE_GEMFILE=gemfiles/mysql2.gemfile bundle exec rake test
+
+# SQL Server
+docker run -e 'ACCEPT_EULA=Y' -e 'SA_PASSWORD=YourStrong!Passw0rd' -p 1433:1433 -d mcr.microsoft.com/mssql/server:2022-latest
+docker exec -it <container-id> /opt/mssql-tools/bin/sqlcmd -S localhost -U SA -P YourStrong\!Passw0rd -Q "CREATE DATABASE active_median_test"
+ADAPTER=sqlserver BUNDLE_GEMFILE=gemfiles/sqlserver.gemfile bundle exec rake test
+
+# MongoDB
+BUNDLE_GEMFILE=gemfiles/mongoid7.gemfile bundle exec rake test
 ```

data/lib/active_median/model.rb

@@ -11,7 +11,8 @@ module ActiveMedian
     private
 
     def calculate_percentile(column, percentile, operation)
-      percentile = percentile.to_f
+      percentile = Float(percentile, exception: false)
+      raise ArgumentError, "invalid percentile" if percentile.nil?
       raise ArgumentError, "percentile is not between 0 and 1" if percentile < 0 || percentile > 1
 
       # basic version of Active Record disallow_raw_sql!
@@ -24,7 +25,13 @@ module ActiveMedian
         end
       end
 
-      column_alias = relation.send(:column_alias_for, "#{operation} #{column.to_s.downcase}")
+      column_alias =
+        if relation.respond_to?(:column_alias_for, true)
+          relation.send(:column_alias_for, "#{operation} #{column.to_s.downcase}")
+        else
+          # Active Record 7.0.5+
+          ActiveRecord::Calculations::ColumnAliasTracker.new(connection).alias_for("#{operation} #{column.to_s.downcase}")
+        end
       # safety check
       # could quote, but want to keep consistent with Active Record
       raise "Bad column alias: #{column_alias}. Please report a bug." unless column_alias =~ /\A[a-z0-9_]+\z/

data/lib/active_median/mongoid.rb

@@ -6,7 +6,8 @@ module ActiveMedian
 
     # https://www.compose.com/articles/mongo-metrics-finding-a-happy-median/
     def percentile(column, percentile)
-      percentile = percentile.to_f
+      percentile = Float(percentile, exception: false)
+      raise ArgumentError, "invalid percentile" if percentile.nil?
       raise ArgumentError, "percentile is not between 0 and 1" if percentile < 0 || percentile > 1
 
       relation =

data/lib/active_median/version.rb

@@ -1,3 +1,3 @@
 module ActiveMedian
-  VERSION = "0.3.3"
+  VERSION = "0.4.0"
 end

data/lib/active_median.rb

@@ -1,25 +1,16 @@
+# dependencies
 require "active_support"
 
-require "active_median/enumerable"
-require "active_median/version"
-
-module ActiveMedian
-  # TODO remove in 0.4.0
-  def self.drop_function
-    ActiveRecord::Base.connection.execute <<-SQL
-      DROP AGGREGATE IF EXISTS median(anyelement);
-      DROP FUNCTION IF EXISTS median(anyarray);
-    SQL
-    true
-  end
-end
+# modules
+require_relative "active_median/enumerable"
+require_relative "active_median/version"
 
 ActiveSupport.on_load(:active_record) do
-  require "active_median/model"
+  require_relative "active_median/model"
   extend(ActiveMedian::Model)
 end
 
 ActiveSupport.on_load(:mongoid) do
-  require "active_median/mongoid"
+  require_relative "active_median/mongoid"
   Mongoid::Document::ClassMethods.include(ActiveMedian::Mongoid)
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_median
 version: !ruby/object:Gem::Version
-  version: 0.3.3
+  version: 0.4.0
 platform: ruby
 authors:
 - Andrew Kane
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-17 00:00:00.000000000 Z
+date: 2023-05-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '6.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '6.1'
 description:
 email: andrew@ankane.org
 executables: []
@@ -51,14 +51,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.6'
+      version: '3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Median and percentile for Active Record, Mongoid, arrays, and hashes