Cleanup trailing whitespace

Trailing whitespace isn't friendly to git or vim users.

CHANGELOG.rdoc

@@ -16,4 +16,4 @@
 * @widget.unique_impression_count now uses request_hash.  This was incorrectly stated in the README, since it was using ip_address.  The README is correct as a result of the method change.
 
 == 0.2.4 (2011-02-17)
-* Fix issue #1 - action_name and controller_name were not being logged for impressionist method inside action
+* Fix issue #1 - action_name and controller_name were not being logged for impressionist method inside action

VERSION

@@ -1 +1 @@
-1.0.1
+1.0.1

app/controllers/impressionist_controller.rb

@@ -39,22 +39,22 @@ def impressionist_subapp_filter(actions=nil,unique_opts=nil)
     end
 
     private
-    
+
     def bypass
       Impressionist::Bots::WILD_CARDS.each do |wild_card|
         return true if request.user_agent and request.user_agent.downcase.include? wild_card
       end
       Impressionist::Bots::LIST.include? request.user_agent
     end
-    
+
     def unique_instance?(impressionable, unique_opts)
       return unique_opts.blank? || !impressionable.impressions.where(unique_query(unique_opts)).exists?
     end
-    
+
     def unique?(unique_opts)
       return unique_opts.blank? || !Impression.where(unique_query(unique_opts)).exists?
     end
-    
+
     # creates the query to check for uniqueness
     def unique_query(unique_opts)
       full_statement = direct_create_statement
@@ -64,7 +64,7 @@ def unique_query(unique_opts)
         query
       end
     end
-    
+
     # creates a statment hash that contains default values for creating an impression via an AR relation.
     def associative_create_statement(query_params={})
       query_params.reverse_merge!(
@@ -77,7 +77,7 @@ def associative_create_statement(query_params={})
         :referrer => request.referer
         )
     end
-    
+
     # creates a statment hash that contains default values for creating an impression.
     def direct_create_statement(query_params={})
       query_params.reverse_merge!(
@@ -86,7 +86,7 @@ def direct_create_statement(query_params={})
         )
       associative_create_statement(query_params)
     end
-    
+
     def session_hash
       # # careful: request.session_options[:id] encoding in rspec test was ASCII-8BIT
       # # that broke the database query for uniqueness. not sure if this is a testing only issue.
@@ -95,12 +95,12 @@ def session_hash
       # # request.session_options[:id].encode("ISO-8859-1")
       request.session_options[:id]
     end
-    
+
     #use both @current_user and current_user helper
     def user_id
       user_id = @current_user ? @current_user.id : nil rescue nil
       user_id = current_user ? current_user.id : nil rescue nil if user_id.blank?
       user_id
     end
   end
-end
+end

app/models/impression.rb

@@ -13,4 +13,4 @@ def update_impressions_counter_cache
       resouce.try(:update_counter_cache)
     end
   end
-end
+end

app/models/impressionist/bots.rb

@@ -1,8 +1,8 @@
 module Impressionist
   module Bots
     WILD_CARDS = ["bot","yahoo","slurp","google","msn","crawler"]
