property_sets 3.6.0 → 3.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 83bde37b6c1a385dc5372f75424eda44a32211093219e2aa793dc6a42e8a40b6
-  data.tar.gz: 33689e9a2946082c6acf3681de60101cd0c1c8e728500fa891adfa91347c6257
+  metadata.gz: d4202c89da527945bdf58692cdd2c99c98090b130b674605ad4d97d6938b48f7
+  data.tar.gz: c1ce9e1690953791c8325405ce6fe398b251acc2095868e238146d5e560fc173
 SHA512:
-  metadata.gz: d5937f372c44d22aee6647680c98be68a1506720e7115b49be0b29e8bdaf85f3f0e80afa2ee3d246875fe2cf076283255101a5d3bc9e34f6aed8643259f9a725
-  data.tar.gz: a477ec210da5626275ca6eca932f56d382f68c3f3aca1a74082522cd6614afb48b113cf822948a11c52008f643edca24e71f01c6f34fdc8f5682ed718a8e79de
+  metadata.gz: 6f62d70197d3fcb43434d5504187deb5667275f9a0a34d7884f2b57e23ab0e84122ed4f56c68f48a55ac74cd97ef9a7d59a042e9e1fa3e005592f09abe5c9bee
+  data.tar.gz: 3225b1bf41f59c63b43dd5ebce79f048b6933fc47345d2089a03bbcbf613b5e1c0877dbc508647a187f2ca999d88375263ec5e3bd9a660b469daf0fad669a027

data/lib/property_sets.rb

@@ -10,16 +10,19 @@ end
 module PropertySets
   def self.ensure_property_set_class(association, owner_class_name)
     const_name = "#{owner_class_name}#{association.to_s.singularize.camelcase}".to_sym
+
     unless Object.const_defined?(const_name)
-      property_class = Object.const_set(const_name, Class.new(ActiveRecord::Base))
-      property_class.class_eval do
+      property_class = Class.new(ActiveRecord::Base) do
         include PropertySets::PropertySetModel::InstanceMethods
         extend  PropertySets::PropertySetModel::ClassMethods
       end
 
+      Object.const_set(const_name, property_class)
+
       property_class.owner_class = owner_class_name
       property_class.owner_assoc = association
     end
+
     Object.const_get(const_name)
   end
 end

data/lib/property_sets/active_record_extension.rb

@@ -6,6 +6,8 @@ module PropertySets
   module ActiveRecordExtension
     module ClassMethods
 
+      RAILS6 = ActiveRecord::VERSION::MAJOR >= 6
+
       def property_set(association, options = {}, &block)
         unless include?(PropertySets::ActiveRecordExtension::InstanceMethods)
           self.send(:prepend, PropertySets::ActiveRecordExtension::InstanceMethods)
@@ -14,52 +16,84 @@ module PropertySets
         end
 
         raise "Invalid association name, letters only" unless association.to_s =~ /[a-z]+/
+        exists = property_set_index.include?(association)
+
         self.property_set_index << association
 
+        # eg AccountSetting - this IS idempotent
         property_class = PropertySets.ensure_property_set_class(
           association,
           options.delete(:owner_class_name) || self.name
         )
-        property_class.instance_eval(&block)
 
-        hash_opts = {:class_name => property_class.name, :autosave => true, :dependent => :destroy}.merge(options)
+        # eg property :is_awesome
+        property_class.instance_eval(&block) if block
+
+        tb_name = options.delete :table_name
+        property_class.table_name = tb_name if tb_name
+
+        hash_opts = {
+          :class_name => property_class.name,
+          :autosave => true,
+          :dependent => :destroy,
+          :inverse_of => self.name.underscore.to_sym,
+        }.merge(options)
+
+        # TODO: should check options are compatible? warn? raise?
+        reflection = self.reflections[association.to_s] # => ActiveRecord::Reflection::HasManyReflection
+        reflection.options.merge! options if reflection && !options.empty?
 
-        silence_warnings do
+        unless exists then # makes has_many idempotent...
           has_many association, hash_opts do
