impressionist 1.3.2 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 383f3c7524c62abeacef5033839a1e695e4f5065
-  data.tar.gz: 784de1284002f83371788a3f3da0a397d3848651
+  metadata.gz: 565d11269849faecb7c83d6cda308c6c1e9eba4c
+  data.tar.gz: ed373d6474fee728c3742b0d475ad5d425f9902e
 SHA512:
-  metadata.gz: 2e28487decb94bc12c205799df6fcb06fae81e1043bfb4e22b7fb4e69a36a0cd6ce15f653bb803d5fb3120f4e6ad95a1048b73715074c90766541bae2280ab1f
-  data.tar.gz: 87a2fad11f4c54e0d7e83f0b21f2c2ee09468dc3aa1cd68b0469dbc8800c99a77f0f2e34c4884352217cb62a06c8ce2c4c65a201bbc8ea0135a77c9c70b30017
+  metadata.gz: 8a4160978bf2c001b1c8f2ea24f18bc563d3cd821de5294de0f5b2da8f1c87e8e3a7c2f0c210bd7420b8a7c216383f23ffb61875bac3930475dd163f90f6a5c6
+  data.tar.gz: bc2f41f6556664349216614eec6fdc9d366f40ac0e5fbd5bcbc1eab2fd470897104581f2b19ae31267067273a49a59693edff525a6c6c34596a8c00573932f65

data/CHANGELOG.rdoc

@@ -1,3 +1,18 @@
+== 1.4.0 (2013-07-06)
+* Fixed #92
+* Pass option to unique for updatind counter_cache
+* Clean up code, make it human readable
+* Changed README in order to explain new features
+* (@nbit001)
+
+== 1.3.3 (2013-06-27)
+* Added Rails4 Branch in order to support rails 4
+* Rails 4 and Strong Parameters compability. (@bennick).
+
+== 1.3.1 (non specified)
+* Change controller_path back to controller_name (@johnmcaliley)
+
+
 == 0.4.0 (2011-06-03)
 * Fix postgres bug
 * New impression count method that accepts options for filter, start_date and end_date

data/README.md

@@ -136,14 +136,29 @@ most basic configuration looks like:
     is_impressionable :counter_cache => true
 
 This will automatically increment the `impressions_count` column in the
-included model. Note: You'll need to add that column to your model. If you'd
-like specific a different column name, you can:
+included model. <b>Note: You'll need to add that column to your model. If you'd
+like specific a different column name, you can:</b>
 
-    is_impressionable :counter_cache => { :column_name => :my_column }
+    is_impressionable :counter_cache => { :column_name => :my_column_name }
 
 If you'd like to include only unique impressions in your count:
 
-    is_impressionable :counter_cache => { :column_name => :my_column, :unique => true }
+    # default will be filtered by ip_address
+    is_impressionable :counter_cache => { :column_name => :my_column_name, :unique => true }
+
+If you'd like to specify what sort of unique impression you'd like to save? Fear not,
+Any option you pass to unique, impressionist_count will use it as its filter to update_counters based on that unique option.
+
+    # options are any column in the impressions' table.
+    is_impressionable :counter_cache => { :column_name => :my_column_name, :unique => :request_hash }
+    is_impressionable :counter_cache => { :column_name => :my_column_name, :unique => :all }
+
+
+Adding column to model
+----------------------
+It is as simple as this:
+
+    t.integer :my_column_name, :default => 0
 
 What if I only want to record unique impressions?
 -------------------------------------------------
@@ -170,7 +185,7 @@ Are you using Mongoid?
 Execute this command on your terminal/console:
 
     rails g impressionist --orm mongoid
-    
+
 This command create a file `impression.rb` on `config/initializer` folder. Add `config.orm = :mongoid` to this file:
 
     # Use this hook to configure impressionist parameters