-    
-    LIST = ["<a href='http://www.unchaos.com/'> UnChaos </a> From Chaos To Order Hybrid Web Search Engine.([email protected])", 
+
+    LIST = ["<a href='http://www.unchaos.com/'> UnChaos </a> From Chaos To Order Hybrid Web Search Engine.([email protected])",
       "<a href='http://www.unchaos.com/'> UnChaos Bot Hybrid Web Search Engine. </a> ([email protected])",
       "<b> UnChaosBot From Chaos To Order UnChaos Hybrid Web Search Engine at www.unchaos.com </b> ([email protected])",
       "<http://www.sygol.com/> http://www.sygol.com",
@@ -1459,4 +1459,4 @@ module Bots
       "zspider/0.9-dev http://feedback.redkolibri.com/",
       "ZyBorg/1.0 ([email protected]; http://www.WISEnut.com)"]
   end
-end
+end

config/routes.rb

@@ -1,2 +1,2 @@
 Rails.application.routes.draw do
-end
+end

lib/generators/impressionist/impressionist_generator.rb

@@ -1,5 +1,5 @@
 require 'rails/generators'
-require 'rails/generators/migration'     
+require 'rails/generators/migration'
 
 class ImpressionistGenerator < Rails::Generators::Base
   include Rails::Generators::Migration
@@ -17,4 +17,4 @@ def self.next_migration_number(dirname)
   def create_migration_file
     migration_template 'create_impressions_table.rb', 'db/migrate/create_impressions_table.rb'
   end
-end
+end

lib/generators/impressionist/templates/create_impressions_table.rb

@@ -34,4 +34,4 @@ def self.down
 
     drop_table :impressions
   end
-end
+end

lib/impressionist/bots.rb

@@ -15,4 +15,4 @@ def self.consume
       list
     end
   end
-end
+end

lib/impressionist/engine.rb

@@ -14,4 +14,4 @@ class Engine < Rails::Engine
       end
     end
   end
-end
+end

test_app/app/controllers/application_controller.rb

@@ -1,7 +1,7 @@
 class ApplicationController < ActionController::Base
   protect_from_forgery
   before_filter :secondary_before_filter
-  
+
   def secondary_before_filter
     @test_secondary_before_filter = "this is a test"
   end

test_app/app/controllers/articles_controller.rb

@@ -1,18 +1,18 @@
 class ArticlesController < ApplicationController
   before_filter :test_current_user_var
-  
+
   def test_current_user_var
     if session[:user_id]
       @current_user = User.new
       @current_user.id = session[:user_id]
     end
   end
-  
+
   def index
     impressionist(Article.first,"this is a test article impression")
   end
-  
+
   def show
     impressionist(Article.first)
   end
-end
+end

test_app/app/controllers/posts_controller.rb

@@ -2,22 +2,22 @@ class PostsController < ApplicationController
   helper_method :current_user
   impressionist
   def index
-    
+
   end
-  
+
   def show
-    
+
   end
-  
+
   def edit
-    
+
   end
-  
+
   def current_user
     if session[:user_id]
       user = User.new
       user.id = session[:user_id]
       @current_user ||= user
     end
   end
-end
+end

test_app/app/controllers/widgets_controller.rb

@@ -1,13 +1,13 @@
 class WidgetsController < ApplicationController
   impressionist :actions=>[:show,:index], :unique => [:controller_name,:action_name,:impressionable_id]
-  
+
   def show
   end
-  
+
   def index
   end
-  
+
   def new
   end
-  
-end
+
+end

test_app/app/models/dummy.rb

@@ -4,4 +4,4 @@
 class Dummy < ActiveRecord::Base
   self.abstract_class = true # doesn't need to be backed by an actual table
   is_impressionable
-end
+end

test_app/app/models/user.rb

@@ -1,3 +1,3 @@
 class User
   attr_accessor :id
-end
+end

test_app/app/views/articles/index.html.erb

@@ -1 +1 @@
-<%=@impressionist_hash==nil%>
+<%=@impressionist_hash==nil%>

test_app/app/views/articles/show.html.erb

@@ -1 +1 @@
-<%=link_to "Same Page", article_url(Article.first)%>
+<%=link_to "Same Page", article_url(Article.first)%>

test_app/config/database.yml

@@ -16,7 +16,7 @@ test: &test
 #  adapter: postgresql
 #  database: impressionist_test
 #  username: johnmcaliley
-# password: 
+# password:
 #  host: localhost
 #  encoding: UTF8
 
@@ -27,4 +27,4 @@ production:
   timeout: 5000
 
 cucumber:
-  <<: *test
+  <<: *test

test_app/db/migrate/20111127195013_create_impressions_table.rb

@@ -16,22 +16,22 @@ def self.up
     end
     add_index :impressions, [:impressionable_type, :impressionable_id, :request_hash], :name => "poly_request_index", :unique => false
     add_index :impressions, [:impressionable_type, :impressionable_id, :ip_address], :name => "poly_ip_index", :unique => false
-    add_index :impressions, [:impressionable_type, :impressionable_id, :session_hash], :name => "poly_session_index", :unique => false    
+    add_index :impressions, [:impressionable_type, :impressionable_id, :session_hash], :name => "poly_session_index", :unique => false
     add_index :impressions, [:controller_name,:action_name,:request_hash], :name => "controlleraction_request_index", :unique => false
     add_index :impressions, [:controller_name,:action_name,:ip_address], :name => "controlleraction_ip_index", :unique => false
-    add_index :impressions, [:controller_name,:action_name,:session_hash], :name => "controlleraction_session_index", :unique => false        
+    add_index :impressions, [:controller_name,:action_name,:session_hash], :name => "controlleraction_session_index", :unique => false
     add_index :impressions, :user_id
   end
 
   def self.down
     remove_index :impressions, :name => :poly_request_index
     remove_index :impressions, :name => :poly_ip_index
-    remove_index :impressions, :name => :poly_session_index        
+    remove_index :impressions, :name => :poly_session_index
     remove_index :impressions, :name => :controlleraction_request_index
     remove_index :impressions, :name => :controlleraction_ip_index
-    remove_index :impressions, :name => :controlleraction_session_index        
+    remove_index :impressions, :name => :controlleraction_session_index
     remove_index :impressions, :user_id
-    
+
     drop_table :impressions
   end
-end
+end

test_app/lib/tasks/cucumber.rake

@@ -1,6 +1,6 @@
 # IMPORTANT: This file is generated by cucumber-rails - edit at your own peril.
-# It is recommended to regenerate this file in the future when you upgrade to a 
-# newer version of cucumber-rails. Consider adding your own code to a new file 
+# It is recommended to regenerate this file in the future when you upgrade to a
+# newer version of cucumber-rails. Consider adding your own code to a new file
 # instead of editing this one. Cucumber will automatically load all features/**/*.rb
 # files.
 

test_app/public/javascripts/controls.js

@@ -962,4 +962,4 @@ Form.Element.DelayedObserver = Class.create({
     this.timer = null;
     this.callback(this.element, $F(this.element));
   }
-});
+});

