active_flag 2.0.2 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 95bb29c60494f354cd2030f43a1661d8bf470761e6c72a55f36fa67bd58a6ca0
-  data.tar.gz: b21537f0466989d7ba6ec4e9dd1508a81a2965dc25dd70fcd7c18569f54b3cce
+  metadata.gz: 92070a4cfdecda2177518adcf91809607614849a64566c05361669b030775c27
+  data.tar.gz: 2f9cb34e47a8cce372521d22a8faca84befd100c2d2cf664e96c44565cd0966f
 SHA512:
-  metadata.gz: 92731cf866d6be1d7165ce0ceea4cfccc60503933cc4d1a1b829e027e391fb778703c3b4c62290bbd49546315010208e896bfdab38b45b62eb2e0638d64bddf0
-  data.tar.gz: 6a44c95b29b5ed5f1c8c214ff6c2facc2c164b6bc48e219c3ca0f68c3e5e5fb9cd971a008e4bcd4b608d0ab4a6a158df07f7706b402a9eefffab0387fca9eb84
+  metadata.gz: 2335c49a3c24629650c5bc197d577b2e3faaff940e0afde74f8c01d1d001329d1f094d0ba7bdb2df65480e10917ff6614e9b8bd2d598af9a664de9d0e6e491f8
+  data.tar.gz: a0f6a1a9ecce9dfb935300762b7baf77d3b37b08a8a3ef8ba74403e1ed3148681f3440269606f9c06d6f6e42e7f70ee14165f208b00fbf66df5017475aaadfc5

data/.github/workflows/ci.yml

@@ -6,12 +6,8 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          - ruby: ruby-head
-            rails_version: '7.2'
-          - ruby: '3.3'
-            rails_version: '7.2'
-          - ruby: '3.2'
-            rails_version: '7.2'
+          - ruby: '3.4'
+            rails_version: '8.0'
           - ruby: '3.1'
             rails_version: '7.2'
           - ruby: '3.1'
@@ -20,7 +16,7 @@ jobs:
     env:
       BUNDLE_GEMFILE: gemfiles/${{ matrix.rails_version }}.gemfile
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
       - uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby }}

data/Rakefile

@@ -7,4 +7,4 @@ Rake::TestTask.new(:test) do |t|
   t.test_files = FileList['test/**/*_test.rb']
 end
 
-task :default => :test
+task default: :test

data/active_flag.gemspec

@@ -1,5 +1,4 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'active_flag/version'
 
@@ -20,11 +19,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'activerecord', '>= 5'
-  spec.add_runtime_dependency 'logger'
-  spec.add_runtime_dependency 'mutex_m'
-  spec.add_runtime_dependency 'bigdecimal'
-  spec.add_development_dependency 'bundler'
   spec.add_development_dependency 'rake'
   spec.add_development_dependency 'minitest'
-  spec.add_development_dependency 'sqlite3', '< 2'
+  spec.add_development_dependency 'sqlite3'
 end

data/gemfiles/6.1.gemfile

@@ -1,5 +1,6 @@
 source 'https://rubygems.org'
 
 gem 'activerecord', '~> 6.1.0'
+gem 'sqlite3', '< 2'
 
 gemspec path: '../'

data/gemfiles/7.2.gemfile

@@ -1,5 +1,6 @@
 source 'https://rubygems.org'
 
-gem 'activerecord', '~> 7.0.0'
+gem 'activerecord', '~> 7.2.0'
+gem 'sqlite3', '< 2'
 
 gemspec path: '../'

data/gemfiles/8.0.gemfile

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+
+gem 'activerecord', '~> 8.0.0'
+
+gemspec path: '../'

data/lib/active_flag/definition.rb

@@ -5,8 +5,15 @@ module ActiveFlag
     def initialize(column, keys, klass)
       @column = column
       @keys = keys.freeze
-      @maps = Hash[keys.map.with_index{ |key, i| [key, 2**i] }].freeze
+      @maps = keys.map.with_index { |key, i| [key, 2**i] }.to_h.freeze
       @klass = klass
+      @scope = klass
+    end
+
+    def with_scope(scope)
+      dup.tap do |d|
+        d.instance_variable_set(:@scope, scope)
+      end
     end
 
     def humans
@@ -25,11 +32,13 @@ module ActiveFlag
     # http://stackoverflow.com/a/12928899/157384
 
     def set_all!(key)
-      @klass.update_all("#{@column} = COALESCE(#{@column}, 0) | #{@maps[key]}")
+      ensure_simple_scope!
+      @scope.update_all("#{@column} = COALESCE(#{@column}, 0) | #{@maps[key]}")
     end
 
     def unset_all!(key)
-      @klass.update_all("#{@column} = COALESCE(#{@column}, 0) & ~#{@maps[key]}")
+      ensure_simple_scope!
+      @scope.update_all("#{@column} = COALESCE(#{@column}, 0) & ~#{@maps[key]}")
     end
 
     def to_i(arg)
@@ -48,6 +57,17 @@ module ActiveFlag
 
   private
 