-            include PropertySets::ActiveRecordExtension::AssociationExtensions
-
-            property_class.keys.each do |key|
-              raise "Invalid property key #{key}" if self.respond_to?(key)
-
-              # Reports the coerced truth value of the property
-              define_method "#{key}?" do
-                type  = property_class.type(key)
-                value = lookup_value(type, key)
-                ![ "false", "0", "", "off", "n" ].member?(value.to_s.downcase)
-              end
-
-              # Returns the value of the property
-              define_method "#{key}" do
-                type = property_class.type(key)
-                lookup_value(type, key)
-              end
-
-              # Assigns a new value to the property
-              define_method "#{key}=" do |value|
-                instance = lookup(key)
-                instance.value = PropertySets::Casting.write(property_class.type(key), value)
-                instance.value
-              end
-
-              define_method "#{key}_record" do
-                lookup(key)
-              end
+            # keep this damn block! -- creates association_module below
+          end
+        end
+
+        # eg 5: AccountSettingsAssociationExtension
+        # eg 6: Account::SettingsAssociationExtension
+
+        # stolen/adapted from AR's collection_association.rb #define_extensions
+
+        module_name = "#{association.to_s.camelize}AssociationExtension"
+        module_name = name.demodulize + module_name unless RAILS6
+
+        target = RAILS6 ? self : self.parent
+        association_module = target.const_get module_name
+
+        association_module.module_eval do
+          include PropertySets::ActiveRecordExtension::AssociationExtensions
+
+          property_class.keys.each do |key|
+            raise "Invalid property key #{key}" if self.respond_to?(key)
+
+            # Reports the coerced truth value of the property
+            define_method "#{key}?" do
+              type  = property_class.type(key)
+              value = lookup_value(type, key)
+              ![ "false", "0", "", "off", "n" ].member?(value.to_s.downcase)
+            end
+
+            # Returns the value of the property
+            define_method "#{key}" do
+              type = property_class.type(key)
+              lookup_value(type, key)
+            end
+
+            # Assigns a new value to the property
+            define_method "#{key}=" do |value|
+              instance = lookup(key)
+              instance.value = PropertySets::Casting.write(property_class.type(key), value)
+              instance.value
             end
 
-            define_method :property_serialized? do |key|
-              property_class.type(key) == :serialized
+            define_method "#{key}_record" do
+              lookup(key)
             end
           end
+
+          define_method :property_serialized? do |key|
+            property_class.type(key) == :serialized
+          end
         end
       end
     end

data/lib/property_sets/property_set_model.rb

@@ -108,13 +108,16 @@ module PropertySets
         base.attr_accessible :name, :value if defined?(ProtectedAttributes)
       end
 
-      def property(key, options = nil)
+      def properties
         @properties ||= HashWithIndifferentAccess.new
-        @properties[key] = options
+      end
+
+      def property(key, options = nil)
+        properties[key] = options
       end
 
       def keys
-        @properties.keys
+        properties.keys
       end
 
       def default(key)
@@ -122,15 +125,15 @@ module PropertySets
       end
 
       def raw_default(key)
-        @properties[key].try(:[], :default)
+        properties[key].try(:[], :default)
       end
 
       def type(key)
-        @properties[key].try(:[], :type) || :string
+        properties[key].try(:[], :type) || :string
       end
 
       def protected?(key)
-        @properties[key].try(:[], :protected) || false
+        properties[key].try(:[], :protected) || false
       end
 
       def owner_class=(owner_class_name)

data/lib/property_sets/version.rb

@@ -1,3 +1,3 @@
 module PropertySets
-  VERSION = "3.6.0"
+  VERSION = "3.7.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: property_sets
 version: !ruby/object:Gem::Version
-  version: 3.6.0
+  version: 3.7.0
 platform: ruby
 authors:
 - Morten Primdahl
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-14 00:00:00.000000000 Z
+date: 2021-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -132,7 +132,7 @@ homepage: http://github.com/zendesk/property_sets
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -147,8 +147,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Property sets for ActiveRecord.
 test_files: []