display_case 0.0.5 → 0.0.6

data/README.md

@@ -32,7 +32,9 @@ Your exhibits will look something like this:
 # app/exhibits/league_exhibit.rb
 
 class LeagueExhibit < DisplayCase::Exhibit
-  def self.applicable_to?(object)
+  # Note: the context parameter is new in the master branch, not yet released in the gem. 
+  # If you get an argument error, that's why
+  def self.applicable_to?(object, context) 
     object.class.name == 'League'
   end
   

data/lib/display_case/basic_exhibit.rb

@@ -0,0 +1,18 @@
+require_relative 'exhibit'
+
+module DisplayCase
+  class BasicExhibit < Exhibit
+    def to_partial_path
+      if __getobj__.respond_to?(:to_partial_path)
+        __getobj__.to_partial_path.dup
+      else
+        partialize_name(__getobj__.class.name)
+      end
+    end
+
+    def render(template)
+      template.render(:partial => to_partial_path, :object => self)
+    end
+  end
+end
+

data/lib/display_case/configuration.rb

@@ -15,7 +15,7 @@ module DisplayCase
     # dynamically collected with the inherited callback. By default, this is false
     # and the list will be generated via the inherited callback.
     attr_accessor :explicit
-    
+
     # An Array of strings specifying locations that should be searched for
     # exhibit classes and definitions. By default, "/app/exhibits" will be searched and
     # existing file will be loaded.
@@ -32,7 +32,7 @@ module DisplayCase
     end
 
     def exhibits
-      [DisplayCase::Exhibit::Exhibited,DisplayCase::EnumerableExhibit] + @exhibits
+      [DisplayCase::Exhibit::Exhibited,DisplayCase::BasicExhibit,DisplayCase::EnumerableExhibit] + @exhibits
     end
 
     def exhibits=(val)
@@ -41,4 +41,4 @@ module DisplayCase
   end
 
   configure {}
-end
+end

data/lib/display_case/enumerable_exhibit.rb

@@ -4,7 +4,7 @@ module DisplayCase
   class EnumerableExhibit < Exhibit
     include Enumerable
 
-    def self.applicable_to?(object)
+    def self.applicable_to?(object, context=nil)
       # ActiveRecord::Relation, surprisingly, is not Enumerable. But it
       # behaves sufficiently similarly for our purposes.
       object_is_any_of?(object, 'Enumerable', 'ActiveRecord::Relation')
@@ -34,9 +34,10 @@ module DisplayCase
     end
 
     def each(*)
-      super do |e|
-        yield exhibit(e)
-      end
+      __getobj__.map do |e|
+        yield exhibit(e) if block_given?
+        exhibit(e)
+      end.each
     end
 
     # `render '...', :collection => self` will call #to_ary on this

data/lib/display_case/exhibit.rb

@@ -1,12 +1,14 @@
 require 'delegate'
 require 'active_support/core_ext'
-require 'display_case/railtie' if defined?(Rails)
+require 'display_case/railtie' if defined?(::Rails)
 require_relative 'configuration'
+require_relative 'is_a_class_comparator'
+require_relative 'name_class_comparator'
 
 module DisplayCase
   class Exhibit < SimpleDelegator
     @@exhibits = []
-  
+
     def self.exhibits
       if DisplayCase.configuration.explicit?
         DisplayCase.configuration.exhibits
@@ -14,36 +16,36 @@ module DisplayCase
         @@exhibits
       end
     end
-  
+
     def self.inherited(child)
       @@exhibits << child
     end
 
-    def self.exhibit(object, context)
+    def self.exhibit(object, context=nil)
       return object if exhibited_object?(object)
-      if defined? Rails
-        Rails.logger.debug "Registered exhibits: #{@@exhibits}"
-        Rails.logger.debug "Exhibiting #{object.inspect}"
-        Rails.logger.debug "Exhibit context: #{context}"
+      if defined? ::Rails
+        ::Rails.logger.debug "Registered exhibits: #{@@exhibits}"
+        ::Rails.logger.debug "Exhibiting #{object.inspect}"
+        ::Rails.logger.debug "Exhibit context: #{context}"
       end
-      
-      object = Exhibited.new(object, context)
+
+      object = BasicExhibit.new(Exhibited.new(object, context), context)
       exhibits.inject(object) do |object, exhibit_class|
         exhibit_class.exhibit_if_applicable(object, context)
       end.tap do |obj|
-        Rails.logger.debug "Exhibits applied: #{obj.inspect_exhibits}" if defined? Rails
+        ::Rails.logger.debug "Exhibits applied: #{obj.inspect_exhibits}" if defined? ::Rails
       end
     end
 
     def self.exhibit_if_applicable(object, context)
-      if applicable_to?(object)
+      if applicable_to?(object, context)
         new(object, context)
       else
         object
       end
     end
 
-    def self.applicable_to?(object)
+    def self.applicable_to?(object, context=nil)
       false
     end
 
@@ -60,18 +62,25 @@ module DisplayCase
     end
     private_class_method :exhibit_query
 