+    def ensure_simple_scope!
+      return unless @scope.is_a?(ActiveRecord::Relation)
+
+      extras = @scope.values.except(:where)
+      return if extras.values.all? {|value| value.nil? || (value.respond_to?(:empty?) && value.empty?) }
+
+      raise ArgumentError, "set_all!/unset_all! only support simple where scopes. " \
+                           "For complex queries, resolve to IDs first: " \
+                           "#{@klass.name}.where(id: scope.ids).#{@column}.set_all!(key)"
+    end
+
     # Human-friendly print on class level
     def human(key, options={})
       return if key.nil? # otherwise, key.to_s.humanize will return ""

data/lib/active_flag/railtie.rb

@@ -1,7 +1,7 @@
 module ActiveFlag
   if defined? Rails::Railtie
     class Railtie < Rails::Railtie
-      initializer 'active_flag.insert_into_active_record' do |app|
+      initializer 'active_flag.insert_into_active_record' do
         ActiveSupport.on_load :active_record do
           include ActiveFlag
         end

data/lib/active_flag/value.rb

@@ -48,11 +48,16 @@ module ActiveFlag
     end
 
     def method_missing(symbol, *args, &block)
-      if key = symbol.to_s.chomp!('?') and @definition.keys.include?(key.to_sym)
-        set?(key.to_sym)
+      if key = symbol.to_s.chomp('?').to_sym and @definition.keys.include?(key)
+        set?(key)
       else
         super
       end
     end
+
+    def respond_to_missing?(symbol, include_private = false)
+      key = symbol.to_s.chomp('?').to_sym
+      @definition.keys.include?(key) || super
+    end
   end
 end

data/lib/active_flag/version.rb

@@ -1,3 +1,3 @@
 module ActiveFlag
-  VERSION = '2.0.2'
+  VERSION = '2.1.0'
 end

data/lib/active_flag.rb

@@ -33,6 +33,15 @@ module ActiveFlag
         active_flags[column]
       end
 
+      if respond_to?(:generated_relation_methods, true)
+        send(:generated_relation_methods).module_eval do
+          define_method column.to_s.pluralize do
+            klass.active_flags[column].with_scope(self)
+          end
+        end
+      end
+
+
       # Scopes
       define_singleton_method "where_#{column}" do |*args|
         integer, column_name = send("_where_#{column}", *args)

data/plan.md

@@ -0,0 +1,52 @@
+# active_flag v2.1.0 Cleanup Plan
+
+## Files to Modify
+
+### 1. `lib/active_flag/definition.rb`
+
+- `Hash[keys.map.with_index{...}]` → `keys.map.with_index{...}.to_h`
+
+### 2. `lib/active_flag/value.rb`
+
+- Add `respond_to_missing?` to match `method_missing` (so `respond_to?(:english?)` works)
+
+### 3. `lib/active_flag/railtie.rb`
+
+- Remove unused `|app|` block param
+
+### 4. `active_flag.gemspec`
+
+- Remove `# coding: utf-8` (unnecessary since Ruby 2.0)
+- `File.expand_path('../lib', __FILE__)` → `File.expand_path('lib', __dir__)`
+- Remove `logger` from dev deps (stdlib)
+- Remove `bundler` from dev deps (assumed)
+
+### 5. `test/load_fixtures.rb`
+
+- `File.dirname(__FILE__)` → `__dir__`
+
+### 6. `test/active_flag_test.rb`
+
+- Fix `assert @profile.languages, 0` → `assert_equal 0, @profile.languages.raw` (2 occurrences)
+- Add `test_respond_to` for `respond_to_missing?`
+
+### 7. `Rakefile`
+
+- `task :default => :test` → `task default: :test`
+
+### 8. `gemfiles/7.2.gemfile`
+
+- `~> 7.0.0` → `~> 7.2.0` (matches CI matrix)
+
+### 9. `lib/active_flag/version.rb`
+
+- Bump `VERSION = '2.0.3'` → `'2.1.0'`
+
+## Not Changing (would be breaking or adds implicit requirements)
+
+- `options={}` → `**options` in value.rb and definition.rb (breaking for positional hash callers)
+- `and` → `&&` in method_missing (needs rewrite, save for later)
+- Ruby/Rails version requirements (keep >= 5 for AR)
+- `s && @maps[...]` guard (protects against nil/false)
+- `map{...}.compact` → `filter_map` (requires Ruby 2.7+)
+- `bin/setup` noise (optional, leave as-is)

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_flag
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 2.1.0
 platform: ruby
 authors:
 - Kenn Ejima
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-04 00:00:00.000000000 Z
+date: 2026-01-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -24,49 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '5'
 - !ruby/object:Gem::Dependency
-  name: logger
-  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: mutex_m
-  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: bigdecimal
-  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: bundler
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -80,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -94,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: sqlite3
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -107,20 +66,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: sqlite3
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2'
 description: Bit array for ActiveRecord
 email:
 - kenn.ejima@gmail.com
@@ -140,14 +85,17 @@ files:
 - bin/setup
 - gemfiles/6.1.gemfile
 - gemfiles/7.2.gemfile
+- gemfiles/8.0.gemfile
 - lib/active_flag.rb
 - lib/active_flag/definition.rb
 - lib/active_flag/railtie.rb
 - lib/active_flag/value.rb
 - lib/active_flag/version.rb
+- plan.md
 homepage: https://github.com/kenn/active_flag
 licenses: []
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -162,7 +110,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.0.dev
+rubygems_version: 3.5.16
+signing_key:
 specification_version: 4
 summary: Bit array for ActiveRecord
 test_files: []