bullet 4.1.1 → 4.2.0

data/.gitignore

@@ -8,4 +8,5 @@ tags
 .bundle
 *.gem
 benchmark_profile*
-/nbproject/private/
+/nbproject/private/
+Gemfile*.lock

data/Gemfile.rails-4-beta

@@ -0,0 +1,19 @@
+# Use `bundle install` in order to install these gems
+# Use `bundle exec rake` in order to run the specs using the bundle
+source "http://rubygems.org"
+
+gemspec
+
+gem 'rails', github: 'rails/rails'
+gem 'journey', github: 'rails/journey'
+gem 'activerecord-deprecated_finders', github: 'rails/activerecord-deprecated_finders'
+gem 'sqlite3'
+gem 'mysql'
+gem 'activerecord-import'
+gem 'mongoid', github: 'slbug/mongoid'
+
+gem "rspec"
+gem "guard"
+gem "guard-rspec"
+
+gem "perftools.rb"

data/README.textile

@@ -1,7 +1,9 @@
-h1. Bullet !https://gemnasium.com/flyerhzm/bullet.png?travis(Dependency Status)!:https://gemnasium.com/flyerhzm/bullet
+h1. Bullet
 
 !https://secure.travis-ci.org/flyerhzm/bullet.png!:http://travis-ci.org/flyerhzm/bullet
 
+!http://api.coderwall.com/flyerhzm/endorsecount.png!:http://coderwall.com/flyerhzm
+
 The Bullet gem is designed to help you increase your application's performance by reducing the number of queries it makes. It will watch your queries while you develop your application and notify you when you should add eager loading (N+1 queries), when you're using eager loading that isn't necessary and when you should use counter cache.
 
 Best practice is to use Bullet in development mode or custom mode (staging, profile, etc.). The last thing you want is your clients getting alerts about how lazy you are.
@@ -10,6 +12,15 @@ The Bullet gem now supports **activerecord** 2.1, 2.2, 2.3, 3.0, 3.1, 3.2 and **
 
 ****************************************************************************
 
+h2. External Introduction
+
+* "http://railscasts.com/episodes/372-bullet":http://railscasts.com/episodes/372-bullet
+* "http://ruby5.envylabs.com/episodes/9-episode-8-september-8-2009":http://ruby5.envylabs.com/episodes/9-episode-8-september-8-2009
+* "http://railslab.newrelic.com/2009/10/23/episode-19-on-the-edge-part-1":http://railslab.newrelic.com/2009/10/23/episode-19-on-the-edge-part-1
+* "http://weblog.rubyonrails.org/2009/10/22/community-highlights":http://weblog.rubyonrails.org/2009/10/22/community-highlights
+
+****************************************************************************
+
 h2. Install
 
 You can install it as a gem:
@@ -39,17 +50,19 @@ config.after_initialize do
                   :receiver => 'your_account@jabber.org',
                   :show_online_status => true }
   Bullet.rails_logger = true
+  Bullet.airbrake = true
   Bullet.disable_browser_cache = true
 end
 </code></pre>
 
 The notifier of bullet is a wrap of "uniform_notifier":https://github.com/flyerhzm/uniform_notifier
 
-The code above will enable all six of the Bullet notification systems:
+The code above will enable all seven of the Bullet notification systems:
 * <code>Bullet.enable</code>: enable Bullet gem, otherwise do nothing
 * <code>Bullet.alert</code>: pop up a JavaScript alert in the browser
 * <code>Bullet.bullet_logger</code>: log to the Bullet log file (Rails.root/log/bullet.log)
 * <code>Bullet.rails_logger</code>: add warnings directly to the Rails log
