junk_drawer 1.4.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 238eb68d411dc0c6089296530a43a2465975d417
-  data.tar.gz: 655fec097bfbdf9aa9ad42d1b3ebb95b96f5e6de
+  metadata.gz: 9ccbaed0545acce12327a2220ffd61a1f2c89a88
+  data.tar.gz: bb229cb6b5d34628b6122a1476bb854afd474726
 SHA512:
-  metadata.gz: 97e29c483b957b9de82874825ec57a185f4b44ec4bfb00db45cdc7666fba99bc1466a0cbec786428efcbeec02843919373e1fd5177bfb4413b9e4b8b7fb5069b
-  data.tar.gz: 361c72c75c5263153d8ca6b0bcdd1fcc0cc2a158710492ba30de723f4ef1bfdc866a9745d7ab712f78a47b7769a5a7bc4297ed068a67213a3bc3f7a792582331
+  metadata.gz: dc88c0d245fe98b677b94b172968605fa4bec3070fb889e226ce2eb92b848cc24fba1c59bbe92ec99a834c66498f5642b69681a4f34ade042c0db3c6c716a287
+  data.tar.gz: 1743c1dc5149e743d231ff3aa0708ef1cfdc52b19b78f094ed03eaa2e5dbbb8a57cb2cf7def040918e47f766ff5d63898d60bd9103be6eb7062ca1adacec95cc

data/.rubocop.yml

@@ -20,7 +20,7 @@ Metrics/MethodLength:
 Metrics/BlockLength:
   Exclude:
     - junk_drawer.gemspec
-    - spec/**/*_spec.rb
+    - spec/**/*
 
 RSpec/ExampleLength:
   Enabled: false

data/Gemfile

@@ -6,3 +6,5 @@ gemspec
 
 gem 'activerecord', '~> 5.1.0'
 gem 'activesupport', '~> 5.1.0'
+gem 'hstore_accessor', '~> 1.1.1'
+gem 'jsonb_accessor', '~> 1.0.0'

data/README.md

@@ -220,6 +220,13 @@ release a new version, update the version number in `version.rb`, and then run
 `bundle exec rake release`, which will create a git tag for the version, push
 git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
+In order to run tests against different Rails versions, you can use
+`BUNDLE_GEMFILE`:
+
+```sh
+$ BUNDLE_GEMFILE=gemfiles/rails_4.2.gems rake spec
+```
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at

data/gemfiles/rails_4.2.gems

@@ -6,3 +6,5 @@ gemspec path: '../'
 
 gem 'activerecord', '~> 4.2.0'
 gem 'activesupport', '~> 4.2.0'
+gem 'hstore_accessor', '~> 1.1.1'
+gem 'jsonb_accessor', '~> 0.3.3'

data/gemfiles/rails_5.0.gems

@@ -6,3 +6,5 @@ gemspec path: '../'
 
 gem 'activerecord', '~> 5.0.0'
 gem 'activesupport', '~> 5.0.0'
+gem 'hstore_accessor', '~> 1.1.1'
+gem 'jsonb_accessor', '~> 1.0.0'

data/junk_drawer.gemspec

@@ -27,6 +27,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'guard-rubocop', '~> 1.2'
   spec.add_development_dependency 'pg', '~> 0.20'
   spec.add_development_dependency 'pry', '~> 0.10'
+  spec.add_development_dependency 'pry-byebug', '~> 3.6.0'
   spec.add_development_dependency 'rake', '~> 12.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rubocop', '~> 0.48.1'

data/lib/junk_drawer/callable.rb

@@ -19,8 +19,8 @@ module JunkDrawer
     # an instance. It also causes an error to be raised if a public instance
     # method is defined with a name other than `call`
     module ClassMethods
-      def call(*args)
-        new.(*args)
+      def call(*args, &block)
+        new.(*args, &block)
       end
 
       def to_proc

data/lib/junk_drawer/rails/bulk_updatable.rb

@@ -7,18 +7,6 @@ require 'active_record/connection_adapters/postgresql_adapter'
 module JunkDrawer
   # module to allow bulk updates for `ActiveRecord` models
   module BulkUpdatable
-    ATTRIBUTE_TYPE_TO_POSTGRES_CAST = {
-      datetime: '::timestamp',
-      hstore: '::hstore',
-      boolean: '::boolean',
-      jsonb: '::jsonb',
-    }.freeze
-
-    POSTGRES_VALUE_CASTERS = {
-      hstore: ActiveRecord::ConnectionAdapters::PostgreSQL::OID::Hstore.new,
-      jsonb: ActiveRecord::ConnectionAdapters::PostgreSQL::OID::Jsonb.new,
-    }.freeze
-
     def bulk_update(objects)
       objects = objects.select(&:changed?)
       return unless objects.any?
@@ -34,7 +22,13 @@ module JunkDrawer
       now = Time.zone.now
       objects.each { |object| object.updated_at = now }
 
-      objects.flat_map(&:changed).uniq
+      changed_attributes = objects.flat_map(&:changed).uniq
+      if ::ActiveRecord::VERSION::MAJOR >= 5
+        column_names & changed_attributes
+      else
+        # to remove virtual columns from jsonb_accessor 0.3.3
+        columns.select(&:sql_type).map(&:name) & changed_attributes
+      end
     end
 
     def build_query_for(objects, attributes)
@@ -47,7 +41,7 @@ module JunkDrawer
         "#{quoted_column_name} = tmp_table.#{quoted_column_name}"
       end.join(', ')
 
-      <<-SQL
+      <<-SQL.squish
         UPDATE #{table_name}
         SET #{assignment_query}
         FROM (VALUES #{object_values}) AS tmp_table(id, #{attributes.join(', ')})
@@ -56,29 +50,28 @@ module JunkDrawer
     end
 
     def sanitized_values(object, attributes)
-      postgres_types = attributes.map do |attribute|
-        attribute_type = columns_hash[attribute.to_s].type
-        "?#{ATTRIBUTE_TYPE_TO_POSTGRES_CAST[attribute_type]}"
-      end
-
       postgres_values = attributes.map do |attribute|
         value = object[attribute]
-        attribute_type = columns_hash[attribute.to_s].type
-        caster = POSTGRES_VALUE_CASTERS[attribute_type]
 
-        caster ? serialized_value(caster, value) : value
+        # AR internal `columns_hash`
+        column = columns_hash[attribute.to_s]
+
+        # AR internal `type_for_attribute`
+        type = type_for_attribute(column.name)
+        type_cast = "::#{column.sql_type}"
+        type_cast = "#{type_cast}[]" if column.array
+
+        "#{connection.quote(serialized_value(type, value))}#{type_cast}"
       end
 
-      sanitize_sql_array(
-        ["(?, #{postgres_types.join(', ')})", object.id, *postgres_values],
-      )
+      "(#{[object.id, *postgres_values].join(', ')})"
     end
 
-    def serialized_value(caster, value)
+    def serialized_value(type, value)
       if ::ActiveRecord::VERSION::MAJOR >= 5
-        caster.serialize(value)
+        type.serialize(value)
       else
-        caster.type_cast_for_database(value)
+        type.type_cast_for_database(value)
       end
     end
   end

data/lib/junk_drawer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module JunkDrawer
-  VERSION = '1.4.0'
+  VERSION = '1.5.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: junk_drawer
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Robert Fletcher
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-09-14 00:00:00.000000000 Z
+date: 2018-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -94,6 +94,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.10'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.6.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.6.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -230,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.6.14
 signing_key: 
 specification_version: 4
 summary: random useful Ruby utilities