commutator 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1a212280a2711c6edae1077d491db74fb9186104
-  data.tar.gz: ea5708c8299dd8cc5446e90417c3e6824b2b70b1
+  metadata.gz: 48ccc3f1fbd5002ddd24c28e9ac78c1e9768c123
+  data.tar.gz: 60eac8308078a21d24cc45112c3093ced30116f8
 SHA512:
-  metadata.gz: bdbe600527356e277e86af158e387d0c78250254ee7a07af6e92d2a91a663e44340c0c9f442049174aaa9123ab921bc9df7cc103e11e0a363ee2bb242ce7d778
-  data.tar.gz: 1dd47ee873c1fe1447eeac8ab25a5c99ae4cf51bd257c065a1537caf77ca8756aa1add7a385cdd98656683fedb40c6b65e2cf9b88c83a0cb1175ca82b5bf76fb
+  metadata.gz: f1e28a291d82044addad086e6715819711c0a1c01e57ebabdd56b4d88a2daa32d38ffcbc375611cea8dd1c5f95e7db3a5eee52f4ea0c79cfd22cb3b0edbd76b6
+  data.tar.gz: d77480d5eec6b8be9be9e77636a83422bbe81ad6011a94c6d9e9671baf2d04e673e0a56ad626c675e13121e245f7878c9be9951a5831766219faff41dd3e1180

data/Changelog.md

@@ -1,6 +1,17 @@
 ### Development
-[Full Changelog](http://github.com/tablexi/commutator/compare/v0.1.0...master)
+[Full Changelog](http://github.com/tablexi/commutator/compare/v0.1.1...master)
 
+### Development
+[Full Changelog](http://github.com/tablexi/commutator/compare/v0.1.0...v0.1.1)
+
+Features:
+
+* Add `Commutator::Collection::CachedLookup` (Bradley Schaefer)
+
+Bug fixes:
+
+* Change lazy-loading of Commutator::Model.client to be eager-loaded
+  to facilitate the ability to override the client. (Bradley Schaefer)
 
 ### 0.1.0 / 2016-01-11
 

data/lib/commutator/collection/cached_lookup.rb

@@ -0,0 +1,34 @@
+module Commutator
+  class Collection
+    # Implemention of a per-collection cache to avoid repeating expensive
+    # lookups. E.g. if several items in the collection refer to the same User
+    # object, the following pattern would ensure the User is only looked up
+    # once:
+    #
+    #     # (inside your model)
+    #     CachedUserLookup = Proc.new do
+    #       Commutator::Collection::CachedLookup.new(:user, :user_id) do |user_id|
+    #         User.find(user_id)
+    #       end
+    #     end
+    #
+    #     modify_collection_items_with CachedUserLookup, factory: true
+    #
+    # Note: The cache itself is not accessible since it's hidden by the closure
+    #       scope. Additionally, this returns the same instance of the object for
+    #       every matching lookup - so be mindful when modifying that object.
+    module CachedLookup
+      def self.new(attr_name, item_key, &block)
+        cache = Hash.new do |h, k|
+          h[k] = block.call(k)
+        end
+
+        Proc.new do |item|
+          item.define_singleton_method(attr_name) do
+            cache[item.send(item_key)]
+          end
+        end
+      end
+    end
+  end
+end

data/lib/commutator/collection.rb

@@ -5,6 +5,8 @@ module Commutator
   # NOTE: This can't use SimpleDelegator because `Seahorse::Client::Response` does
   #       not implement `#respond_to?` as needed.
   class Collection
+    autoload :CachedLookup, "commutator/collection/cached_lookup"
+
     delegate :count,
              :scanned_count,
              :last_evaluated_key,

data/lib/commutator/model.rb

@@ -48,9 +48,11 @@ module Commutator
       before_get_item :configure_default_get_item
 
       class_attribute :collection_item_modifiers, instance_accessor: false
+      class_attribute :client
+      self.client = ::Commutator::SimpleClient.new
     end
 
-    delegate :client, :options_class, to: 'self.class'
+    delegate :options_class, to: 'self.class'
 
     def initialize(attrs = {})
       assign_attributes(attrs.symbolize_keys)
@@ -130,8 +132,6 @@ module Commutator
 
     # :nodoc:
     module ClassMethods
-      attr_writer :client
-
       def inherited(subclass)
         subclass.attribute_names.merge(attribute_names)
         before_hooks.each { |k, v| subclass.before_hooks[k] = v.dup }
@@ -144,16 +144,14 @@ module Commutator
         subclass.const_set("Scopes", Module.new { include scopes }) if scopes
       end
 
-      def client
-        @client ||= ::Commutator::SimpleClient.new
-      end
-
       def create(attrs)
         new(attrs).tap { |dp| dp.put_item_options.execute }
       end
 
       def modify_collection_items_with(*modifiers, factory: false)
-        self.collection_item_modifiers = [ItemModifiers.new(modifiers, factory: factory)].unshift(*collection_item_modifiers)
+        self.collection_item_modifiers = [
+          ItemModifiers.new(modifiers, factory: factory)
+        ].unshift(*collection_item_modifiers)
       end
 
       def get_item_options

data/lib/commutator/version.rb

@@ -1,3 +1,3 @@
 module Commutator
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: commutator
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Bradley Schaefer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-01-11 00:00:00.000000000 Z
+date: 2016-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -129,6 +129,7 @@ files:
 - commutator.gemspec
 - lib/commutator.rb
 - lib/commutator/collection.rb
+- lib/commutator/collection/cached_lookup.rb
 - lib/commutator/expressions/attribute_names.rb
 - lib/commutator/expressions/attribute_values.rb
 - lib/commutator/expressions/condition_expression.rb