+* <code>Bullet.airbrake</code>: add notifications to airbrake
 * <code>Bullet.console</code>: log warnings to your browser's console.log (Safari/Webkit browsers or Firefox w/Firebug installed)
 * <code>Bullet.growl</code>: pop up Growl warnings if your system has Growl installed. Requires a little bit of configuration
 * <code>Bullet.xmpp</code>: send XMPP/Jabber notifications to the receiver indicated. Note that the code will currently not handle the adding of contacts, so you will need to make both accounts indicated know each other manually before you will receive any notifications. If you restart the development server frequently, the 'coming online' sound for the bullet account may start to annoy - in this case set :show_online_status to false; you will still get notifications, but the bullet account won't announce it's online status anymore.
@@ -90,32 +103,9 @@ These two lines are notifications that unused eager loadings have been encounter
 
 ****************************************************************************
 
-h2. Growl Support
-
-To get Growl support up-and-running for Bullet, follow the steps below:
-* Install the ruby-growl gem: <code>gem install ruby-growl</code>
-* Open the Growl preference pane in Systems Preferences
-* Click the "Network" tab
-* Make sure both "Listen for incoming notifications" and "Allow remote application registration" are checked. *Note*: If you set a password, you will need to set <code>Bullet.growl = { :password => 'growl password' }</code> in the config file.
-* Restart Growl ("General" tab -> Stop Growl -> Start Growl)
-* Boot up your application. Bullet will automatically send a Growl notification when Growl is turned on. If you do not see it when your application loads, make sure it is enabled in your initializer and double-check the steps above.
-
-****************************************************************************
-
-h2. Ruby 1.9 issue
+h2. Growl, XMPP/Jabber and Airbrake Support
 
-ruby-growl gem has an issue about md5 in ruby 1.9, if you use growl and ruby 1.9, check this gist http://gist.github.com/300184
-
-****************************************************************************
-
-h2. XMPP/Jabber Support
-
-To get XMPP support up-and-running for Bullet, follow the steps below:
-* Install the xmpp4r gem: <code>sudo gem install xmpp4r</code>
-* Make both the bullet and the receipient account add each other as contacts.
-  This will require you to manually log into both accounts, add each other
-  as contact and confirm each others contact request.
-* Boot up your application. Bullet will automatically send an XMPP notification when XMPP is turned on.
+see "https://github.com/flyerhzm/uniform_notifier":https://github.com/flyerhzm/uniform_notifier
 
 ****************************************************************************
 
@@ -158,14 +148,6 @@ Don't forget enabling bullet in test environment.
 
 ****************************************************************************
 
-h2. Links
-
-* "http://weblog.rubyonrails.org/2009/10/22/community-highlights":http://weblog.rubyonrails.org/2009/10/22/community-highlights
-* "http://ruby5.envylabs.com/episodes/9-episode-8-september-8-2009":http://ruby5.envylabs.com/episodes/9-episode-8-september-8-2009
-* "http://railslab.newrelic.com/2009/10/23/episode-19-on-the-edge-part-1":http://railslab.newrelic.com/2009/10/23/episode-19-on-the-edge-part-1
-
-****************************************************************************
-
 h2. Contributors
 
 "https://github.com/flyerhzm/bullet/contributors":https://github.com/flyerhzm/bullet/contributors

data/README_for_rails2.textile

@@ -1,5 +1,9 @@
 h1. Bullet
 
+!https://secure.travis-ci.org/flyerhzm/bullet.png!:http://travis-ci.org/flyerhzm/bullet
+
+!http://api.coderwall.com/flyerhzm/endorsecount.png!:http://coderwall.com/flyerhzm
+
 The Bullet plugin/gem is designed to help you increase your application's performance by reducing the number of queries it makes. It will watch your queries while you develop your application and notify you when you should add eager loading (N+1 queries), when you're using eager loading that isn't necessary and when you should use counter cache.
 
 Best practice is to use Bullet in development mode or custom mode (staging, profile, etc.). The last thing you want is your clients getting alerts about how lazy you are.

data/bullet.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
 
   s.required_rubygems_version = ">= 1.3.6"
 
-  s.add_dependency "uniform_notifier", "~> 1.0.0"
+  s.add_dependency "uniform_notifier"
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

data/lib/bullet/dependency.rb

