enum_ish 1.5.1 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cb60bdd7ea33e3c85bee0ed814e651bd00736970de81cc3c6e0b47b384f42017
-  data.tar.gz: 1a206653955294e0bd6784f7e34098df04ff588fc7c16f0417d3da9c3425ee32
+  metadata.gz: 9578ab7ba6e720321c8706fabc463af41bcd7244e24d65ff736943a197abdc9b
+  data.tar.gz: 69b73002c3b790e6a737db306fadfb411e418c1ed5e25faf53a96c57f9f44582
 SHA512:
-  metadata.gz: 825669f678a2d0eb6dd23a656bf00d929920f41bd6b44daeda81a1a257c544219b0e57c0c0e9ef392f53c7b79ec4980a6e976b40f533d71d34496e7546302cef
-  data.tar.gz: 737648557aa906080bf70f1f3f2eb9cbf007490853a1c0f61a604d15b8201178083eeb9c503aec297d347f87555131f8765e7b0b9b9bd262dad6f9db3942ca27
+  metadata.gz: 955eb532c9bc238d8721589240bf600412e679f97b6d0c782f2b34916db04dbd757b389f551fa32605516216a5031730ac37c9778df9ff947cd81825c74e7cf7
+  data.tar.gz: ba5eabfa271493a71ea0d08950b674662535df79a37386b4b59b46000e365ed95f8175946edbd2236310de21b16fb6de3af827766b61db35f3d16820e56cdaf9

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 1.6.0
+
+* Add `EnumIsh.config.defaults` which allows to set default options for `enum_ish`.
+* Don't convert dictionary value to string.
+
 ## 1.5.1
 
 * Fix wrong cache lookup.

data/README.md