test_app/public/javascripts/dragdrop.js

@@ -971,4 +971,4 @@ Element.findChildren = function(element, only, recursive, tagName) {
 
 Element.offsetSize = function (element, type) {
   return element['offset' + ((type=='vertical' || type=='height') ? 'Height' : 'Width')];
-};
+};

test_app/public/javascripts/effects.js

@@ -1120,4 +1120,4 @@ $w('getInlineOpacity forceRerendering setContentZoom collectTextNodes collectTex
   function(f) { Effect.Methods[f] = Element[f]; }
 );
 
-Element.addMethods(Effect.Methods);
+Element.addMethods(Effect.Methods);

test_app/spec/controllers/controller_spec.rb

@@ -95,9 +95,9 @@
     get "show", :id=> 1
     Impression.all.size.should eq 11
   end
-  
+
   describe "impressionist unique options" do
-    
+
     it "should log unique impressions at the per action level" do
       get "show", :id=> 1
       Impression.all.size.should eq 12
@@ -108,7 +108,7 @@
       get "index"
       Impression.all.size.should eq 14
     end
-    
+
     it "should log unique impressions only once per id" do
       get "show", :id=> 1
       Impression.all.size.should eq 12
@@ -119,7 +119,7 @@
       get "index"
       Impression.all.size.should eq 14
     end
-  
+
   end
-  
+
 end

test_app/spec/fixtures/articles.yml

@@ -1,3 +1,3 @@
 one:
   id: 1
-  name: Test Article
+  name: Test Article