@@ -24,7 +24,7 @@ module Bullet
                                      'active_record2'
                                    elsif active_record30?
                                      'active_record3'
-                                   elsif active_record31? || active_record32?
+                                   elsif active_record31? || active_record32? || active_record4?
                                      'active_record31'
                                    end
                                  end
@@ -60,6 +60,10 @@ module Bullet
       ::ActiveRecord::VERSION::MAJOR == 3
     end
 
+    def active_record4?
+      ::ActiveRecord::VERSION::MAJOR == 4
+    end
+
     def active_record30?
       active_record3? && ::ActiveRecord::VERSION::MINOR == 0
     end

data/lib/bullet/mongoid24.rb

@@ -11,13 +11,13 @@ module Bullet
 
         def first
           result = origin_first
-          Bullet::Detector::Association.add_impossible_object(result)
+          Bullet::Detector::Association.add_impossible_object(result) if result
           result
         end
 
         def last
           result = origin_last
-          Bullet::Detector::Association.add_impossible_object(result)
+          Bullet::Detector::Association.add_impossible_object(result) if result
           result
         end
 
@@ -45,7 +45,9 @@ module Bullet
         alias_method :origin_set_relation, :set_relation
 
         def set_relation(name, relation)
-          Bullet::Detector::NPlusOneQuery.call_association(self, name)
+          if relation && relation.metadata.macro !~ /embed/
+            Bullet::Detector::NPlusOneQuery.call_association(self, name)
+          end
           origin_set_relation(name, relation)
         end
       end

data/lib/bullet/mongoid3.rb

@@ -10,13 +10,13 @@ module Bullet
 
         def first
           result = origin_first
-          Bullet::Detector::Association.add_impossible_object(result)
+          Bullet::Detector::Association.add_impossible_object(result) if result
           result
         end
 
         def last
           result = origin_last
-          Bullet::Detector::Association.add_impossible_object(result)
+          Bullet::Detector::Association.add_impossible_object(result) if result
           result
         end
 
@@ -44,7 +44,9 @@ module Bullet
         alias_method :origin_set_relation, :set_relation
 
         def set_relation(name, relation)
-          Bullet::Detector::NPlusOneQuery.call_association(self, name)
+          if relation && relation.metadata.macro !~ /embed/
+            Bullet::Detector::NPlusOneQuery.call_association(self, name)
+          end
           origin_set_relation(name, relation)
         end
       end

data/lib/bullet/notification/base.rb

@@ -19,7 +19,12 @@ module Bullet
       end
 
       def whoami
-        "user: " << `whoami`.chomp
+        user = `whoami`
+        if user
+          "user: #{user.chomp}"
+        else
+          ""
+        end
       end
 
       def body_with_caller

data/lib/bullet/rack.rb

@@ -28,7 +28,7 @@ module Bullet
     def empty?(response)
       # response may be ["Not Found"], ["Move Permanently"], etc.
       (response.is_a?(Array) && response.size <= 1) ||
-      !response.body.is_a?(String) || response.body.empty?
+        !response.respond_to?(:body) || response.body.empty?
     end
 
     # if send file?
@@ -37,7 +37,7 @@ module Bullet
     end
 
     def html_request?(headers, response)
-      headers['Content-Type'] && headers['Content-Type'].include?('text/html') && response.body.include?("<html>")
+      headers['Content-Type'] && headers['Content-Type'].include?('text/html') && response.body.include?("<html")
     end
 
     def no_browser_cache(headers)

data/lib/bullet/version.rb

@@ -1,5 +1,4 @@
 # encoding: utf-8
 module Bullet
-  VERSION = "4.1.1"
+  VERSION = "4.2.0"
 end
-

data/lib/bullet.rb

@@ -33,7 +33,7 @@ module Bullet
     attr_accessor :enable, :disable_browser_cache
     attr_reader :notification_collector
 