@@ -206,7 +206,7 @@ Rails.application.config.middleware.use EnumIsh::DictionaryCache
 ```
 
 This middleware enables dictionary cache for each request.
-Prformance of dictionary lookup will be improved in case one dictionary is used repeatedly in the same request.
+Performance of dictionary lookup will be improved in case one dictionary is used repeatedly in the same request.
 
 ## Contributing
 

data/lib/enum_ish/active_record_definer.rb

@@ -11,7 +11,7 @@ module EnumIsh
         after_initialize method
         define_method method do
           if respond_to?(enum.name) && public_send(enum.name).nil?
-            default = enum.setting[:default]
+            default = enum.default
             default = instance_exec(&default) if default.kind_of?(Proc)
             public_send("#{enum.name}=", default)
           end
@@ -21,7 +21,7 @@ module EnumIsh
 
     def define_accessor(enum)
       @klass.class_eval do
-        define_method "#{Config.raw_prefix}#{enum.name}#{Config.raw_suffix}" do
+        define_method enum.raw_method do
           value = read_attribute(enum.name)
           enum.mapping.fetch(value, value)
         end
@@ -40,10 +40,10 @@ module EnumIsh
 
     def define_scope(enum)
       @klass.class_eval do
-        scope "#{Config.scope_prefix}#{enum.name}#{Config.scope_suffix}", ->(*value) {
+        scope enum.scope_method, ->(*value) {
           where(enum.name => enum.mapping.fetch(value, value))
         }
-        scope "#{Config.scope_prefix}#{enum.name}_not#{Config.scope_suffix}", ->(*value) {
+        scope enum.scope_method(:negative), ->(*value) {
           where.not(enum.name => enum.mapping.fetch(value, value))
         }
       end

data/lib/enum_ish/base.rb

@@ -14,11 +14,11 @@ module EnumIsh
     end
 
     class_methods do
-      def enum_ish(name, map, config = {})
-        enum = Enum.new(name, map, config)
+      def enum_ish(name, map, setting = {})
+        enum = Enum.new(name, map, Config.defaults.merge(setting))
         self._enum_ish_enums = _enum_ish_enums.merge(name.to_sym => enum)
 
-        if defined?(ActiveRecord::Base) && self.ancestors.include?(ActiveRecord::Base)
+        if defined?(ActiveRecord::Base) && ancestors.include?(ActiveRecord::Base)
           ActiveRecordDefiner.new(self).define(enum)
         else
           Definer.new(self).define(enum)

data/lib/enum_ish/config.rb

@@ -5,6 +5,7 @@ module EnumIsh
     class_attribute :data
 
     self.data = {
+      defaults: { text: true, options: true },
       text_prefix: '',
       text_suffix: '_text',
       options_prefix: '',

data/lib/enum_ish/definer.rb

@@ -7,8 +7,8 @@ module EnumIsh
     end
 
     def define(enum)
-      [:text, :options, :predicate, :default, :accessor, :validate, :scope].each do |type|
-        send("define_#{type}", enum) if enum.setting[type]
+      enum.features.each do |feature|
+        send("define_#{feature}", enum)
       end
     end
 
@@ -16,7 +16,7 @@ module EnumIsh
 
     def define_text(enum)
       @klass.class_eval do
-        define_method "#{Config.text_prefix}#{enum.name}#{Config.text_suffix}" do |options = {}|
+        define_method enum.text_method do |options = {}|
           dict = Dictionary.new(self.class, enum, options)
           dict.translate_value(public_send(enum.name))
         end
@@ -25,7 +25,7 @@ module EnumIsh
 
     def define_options(enum)
       @klass.class_eval do
-        define_singleton_method "#{Config.options_prefix}#{enum.name}#{Config.options_suffix}" do |options = {}|
+        define_singleton_method enum.options_method do |options = {}|
           dict = Dictionary.new(self, enum, options)
           dict.translate_options
         end
@@ -33,24 +33,14 @@ module EnumIsh
     end
 
     def define_predicate(enum)
-      enum.mapping.each do |enum_key, enum_value|
-        method_name = if enum.setting[:predicate].is_a?(Hash) && enum.setting[:predicate][:prefix] == false
-          "#{enum_key}?"
-        else
-          "#{enum.name}_#{enum_key}?"
-        end
-        target_value = if enum.setting[:accessor]
-          enum_key
-        else
-          enum_value
-        end
+      enum.mapping.keys.each do |key|
         @klass.class_eval do
-          define_method method_name.tr('.', '_') do
+          define_method enum.predicate_method(key) do
             value = public_send(enum.name)
             if value.is_a?(Array)
-              value == [target_value]
+              value == [enum.value(key)]
             else
-              value == target_value
+              value == enum.value(key)
             end
           end
         end
@@ -63,7 +53,7 @@ module EnumIsh
         define_method :initialize do |*args|
           super(*args)
           if respond_to?(enum.name) && public_send(enum.name).nil?
-            default = enum.setting[:default]
+            default = enum.default
             default = instance_exec(&default) if default.kind_of?(Proc)
             public_send("#{enum.name}=", default)
           end
@@ -74,7 +64,7 @@ module EnumIsh
 
     def define_accessor(enum)
       @klass.class_eval do
-        define_method "#{Config.raw_prefix}#{enum.name}#{Config.raw_suffix}" do
+        define_method enum.raw_method do
           instance_variable_get("@#{enum.name}")
         end
         define_method "#{enum.name}" do
@@ -87,13 +77,8 @@ module EnumIsh
     end
 
     def define_validate(enum)
-      targets = if enum.setting[:accessor]
-        enum.mapping.keys
-      else
-        enum.mapping.values
-      end
       @klass.class_eval do
-        validates enum.name, inclusion: { in: targets }, allow_nil: true
+        validates enum.name, inclusion: { in: enum.values }, allow_nil: true
       end
     end
 

data/lib/enum_ish/dictionary.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
+require_relative 'dictionary_lookup'
+
 module EnumIsh
   class Dictionary
-    CACHE_KEY = :_enum_ish_dictionary_cache
-
     def initialize(klass, enum, options = {})
       @klass = klass
       @enum = enum
       @options = options
-      @dict = cache { Lookup.new(@klass, @enum, @options).call }
+      @dict = cache { DictionaryLookup.new(@klass, @enum, @options).call }
     end
 
     def translate_value(value)
@@ -20,13 +20,13 @@ module EnumIsh
     end
 
     def translate_options
-      @dict.to_a.map { |value, label| [label, value] }
+      @dict.to_a.map { |value, text| [text, value] }
     end
 
     private
 
     def cache
-      if (cache = Thread.current[CACHE_KEY]) != nil
+      if (cache = EnumIsh::DictionaryCache.cache) != nil
         cache[I18n.locale] ||= {}
         cache[I18n.locale][@klass] ||= {}
         cache[I18n.locale][@klass][@enum] ||= {}
@@ -35,62 +35,5 @@ module EnumIsh
         yield
       end
     end
-
-    class << self
-      def cache
-        Thread.current[CACHE_KEY] = {}
-        yield
-      ensure
-        Thread.current[CACHE_KEY] = nil
-      end
-    end
-  end
-
-  class Lookup
-    def initialize(klass, enum, options = {})
-      @klass = klass
-      @enum = enum
-      @options = options
-    end
-
-    def call
-      i18n = lookup_for(@klass).transform_keys { |k| k.to_s.to_sym }
-
-      dict = {}
-      if @enum.setting[:accessor]
-        @enum.mapping.each { |k, v| dict[k] = i18n[k].to_s }
-      else
-        @enum.mapping.each { |k, v| dict[v] = i18n[k].to_s }
-      end
-
-      filter(dict)
-    end
-
-    private
-
-    def lookup_for(klass)
-      key = [@enum.name, @options[:format]].compact.join('/')
-      options = (@options[:i18n_options] || {}).merge(default: nil)
-
-      if klass.name.to_s.in?(['ActiveRecord::Base', 'Object'])
-        I18n.t(:"enum_ish.defaults.#{key}", **options) || @enum.mapping.invert
-      elsif klass.name.blank? || !klass.is_a?(Class)
-        resolve(klass.superclass)
-      else
-        I18n.t(:"enum_ish.#{klass.name.underscore}.#{key}", **options) || lookup_for(klass.superclass)
-      end
-    end
-
-    def filter(dict)
-      if @options[:except]
-        except = Array(@options[:except])
-        dict.reject! { |k, v| except.include?(k) }
-      end
-      if @options[:only]
-        only = Array(@options[:only])
-        dict.select! { |k, v| only.include?(k) }
-      end
-      dict
-    end
   end
 end

data/lib/enum_ish/dictionary_cache.rb

@@ -7,9 +7,25 @@ module EnumIsh
     end
   
     def call(env)
-      EnumIsh::Dictionary.cache do
+      EnumIsh::DictionaryCache.enable do
         @app.call(env)
       end
     end
+
+    class << self
+      class_attribute :cache_key
+      self.cache_key = :_enum_ish_dictionary_cache
+
+      def cache
+        Thread.current[cache_key]
+      end
+
+      def enable
+        Thread.current[cache_key] = {}
+        yield
+      ensure
+        Thread.current[cache_key] = nil
+      end
+    end
   end
 end

data/lib/enum_ish/dictionary_lookup.rb

@@ -0,0 +1,51 @@
+module EnumIsh
+  class DictionaryLookup
+    def initialize(klass, enum, options = {})
+      @klass = klass
+      @enum = enum
+      @options = options
+    end
+
+    def call
+      i18n = lookup_for(@klass).transform_keys { |k| k.to_s.to_sym }
+
+      dict = {}
+      if @enum.use?(:accessor)
+        @enum.mapping.each { |k, v| dict[k] = i18n[k] }
+      else
+        @enum.mapping.each { |k, v| dict[v] = i18n[k] }
+      end
+
+      filter(dict)
+    end
+
+    private
+
+    def lookup_for(klass)
+      if klass.name.blank? || !klass.is_a?(Class)
+        return lookup_for(klass.superclass)
+      end
+
+      key = [@enum.name, @options[:format]].compact.join('/')
+      options = (@options[:i18n_options] || {}).merge(default: nil)
+
+      if klass.name.to_s.in?(['ActiveRecord::Base', 'Object'])
+        I18n.t(:"enum_ish.defaults.#{key}", **options) || @enum.mapping
+      else
+        I18n.t(:"enum_ish.#{klass.name.underscore}.#{key}", **options) || lookup_for(klass.superclass)
+      end
+    end
+
+    def filter(dict)
+      if @options[:except]
+        except = Array(@options[:except])
+        dict.reject! { |k, v| except.include?(k) }
+      end
+      if @options[:only]
+        only = Array(@options[:only])
+        dict.select! { |k, v| only.include?(k) }
+      end
+      dict
+    end
+  end
+end

data/lib/enum_ish/enum.rb

@@ -2,6 +2,9 @@
 
 module EnumIsh
   class Enum
+    class_attribute :features
+    self.features = [:text, :options, :predicate, :default, :accessor, :validate, :scope]
+
     attr_accessor :name
     attr_accessor :mapping
     attr_accessor :setting
@@ -9,7 +12,37 @@ module EnumIsh
     def initialize(name, mapping, setting)
       @name = name
       @mapping = init_mapping(mapping)
-      @setting = init_setting(setting)
+      @setting = setting
+    end
+
+    features.each do |feature|
+      define_method feature do
+        setting.fetch(feature)
+      end
+    end
+
+    def use?(feature)
+      @setting.key?(feature)
+    end
+
+    def features
+      self.class.features.select { |feature| use?(feature) }
+    end
+
+    def values
+      if use?(:accessor)
+        @mapping.keys
+      else
+        @mapping.values
+      end
+    end
+
+    def value(key)
+      if use?(:accessor)
+        key
+      else
+        @mapping[key]
+      end
     end
 
     def to_raw(value)
@@ -29,6 +62,35 @@ module EnumIsh
       end
     end
 
+    def text_method
+      "#{Config.text_prefix}#{@name}#{Config.text_suffix}"
+    end
+
+    def options_method
+      "#{Config.options_prefix}#{@name}#{Config.options_suffix}"
+    end
+
+    def predicate_method(key)
+      key = key.to_s.tr('.', '_')
+      if predicate.is_a?(Hash) && predicate[:prefix] == false
+        "#{key}?"
+      else
+        "#{@name}_#{key}?"
+      end
+    end
+
+    def raw_method
+      "#{Config.raw_prefix}#{@name}#{Config.raw_suffix}"
+    end
+
+    def scope_method(type = nil)
+      if type == :negative
+        "#{Config.scope_prefix}#{@name}_not#{Config.scope_suffix}"
+      else
+        "#{Config.scope_prefix}#{@name}#{Config.scope_suffix}"
+      end
+    end
+
     private
 
     def init_mapping(mapping)
@@ -42,12 +104,5 @@ module EnumIsh
         mapping
       end
     end
-
-    def init_setting(setting)
-      [:text, :options].each do |key|
-        setting[key] = true unless setting.key?(key)
-      end
-      setting
-    end
   end
 end

data/lib/enum_ish/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module EnumIsh
-  VERSION = '1.5.1'
+  VERSION = '1.6.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: enum_ish
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.6.0
 platform: ruby
 authors:
 - Yoshikazu Kaneta
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-07-05 00:00:00.000000000 Z
+date: 2021-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -123,6 +123,7 @@ files:
 - lib/enum_ish/definer.rb
 - lib/enum_ish/dictionary.rb
 - lib/enum_ish/dictionary_cache.rb
+- lib/enum_ish/dictionary_lookup.rb
 - lib/enum_ish/enum.rb
 - lib/enum_ish/errors.rb
 - lib/enum_ish/version.rb
@@ -144,7 +145,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: A ruby and rails extension to generate enum-like methods