+    def self.class_comparator
+      @@class_comparator ||= begin
+        comparator = nil
+
+        if defined? ::Rails
+          config = ::Rails.respond_to?(:config) ? ::Rails.config : ::Rails.application.config
+          comparator = NameClassComparator.new unless config.cache_classes
+        end
+
+        comparator || IsAClassComparator.new
+      end
+    end
+
     # A helper for matching models to classes/modules, intended for use
     # in .applicable_to?.
     def self.object_is_any_of?(object, *classes)
-      # What with Rails development mode reloading making class matching
-      # unreliable, plus wanting to avoid adding dependencies to
-      # external class definitions if we can avoid it, we just match
-      # against class/module name strings rather than the actual class
-      # objects.
-
-      # Note that '&' is the set intersection operator for Arrays.
-      (classes.map(&:to_s) & object.class.ancestors.map(&:name)).any?
+      self.class_comparator.call(object, *classes)
     end
+
     private_class_method :object_is_any_of?
 
     attr_reader :context
@@ -85,25 +94,26 @@ module DisplayCase
     def class
       __getobj__.class
     end
-
-    def exhibit(model)
-      Exhibit.exhibit(model, context)
+    
+    alias_method :__kind_of__?, :kind_of?
+    def kind_of?(klass)
+      __getobj__.kind_of?(klass)
     end
-
-    def to_partial_path
-      if __getobj__.respond_to?(:to_partial_path)
-        __getobj__.to_partial_path.dup
-      else
-        partialize_name(__getobj__.class.name)
-      end
+    
+    alias_method :__is_a__?, :is_a?
+    alias_method :is_a?, :kind_of?
+    
+    alias_method :__instance_of__?, :instance_of?
+    def instance_of?(klass)
+      __getobj__.instance_of?(klass)
     end
 
-    def render(template)
-      template.render(:partial => to_partial_path, :object => self)
+    def exhibit(model)
+      Exhibit.exhibit(model, context)
     end
 
     def exhibit_chain
-      inner_exhibits = defined?(super) ? super : []
+      inner_exhibits = __getobj__.respond_to?(:exhibit_chain) ? __getobj__.exhibit_chain : []
       [__class__] + inner_exhibits
     end
 
@@ -137,4 +147,4 @@ module DisplayCase
       "/#{name.underscore.pluralize}/#{name.demodulize.underscore}"
     end
   end
-end
+end

data/lib/display_case/is_a_class_comparator.rb

@@ -0,0 +1,38 @@
+module DisplayCase
+  class IsAClassComparator
+    def call(object, *classes)
+      classes.any?{|klass| object.is_a?(class_object(klass))}
+    end
+
+    private
+    
+    # A simple memoizing class name tracker
+    class ClassTracker
+      def initialize
+        @context = Object
+        @data = Hash.new {|h, k| h[k] = constantize(k)}
+      end
+
+      def class_for(class_name)
+        @data[class_name]
+      end
+      
+      def constantize(class_name)
+        return class_name if Module === class_name
+        class_name.to_s.split('::').inject(@context){|context, name|
+          context.const_get(name)
+        }
+      end
+    end
+
+    # holds a reference to the tracker
+    def class_tracker
+      @class_tracker ||= ClassTracker.new
+    end
+
+    # helper method for object_is_any_of to use
+    def class_object(cls)
+      class_tracker.class_for(cls)
+    end
+  end
+end

data/lib/display_case/name_class_comparator.rb

@@ -0,0 +1,13 @@
+module DisplayCase
+  # What with Rails development mode reloading making class matching
+  # unreliable, plus wanting to avoid adding dependencies to external
+  # class definitions if we can avoid it, this class just matches
+  # against class/module name strings rather than the actual class
+  # objects.
+  class NameClassComparator
+    def call(object, *classes)
+      # Note that '&' is the set intersection operator for Arrays.
+      (classes.map(&:to_s) & object.class.ancestors.map(&:name)).any?
+    end
+  end
+end

data/lib/display_case.rb

@@ -1,6 +1,7 @@
 require_relative 'display_case/configuration'
 require_relative 'display_case/exhibit'
 require_relative 'display_case/enumerable_exhibit'
+require_relative 'display_case/basic_exhibit'
 require_relative 'display_case/exhibits_helper'
 require_relative 'display_case/find_definitions'
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: display_case
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-10-15 00:00:00.000000000 Z
+date: 2013-02-14 00:00:00.000000000Z
 dependencies: []
 description: An implementation of the Exhibit pattern, as described in Objects on
   Rails
@@ -19,11 +19,14 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- lib/display_case/basic_exhibit.rb
 - lib/display_case/configuration.rb
 - lib/display_case/enumerable_exhibit.rb
 - lib/display_case/exhibit.rb
 - lib/display_case/exhibits_helper.rb
 - lib/display_case/find_definitions.rb
+- lib/display_case/is_a_class_comparator.rb
+- lib/display_case/name_class_comparator.rb
 - lib/display_case/railtie.rb
 - lib/display_case.rb
 - LICENSE
@@ -48,7 +51,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.8
 signing_key: 
 specification_version: 3
 summary: An implementation of the Exhibit pattern, as described in Objects on Rails