-    delegate :alert=, :console=, :growl=, :rails_logger=, :xmpp=, :to => UniformNotifier
+    delegate :alert=, :console=, :growl=, :rails_logger=, :xmpp=, :airbrake=, :to => UniformNotifier
 
     DETECTORS = [ Bullet::Detector::NPlusOneQuery,
                   Bullet::Detector::UnusedEagerAssociation,
@@ -59,7 +59,7 @@ module Bullet
     def bullet_logger=(active)
       if active
         bullet_log_file = File.open('log/bullet.log', 'a+')
-        bullet_log_file.sync
+        bullet_log_file.sync = true
         UniformNotifier.customized_logger = bullet_log_file
       end
     end

data/spec/bullet/rack_spec.rb

@@ -22,6 +22,12 @@ module Bullet
         middleware.should be_html_request(headers, response)
       end
 
+      it "should be true if Content-Type is text/html and http body contains html tag with attributes" do
+        headers = {"Content-Type" => "text/html"}
+        response = stub(:body => "<html attr='hello'><head></head><body></body></html>")
+        middleware.should be_html_request(headers, response)
+      end
+
       it "should be false if there is no Content-Type header" do
         headers = {}
         response = stub(:body => "<html><head></head><body></body></html>")

data/spec/integration/mongoid/association_spec.rb

@@ -11,6 +11,20 @@ if mongoid?
       Mongoid::IdentityMap.clear
     end
 
+    context 'embeds_many' do
+      context "posts => users" do
+        it "should detect nothing" do
+          Mongoid::Post.all.each do |post|
+            post.users.map(&:name)
+          end
+          Bullet::Detector::UnusedEagerAssociation.check_unused_preload_associations
+          Bullet::Detector::Association.should_not be_has_unused_preload_associations
+
+          Bullet::Detector::Association.should be_completely_preloading_associations
+        end
+      end
+    end
+
     context 'has_many' do
       context "posts => comments" do
         it "should detect non preload posts => comments" do

data/spec/models/mongoid/post.rb

@@ -4,5 +4,7 @@ class Mongoid::Post
   has_many :comments, :class_name => "Mongoid::Comment"
   belongs_to :category, :class_name => "Mongoid::Category"
 
+  embeds_many :users, :class_name => "Mongoid::User"
+
   scope :preload_comments, lambda { includes(:comments) }
 end

data/spec/models/mongoid/user.rb

@@ -0,0 +1,3 @@
+class Mongoid::User
+  include Mongoid::Document
+end

data/spec/support/mongo_seed.rb

@@ -8,6 +8,10 @@ module Support
       post1a = category1.posts.create(:name => 'like first')
       post2 = category2.posts.create(:name => 'second')
 
+      post1.users << Mongoid::User.create(:name => 'first')
+      post1.users << Mongoid::User.create(:name => 'another')
+      post2.users << Mongoid::User.create(:name => 'second')
+
       comment1 = post1.comments.create(:name => 'first')
       comment2 = post1.comments.create(:name => 'first2')
       comment3 = post1.comments.create(:name => 'first3')

data/test.sh

@@ -2,3 +2,4 @@ bundle && bundle exec rspec spec
 BUNDLE_GEMFILE=Gemfile.rails-3.1.5 bundle && bundle exec rspec spec
 BUNDLE_GEMFILE=Gemfile.rails-3.0.13 bundle && bundle exec rspec spec
 BUNDLE_GEMFILE=Gemfile.rails-2.3.14 bundle && bundle exec rspec spec
+BUNDLE_GEMFILE=Gemfile.rails-4-beta bundle && bundle exec rspec spec

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bullet
 version: !ruby/object:Gem::Version
-  version: 4.1.1
+  version: 4.2.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,19 +9,19 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-06 00:00:00.000000000 Z
+date: 2012-09-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: uniform_notifier
-  requirement: &70259190750800 !ruby/object:Gem::Requirement
+  requirement: &70294547787940 !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70259190750800
+  version_requirements: *70294547787940
 description: A rails plugin to kill N+1 queries and unused eager loading.
 email:
 - flyerhzm@gmail.com
@@ -35,13 +35,10 @@ files:
 - .rvmrc.example
 - .travis.yml
 - Gemfile
-- Gemfile.lock
 - Gemfile.rails-2.3.14
-- Gemfile.rails-2.3.14.lock
 - Gemfile.rails-3.0.13
-- Gemfile.rails-3.0.13.lock
 - Gemfile.rails-3.1.5
-- Gemfile.rails-3.1.5.lock
+- Gemfile.rails-4-beta
 - Guardfile
 - Hacking.textile
 - MIT-LICENSE
@@ -119,6 +116,7 @@ files:
 - spec/models/mongoid/company.rb
 - spec/models/mongoid/entry.rb
 - spec/models/mongoid/post.rb
+- spec/models/mongoid/user.rb
 - spec/models/newspaper.rb
 - spec/models/page.rb
 - spec/models/person.rb
@@ -136,7 +134,6 @@ files:
 - spec/support/rack_double.rb
 - spec/support/sqlite_seed.rb
 - tasks/bullet_tasks.rake
-- test.result
 - test.sh
 homepage: http://github.com/flyerhzm/bullet
 licenses: []
@@ -152,7 +149,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3239988902273692039
+      hash: -1372591245763188534
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -206,6 +203,7 @@ test_files:
 - spec/models/mongoid/company.rb
 - spec/models/mongoid/entry.rb
 - spec/models/mongoid/post.rb
+- spec/models/mongoid/user.rb
 - spec/models/newspaper.rb
 - spec/models/page.rb
 - spec/models/person.rb

data/Gemfile.lock

@@ -1,129 +0,0 @@
-PATH
-  remote: .
-  specs:
-    bullet (4.1.1)
-      uniform_notifier (~> 1.0.0)
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    actionmailer (3.2.5)
-      actionpack (= 3.2.5)
-      mail (~> 2.4.4)
-    actionpack (3.2.5)
-      activemodel (= 3.2.5)
-      activesupport (= 3.2.5)
-      builder (~> 3.0.0)
-      erubis (~> 2.7.0)
-      journey (~> 1.0.1)
-      rack (~> 1.4.0)
-      rack-cache (~> 1.2)
-      rack-test (~> 0.6.1)
-      sprockets (~> 2.1.3)
-    activemodel (3.2.5)
-      activesupport (= 3.2.5)
-      builder (~> 3.0.0)
-    activerecord (3.2.5)
-      activemodel (= 3.2.5)
-      activesupport (= 3.2.5)
-      arel (~> 3.0.2)
-      tzinfo (~> 0.3.29)
-    activerecord-import (0.2.9)
-      activerecord (~> 3.0)
-      activerecord (~> 3.0)
-    activeresource (3.2.5)
-      activemodel (= 3.2.5)
-      activesupport (= 3.2.5)
-    activesupport (3.2.5)
-      i18n (~> 0.6)
-      multi_json (~> 1.0)
-    arel (3.0.2)
-    builder (3.0.0)
-    diff-lcs (1.1.3)
-    erubis (2.7.0)
-    ffi (1.0.11)
-    guard (1.0.1)
-      ffi (>= 0.5.0)
-      thor (~> 0.14.6)
-    guard-rspec (0.6.0)
-      guard (>= 0.10.0)
-    hike (1.2.1)
-    i18n (0.6.0)
-    journey (1.0.3)
-    json (1.7.3)
-    mail (2.4.4)
-      i18n (>= 0.4.0)
-      mime-types (~> 1.16)
-      treetop (~> 1.4.8)
-    mime-types (1.18)
-    mongoid (3.0.0.rc)
-      activemodel (~> 3.1)
-      moped (~> 1.0.0.rc)
-      origin (~> 1.0.0.rc)
-      tzinfo (~> 0.3.22)
-    moped (1.0.0.rc)
-    multi_json (1.3.6)
-    mysql (2.8.1)
-    origin (1.0.0.rc)
-    perftools.rb (2.0.0)
-    polyglot (0.3.3)
-    rack (1.4.1)
-    rack-cache (1.2)
-      rack (>= 0.4)
-    rack-ssl (1.3.2)
-      rack
-    rack-test (0.6.1)
-      rack (>= 1.0)
-    rails (3.2.5)
-      actionmailer (= 3.2.5)
-      actionpack (= 3.2.5)
-      activerecord (= 3.2.5)
-      activeresource (= 3.2.5)
-      activesupport (= 3.2.5)
-      bundler (~> 1.0)
-      railties (= 3.2.5)
-    railties (3.2.5)
-      actionpack (= 3.2.5)
-      activesupport (= 3.2.5)
-      rack-ssl (~> 1.3.2)
-      rake (>= 0.8.7)
-      rdoc (~> 3.4)
-      thor (>= 0.14.6, < 2.0)
-    rake (0.9.2.2)
-    rdoc (3.12)
-      json (~> 1.4)
-    rspec (2.9.0)
-      rspec-core (~> 2.9.0)
-      rspec-expectations (~> 2.9.0)
-      rspec-mocks (~> 2.9.0)
-    rspec-core (2.9.0)
-    rspec-expectations (2.9.0)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.9.0)
-    sprockets (2.1.3)
-      hike (~> 1.2)
-      rack (~> 1.0)
-      tilt (~> 1.1, != 1.3.0)
-    sqlite3 (1.3.5)
-    thor (0.14.6)
-    tilt (1.3.3)
-    treetop (1.4.10)
-      polyglot
-      polyglot (>= 0.3.1)
-    tzinfo (0.3.33)
-    uniform_notifier (1.0.2)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  activerecord-import
-  bullet!
-  guard
-  guard-rspec
-  mongoid (= 3.0.0.rc)
-  mysql
-  perftools.rb
-  rails (= 3.2.5)
-  rspec
-  sqlite3