@@ -179,7 +194,7 @@ This command create a file `impression.rb` on `config/initializer` folder. Add `
       # config.orm = :active_record
       config.orm = :mongoid
     end
-    
+
 
 Contributing to impressionist
 -----------------------------
@@ -197,10 +212,21 @@ Contributing to impressionist
 * All testing is done inside a small Rails app (test_app).  You will find specs
   within this app.
 
+
+Want to run the tests? Ok mummy
+-------------------------------
+* bundle install
+* rails g impressionist
+* rake db:migrate && rake db:test:prepare
+* Run rake or rspec spec inside test_app dir
+* nothing else.
+* :wq
+
 Contributors
 ------------
 * [johnmcaliley](https://github.com/johnmcaliley)
 * [coryschires](https://github.com/coryschires)
 * [georgmittendorfer](https://github.com/georgmittendorfer)
+* [Antonio C Nalesso](https://github.com/nbit001)
 
 Copyright (c) 2011 John McAliley. See LICENSE.txt for further details.

data/app/models/impressionist/impressionable.rb

@@ -4,59 +4,43 @@ module Impressionist
 
     module ClassMethods
       attr_accessor :impressionist_cache_options
-      @impressionist_cache_options = nil
+
+      DEFAULT_CACHE = { :counter_cache => false, :column_name => :impressions_count, :unique => false }
 
       def impressionist_counter_cache_options
-        if @impressionist_cache_options
-          options = { :column_name => :impressions_count, :unique => false }
-          options.merge!(@impressionist_cache_options) if @impressionist_cache_options.is_a?(Hash)
-          options
-        end
+        @impressionist_cache_options ||= {}
+        @impressionist_cache_options.reverse_merge!(DEFAULT_CACHE)
       end
 
+      # true or false
       def impressionist_counter_caching?
-        impressionist_counter_cache_options.present?
+        impressionist_counter_cache_options[:counter_cache]
       end
 
       def counter_caching?
-        ::ActiveSupport::Deprecation.warn("#counter_caching? is deprecated; please use #impressionist_counter_caching? instead")
-        impressionist_counter_caching?
+          ::ActiveSupport::Deprecation.warn("#counter_caching? is deprecated; please use #impressionist_counter_caching? instead")
+          impressionist_counter_caching?
       end
-    end
 
-    def impressionable?
-      true
     end
 
+    # ------------------------------------------
+    # TODO: CLEAN UP, make it HUMAN readable
     def impressionist_count(options={})
       options.reverse_merge!(:filter=>:request_hash, :start_date=>nil, :end_date=>Time.now)
       imps = options[:start_date].blank? ? impressions : impressions.where("created_at>=? and created_at<=?",options[:start_date],options[:end_date])
       options[:filter] == :all ? imps.count : imps.count(options[:filter], :distinct => true)
     end
 
-    def update_impressionist_counter_cache
-      cache_options = self.class.impressionist_counter_cache_options
-      column_name = cache_options[:column_name].to_sym
-      count = cache_options[:unique] ? impressionist_count(:filter => :ip_address) : impressionist_count
-      old_count = send(column_name) || 0
-      self.class.update_counters(id, column_name => (count - old_count))
-    end
-
-    # OLD METHODS - DEPRECATE IN V0.5
-    def impression_count(start_date=nil,end_date=Time.now)
-      impressionist_count({:start_date=>start_date, :end_date=>end_date, :filter=>:all})
-    end
+    def update_impressionist_counter_cache 
+      slave = Impressionist::UpdateCounters.new(self)
+      slave.update
+    end 
 
-    def unique_impression_count(start_date=nil,end_date=Time.now)
-      impressionist_count({:start_date=>start_date, :end_date=>end_date, :filter=> :request_hash})
-    end
-
-    def unique_impression_count_ip(start_date=nil,end_date=Time.now)
-      impressionist_count({:start_date=>start_date, :end_date=>end_date, :filter=> :ip_address})
+    def impressionable?
+      true
     end
 
-    def unique_impression_count_session(start_date=nil,end_date=Time.now)
-      impressionist_count({:start_date=>start_date, :end_date=>end_date, :filter=> :session_hash})
-    end
   end
+
 end

data/lib/impressionist.rb

@@ -1,4 +1,4 @@
-require 'impressionist/engine'
+require 'impressionist/load'
 
 module Impressionist
   # Define default ORM

data/lib/impressionist/counter_cache.rb

@@ -0,0 +1,83 @@
+module Impressionist
+  module CounterCache
+
+    attr_reader :impressionable_class, :entity
+
+    private
+      LOG_MESSAGE = "Can't find impressionable_type or impressionable_id. Will not update_counters!"
+
+      # if updatable returns true, it must be qualified to update_counters
+      # Therefore there's no need to validate again
+      # impressionable_class instance var is set when updatable? is called
+      def impressionable_counter_cache_updatable?
+        updatable? ? update_impression_cache : impressionist_log(LOG_MESSAGE)
+      end
+
+      def update_impression_cache
+        impressionable_find
+        impressionable_try
+      end
+
+      # asks imp_id whether it's present or not
+      # also expect imp_class to be true
+      # all should be true, so that it is updatable
+      def updatable?
+        @impressionable_class = impressionable_class_set
+        impressionable_valid?
+      end
+
+      # imps_type == nil, constantize returns Object
+      # Therefore it attemps to return false so it won't be updatable
+      # calls to_s otherwise it would try to constantize nil
+      # and it would raise an exeception..
+      def impressionable_class_set
+        _type_ = self.impressionable_type.to_s.constantize
+        ((_type_.to_s !~ /Object/) && _type_.impressionist_counter_caching? ? _type_ : false)
+      end
+
+      # Either true or false
+      # needs true to be updatable
+      def impressionable_valid?
+        (self.impressionable_id.present? && impressionable_class && impressionable_find)
+      end
+
+      # Logs to log file, expects a message to be passed
+      # default mode is ERROR
+      def impressionist_log(mode=:error, str)
+        Rails.logger.send(mode.to_s, str)
+      end
+
+      # read it out and LOUD
+      def impressionable_find
+        exeception_rescuer {
+          @entity = impressionable_class.find(self.impressionable_id)
+        }
+        @entity
+
+      end
+
+      # receives an entity(instance of a Model) and then tries to update
+      # counter_cache column
+      # entity is a impressionable_model
+      def impressionable_try
+        entity.try(:update_impressionist_counter_cache)
+      end
+
+      # Returns false, as it is only handling one exeception
+      # It would make updatable to fail thereafter it would not try
+      # to update cache_counter
+      def exeception_rescuer
+          begin
+            yield
+          rescue ActiveRecord::RecordNotFound
+            exeception_to_log
+            false
+          end
+      end
+
+      def exeception_to_log
+        impressionist_log("Couldn't find Widget with id=#{self.impressionable_id}")
+      end
+
+  end
+end

data/lib/impressionist/load.rb

@@ -0,0 +1,7 @@
+require "impressionist/engine"
+
+require "impressionist/set_up_association"
+
+require "impressionist/counter_cache"
+
+require "impressionist/update_counters"

data/lib/impressionist/models/active_record/impression.rb

@@ -1,21 +1,13 @@
-class Impression < ActiveRecord::Base
-  attr_accessible :impressionable_type, :impressionable_id, :user_id,
-  :controller_name, :action_name, :view_name, :request_hash, :ip_address,
-  :session_hash, :message, :referrer
-
-  belongs_to :impressionable, :polymorphic=>true
+# Responsability
+# * be able to update_counters
+# * log an error if imps_id and imps_type can not be found
+# asks updatable? whether it may or may not be updated
+# FIX exeception raising when no imps_id is found
 
-  after_save :update_impressions_counter_cache
+class Impression < ActiveRecord::Base
+  include Impressionist::CounterCache
+  include Impressionist::SetUpAssociation 
 
-  private
+  after_save :impressionable_counter_cache_updatable?
 
-  def update_impressions_counter_cache
-    if self.impressionable_type && self.impressionable_id
-      impressionable_class = self.impressionable_type.constantize
-      if impressionable_class.impressionist_counter_cache_options
-        resouce = impressionable_class.find(self.impressionable_id)
-        resouce.try(:update_impressionist_counter_cache)
-      end
-    end
-  end
 end

data/lib/impressionist/models/active_record/impressionist/impressionable.rb

@@ -2,13 +2,27 @@ ActiveRecord::Base.send(:include, Impressionist::Impressionable)
 
 module Impressionist
   module Impressionable
+
     extend ActiveSupport::Concern
 
     module ClassMethods
       def is_impressionable(options={})
-        has_many :impressions, :as => :impressionable, :dependent => :destroy
-        @impressionist_cache_options = options[:counter_cache]
+        define_association
+        imp_cache_options_set(options)
+      end
+
+      def define_association
+        has_many(:impressions,
+        :as => :impressionable,
+        :dependent => :destroy)
+      end
+
+      def imp_cache_options_set(options)
+        @impressionist_cache_options = options
       end
+
     end
+
   end
+
 end

data/lib/impressionist/set_up_association.rb

@@ -0,0 +1,14 @@
+module Impressionist
+  module SetUpAssociation
+
+    def self.included(base)
+      base.attr_accessible(:impressionable_type,:impressionable_id,
+      :user_id,:controller_name,:action_name,:view_name,:request_hash,
+      :ip_address,:session_hash,:message,:referrer)
+
+      base.belongs_to(:impressionable, :polymorphic => true)
+    end
+
+  end
+
+end

data/lib/impressionist/update_counters.rb

@@ -0,0 +1,63 @@
+module Impressionist
+  class UpdateCounters
+    attr_reader :receiver, :master
+
+    def initialize(receiver)
+      @receiver = receiver
+      @master = receiver.class
+    end
+
+    def update
+      result = (impressions_total - impressions_cached)
+
+      master.
+      update_counters(id, column_name => result)
+    end
+
+    private
+      def id
+        receiver.id
+      end
+
+      # if unique == true then uses it
+      # otherwise just count all impressions
+      # using filter: :all
+      def impressions_total
+        receiver.impressionist_count filter(unique_filter)
+      end
+
+      # from a given db column
+      # default should be impressions_count
+      def impressions_cached
+        receiver.send(column_name) || 0
+      end
+
+      def column_name
+        cache_options[:column_name].to_sym
+      end
+
+      def cache_options
+        master. 
+        impressionist_counter_cache_options
+      end 
+
+      # default is ip_address if what_is_unique is TRUE or FALSE 
+      def unique_filter
+       what_is_unique? ? :ip_address : cache_options[:unique]
+      end
+
+      # Either true or false
+      # :filter gets assigned to :ip_address as default
+      # One could do
+      # is_impressionable :counter_cache => true, :unique => :any_other_colum
+      def what_is_unique?
+        cache_options[:unique].to_s =~ /true|false/
+      end
+
+      def filter(filter_will_be)
+        {:filter => filter_will_be.to_sym}
+      end
+
+  end
+
+end

data/lib/impressionist/version.rb

@@ -1,3 +1,3 @@
 module Impressionist
-  VERSION = "1.3.2"
+  VERSION = "1.4.0"
 end

data/test_app/Gemfile

@@ -1,6 +1,6 @@
 source 'https://rubygems.org'
 
-gem 'rails', '3.2.2'
+gem 'rails', '3.2.12'
 
 gem 'impressionist', :path => '../'
 
@@ -41,7 +41,7 @@ group :test do
   gem 'systemu'
 end
 
-#gem 'jquery-rails'
+gem 'jquery-rails'
 
 # To use ActiveModel has_secure_password
 # gem 'bcrypt-ruby', '~> 3.0.0'

data/test_app/app/models/widget.rb

@@ -1,3 +1,3 @@
 class Widget < ActiveRecord::Base
-  is_impressionable :counter_cache => true
+  is_impressionable :counter_cache => true, :unique => :request_hash
 end

data/test_app/db/migrate/20111127184039_create_widgets.rb

@@ -2,7 +2,7 @@ class CreateWidgets < ActiveRecord::Migration
   def self.up
     create_table :widgets do |t|
       t.string :name
-      t.integer :impressions_count
+      t.integer :impressions_count, :default => 0
 
       t.timestamps
     end

data/test_app/spec/controllers/controller_spec.rb

@@ -131,4 +131,4 @@ describe DummyController do
     get "index"
     Impression.all.size.should eq 12
   end
-end
+end

data/test_app/spec/models/counter_caching_spec.rb

@@ -3,8 +3,9 @@ require 'spec_helper'
 describe Impression do
   fixtures :widgets
 
+  let(:widget) { Widget.find(1) }
+
   before(:each) do
-    @widget = Widget.find(1)
     Impression.destroy_all
   end
 
@@ -33,17 +34,17 @@ describe Impression do
 
   describe "#update_impressionist_counter_cache" do
     it "should update the counter cache column to reflect the correct number of impressions" do
-      lambda {
-         @widget.impressions.create(:request_hash => 'abcd1234')
-         @widget.reload
-       }.should change(@widget, :impressions_count).from(0).to(1)
+      expect {
+         widget.impressions.create(:request_hash => 'abcd1234')
+         widget.reload
+       }.to change(widget, :impressions_count).from(0).to(1)
     end
 
     it "should not update the timestamp on the impressable" do
-      lambda {
-         @widget.impressions.create(:request_hash => 'abcd1234')
-         @widget.reload
-       }.should_not change(@widget, :updated_at)
+      expect {
+         widget.impressions.create(:request_hash => 'abcd1234')
+         widget.reload
+       }.to_not change(widget, :updated_at)
     end
   end
 

data/test_app/spec/models/model_spec.rb

@@ -51,7 +51,7 @@ describe Impression do
 
   # tests :dependent => :destroy
   it "should delete impressions on deletion of impressionable" do
-    impressions_count = Impression.all.size
+    #impressions_count = Impression.all.size
     a = Article.create
     i = a.impressions.create
     a.destroy
@@ -59,36 +59,4 @@ describe Impression do
     i.destroyed?.should be_true
   end
 
-  #OLD COUNT METHODS.  DEPRECATE SOON
-  it "should return the impression count with no date range specified" do
-    @article.impression_count.should eq 11
-  end
-
-  it "should return unique impression count with no date range specified" do
-    @article.unique_impression_count.should eq 9
-  end
-
-  it "should return impression count with only start date specified" do
-    @article.impression_count("2011-01-01").should eq 8
-  end
-
-  it "should return impression count with whole date range specified" do
-    @article.impression_count("2011-01-01","2011-01-02").should eq 7
-  end
-
-  it "should return unique impression count with only start date specified" do
-    @article.unique_impression_count("2011-01-01").should eq 7
-  end
-
-  it "should return unique impression count with date range specified" do
-    @article.unique_impression_count("2011-01-01","2011-01-02").should eq 7
-  end
-
-  it "should return unique impression count using ip address (which in turn eliminates duplicate request_hashes)" do
-    @article.unique_impression_count_ip.should eq 8
-  end
-
-  it "should return unique impression count using session_hash (which in turn eliminates duplicate request_hashes)" do
-    @article.unique_impression_count_session.should eq 7
-  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: impressionist
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.4.0
 platform: ruby
 authors:
 - johnmcaliley
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-27 00:00:00.000000000 Z
+date: 2013-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httpclient
@@ -178,13 +178,17 @@ files:
 - lib/impressionist.rb
 - lib/impressionist/bots.rb
 - lib/impressionist/controllers/mongoid/impressionist_controller.rb
+- lib/impressionist/counter_cache.rb
 - lib/impressionist/engine.rb
+- lib/impressionist/load.rb
 - lib/impressionist/models/active_record/impression.rb
 - lib/impressionist/models/active_record/impressionist/impressionable.rb
 - lib/impressionist/models/mongo_mapper/impression.rb
 - lib/impressionist/models/mongo_mapper/impressionist/impressionable.rb
 - lib/impressionist/models/mongoid/impression.rb
 - lib/impressionist/models/mongoid/impressionist/impressionable.rb
+- lib/impressionist/set_up_association.rb
+- lib/impressionist/update_counters.rb
 - lib/impressionist/version.rb
 - logo.png
 - spec/spec_helper.rb