data/Gemfile.rails-2.3.14.lock

@@ -1,65 +0,0 @@
-PATH
-  remote: .
-  specs:
-    bullet (4.1.1)
-      uniform_notifier (~> 1.0.0)
-
-GEM
-  remote: http://rubygems.org/
-  specs:
-    actionmailer (2.3.14)
-      actionpack (= 2.3.14)
-    actionpack (2.3.14)
-      activesupport (= 2.3.14)
-      rack (~> 1.1.0)
-    activerecord (2.3.14)
-      activesupport (= 2.3.14)
-    activeresource (2.3.14)
-      activesupport (= 2.3.14)
-    activesupport (2.3.14)
-    bson (1.6.2)
-    bson_ext (1.6.2)
-      bson (~> 1.6.2)
-    diff-lcs (1.1.3)
-    ffi (1.0.11)
-    guard (1.0.2)
-      ffi (>= 0.5.0)
-      thor (~> 0.14.6)
-    guard-rspec (0.7.0)
-      guard (>= 0.10.0)
-    mysql (2.8.1)
-    perftools.rb (2.0.0)
-    rack (1.1.3)
-    rails (2.3.14)
-      actionmailer (= 2.3.14)
-      actionpack (= 2.3.14)
-      activerecord (= 2.3.14)
-      activeresource (= 2.3.14)
-      activesupport (= 2.3.14)
-      rake (>= 0.8.3)
-    rake (0.9.2.2)
-    rspec (2.10.0)
-      rspec-core (~> 2.10.0)
-      rspec-expectations (~> 2.10.0)
-      rspec-mocks (~> 2.10.0)
-    rspec-core (2.10.0)
-    rspec-expectations (2.10.0)
-      diff-lcs (~> 1.1.3)
-    rspec-mocks (2.10.1)
-    sqlite3 (1.3.6)
-    thor (0.14.6)
-    uniform_notifier (1.0.2)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  bson_ext
-  bullet!
-  guard
-  guard-rspec
-  mysql
-  perftools.rb
-  rails (= 2.3.14)
-  rspec
-  sqlite3