bullet 4.11.3 → 4.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2ca96bf9ce0ec1f8610db4044ca48bc25ebf8ded
-  data.tar.gz: b2f6067c61713e0c0bb5c868d7f8e5464e09476c
+  metadata.gz: b201e99b1db5ea0ae016eb415b8bdcb2141a0321
+  data.tar.gz: 072c423687b8482273549f30c2538edec5c3c03f
 SHA512:
-  metadata.gz: 4d6ceb5f5ad0c7357a741bbd04b42bd188e38802a7a448aca7e289166570947a9941a224a306634ccd7e4ad99279c2e9dd1692935526dc0ef6b16fbdee26a0c3
-  data.tar.gz: fd48101ec26800797c4cd834905b3f8b1666631a39ac3f42f088430b82611d2b144daefc307fcc0cfffb07d788971b5e4d155269df642e7f2b2b83edde10128b
+  metadata.gz: 624bfcc8a8f2044699d19e598729cba0bf858e054002b435a31cce643d1bc3834139dad14ea510ef573a31059ce13f4b5c1dd892404645cca8d77e47264db794
+  data.tar.gz: ffaf8125f6797fd85b42be44f645e835796d623bc8b656662b8bd7387ac3b92882d4f2daaf2c50c04aced38113d7c96fbd8b3bdf631bd532c0a02663e221e1d1

data/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Next Release
 
-## 4.12.0 (not released yet)
+## 4.12.0
 
-* Use primary_key rather than id for bullet_ar_key
+* Fix false n+1 queries caused by inversed objects.
+* Replace .id with .primary_key_value
 * Rename bullet_ar_key to bullet_key
 * Fix rails sse detect
-* Fix bullet in rails test
+* Fix bullet using in test environment
 * Memoize whoami
 
 ## 4.11.0 (06/24/2014)

data/Gemfile.mongoid-4.0

@@ -5,7 +5,7 @@ gemspec
 gem 'rails', '~> 4.0.5'
 gem 'sqlite3', platforms: [:ruby]
 gem 'activerecord-jdbcsqlite3-adapter', platforms: [:jruby]
-gem 'mongoid', '~> 4.0.0.beta2'
+gem 'mongoid', '~> 4.0.0'
 
 gem "rspec"
 gem "guard"

data/Gemfile.rails-3.2

@@ -2,7 +2,7 @@ source "https://rubygems.org"
 
 gemspec
 
-gem 'rails', '~> 3.2.18'
+gem 'rails', '~> 3.2.19'
 gem 'sqlite3', platforms: [:ruby]
 gem 'activerecord-jdbcsqlite3-adapter', platforms: [:jruby]
 gem 'activerecord-import'

data/Gemfile.rails-4.0

@@ -2,7 +2,7 @@ source "https://rubygems.org"
 
 gemspec
 
-gem 'rails', '~> 4.0.5'
+gem 'rails', '~> 4.0.8'
 gem 'sqlite3', platforms: [:ruby]
 gem 'activerecord-jdbcsqlite3-adapter', platforms: [:jruby]
 gem 'activerecord-import'

data/Gemfile.rails-4.1

@@ -2,7 +2,7 @@ source "https://rubygems.org"
 
 gemspec
 
-gem 'rails', '~> 4.1.1'
+gem 'rails', '~> 4.1.4'
 gem 'sqlite3'
 gem 'activerecord-jdbcsqlite3-adapter', platforms: [:jruby]
 gem 'activerecord-import'

data/README.md

@@ -60,7 +60,7 @@ config.after_initialize do
 end
 ```
 
-The notifier of bullet is a wrap of [uniform_notifier](https://github.com/flyerhzm/uniform_notifier)
+The notifier of Bullet is a wrap of [uniform_notifier](https://github.com/flyerhzm/uniform_notifier)
 
 The code above will enable all seven of the Bullet notification systems:
 * `Bullet.enable`: enable Bullet gem, otherwise do nothing
@@ -70,25 +70,31 @@ The code above will enable all seven of the Bullet notification systems:
 * `Bullet.airbrake`: add notifications to airbrake
 * `Bullet.console`: log warnings to your browser's console.log (Safari/Webkit browsers or Firefox w/Firebug installed)
 * `Bullet.growl`: pop up Growl warnings if your system has Growl installed. Requires a little bit of configuration
-* `Bullet.xmpp`: 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.
+* `Bullet.xmpp`: 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.
 * `Bullet.raise`: raise errors, useful for making your specs fail unless they have optimized queries
 * `Bullet.add_footer`: adds the details in the bottom left corner of the page
 * `Bullet.stacktrace_includes`: include paths with any of these substrings in the stack trace, even if they are not in your main app
 
-Bullet also allows you to disable n_plus_one_query, unused_eager_loading
-and counter_cache detectors respectively.
+Bullet also allows you to disable any of its detectors.
 
 ```ruby
-Bullet.n_plus_one_query_enable = false
+# Each of these settings defaults to true
+
+# Detect N+1 queries
+Bullet.n_plus_one_query_enable     = false
+
+# Detect eager-loaded associations which are not used
 Bullet.unused_eager_loading_enable = false
-Bullet.counter_cache_enable = false
+
+# Detect unnecessary COUNT queries which could be avoided
+# with a counter_cache
+Bullet.counter_cache_enable        = false
 ```
 
 ## Whitelist
 
-Sometimes bullet may notify n plus one query, unused eager loading or
-counter cache you don't care about or they occur in the third party gems
-that you can't fix, you can add whitelist to bullet
+Sometimes Bullet may notify you of query problems you don't care to fix, or
+which come from outside your code. You can whitelist these to ignore them:
 
 ```ruby
 Bullet.add_whitelist :type => :n_plus_one_query, :class_name => "Post", :association => :comments
@@ -136,13 +142,13 @@ see [https://github.com/flyerhzm/uniform_notifier](https://github.com/flyerhzm/u
 
 ## Important
 
-If you find bullet does not work for you, *please disable your browser's cache*.
+If you find Bullet does not work for you, *please disable your browser's cache*.
 
 ## Advanced
 
 ### Profile a job
 
-The bullet gem uses rack middleware to profile requests. If you want to use bullet without an http server, like to profile a job, you can use use profile method and fetch warnings
+The Bullet gem uses rack middleware to profile requests. If you want to use Bullet without an http server, like to profile a job, you can use use profile method and fetch warnings
 
 ```ruby
 Bullet.profile do
@@ -153,7 +159,7 @@ warnings = Bullet.warnings
 
 ### Work with sinatra
 
-Configure and use bullet rack
+Configure and use `Bullet::Rack`
 
 ```ruby
 configure :development do
@@ -165,7 +171,7 @@ end
 
 ### Run in tests
 
-First you need to enable bullet in test environment.
+First you need to enable Bullet in test environment.
 
 ```ruby
 # config/environments/test.rb
@@ -176,7 +182,7 @@ config.after_initialize do
 end
 ```
 
-Then wrap each test in bullet api.
+Then wrap each test in Bullet api.
 
 ```ruby
 # spec/spec_helper.rb
@@ -194,18 +200,18 @@ end
 
 ## Debug Mode
 
-Bullet outputs some details info, to enable debug mode, set DEBUG=true
-env.
+Bullet outputs some details info, to enable debug mode, set
+`BULLET_DEBUG=true` env.
 
 ## Contributors
 
 [https://github.com/flyerhzm/bullet/contributors](https://github.com/flyerhzm/bullet/contributors)
 
-## Step by step example
+## Demo
 
-Bullet is designed to function as you browse through your application in development. It will alert you whenever it encounters N+1 queries or unused eager loading.
+Bullet is designed to function as you browse through your application in development. To see it in action, follow these steps to create, detect, and fix example query problems.
 
-1\. setup test environment
+1\. Create an example application
 
 ```
 $ rails new test_bullet
@@ -215,7 +221,7 @@ $ rails g scaffold comment name:string post_id:integer
 $ bundle exec rake db:migrate
 ```
 
-2\. change `app/model/post.rb` and `app/model/comment.rb`
+2\. Change `app/model/post.rb` and `app/model/comment.rb`
 
 ```ruby
 class Post < ActiveRecord::Base
@@ -227,7 +233,7 @@ class Comment < ActiveRecord::Base
 end
 ```
 
-3\. go to `rails c` and execute
+3\. Go to `rails c` and execute
 
 ```ruby
 post1 = Post.create(:name => 'first')
@@ -238,7 +244,7 @@ post2.comments.create(:name => 'third')
 post2.comments.create(:name => 'fourth')
 ```
 
-4\. change the `app/views/posts/index.html.erb` to produce a N+1 query
+4\. Change the `app/views/posts/index.html.erb` to produce a N+1 query
 
 ```
 <% @posts.each do |post| %>
@@ -252,7 +258,7 @@ post2.comments.create(:name => 'fourth')
 <% end %>
 ```
 
-5\. add bullet gem to `Gemfile`
+5\. Add the `bullet` gem to the `Gemfile`
 
 ```ruby
 gem "bullet"
@@ -264,7 +270,7 @@ And run
 bundle install
 ```
 
-6\. enable the bullet gem in development, add a line to
+6\. enable the Bullet gem in development, add a line to
 `config/environments/development.rb`
 
 ```ruby
@@ -279,13 +285,13 @@ config.after_initialize do
 end
 ```
 
-7\. start server
+7\. Start the server
 
 ```
 $ rails s
 ```
 
-8\. input http://localhost:3000/posts in browser, then you will see a popup alert box says
+8\. Visit `http://localhost:3000/posts` in browser, and you will see a popup alert box that says
 
 ```
 The request has unused preload associations as follows:
@@ -294,7 +300,7 @@ The request has N+1 queries as follows:
 model: Post => associations: [comment]
 ```
 
-which means there is a N+1 query from post object to comments associations.
+which means there is a N+1 query from the Post object to its Comment association.
 
 In the meanwhile, there's a log appended into `log/bullet.log` file
 
@@ -309,7 +315,7 @@ In the meanwhile, there's a log appended into `log/bullet.log` file
   /home/flyerhzm/Downloads/test_bullet/app/controllers/posts_controller.rb:7:in `index'
 ```
 
-The generated SQLs are
+The generated SQL is:
 
 ```
 Post Load (1.0ms)   SELECT * FROM "posts"
@@ -317,8 +323,7 @@ Comment Load (0.4ms)   SELECT * FROM "comments" WHERE ("comments".post_id = 1)
 Comment Load (0.3ms)   SELECT * FROM "comments" WHERE ("comments".post_id = 2)
 ```
 
-
-9\. fix the N+1 query, change `app/controllers/posts_controller.rb` file
+9\. To fix the N+1 query, change `app/controllers/posts_controller.rb` file
 
 ```ruby
 def index
@@ -331,18 +336,18 @@ def index
 end
 ```
 
-10\. refresh http://localhost:3000/posts page, no alert box and no log appended.
+10\. Refresh `http://localhost:3000/posts`. Now there's no alert box and nothing new in the log.
 
-The generated SQLs are
+The generated SQL is:
 
 ```
 Post Load (0.5ms)   SELECT * FROM "posts"
 Comment Load (0.5ms)   SELECT "comments".* FROM "comments" WHERE ("comments".post_id IN (1,2))
 ```
 
-a N+1 query fixed. Cool!
+N+1 query fixed. Cool!
 
-11\. now simulate unused eager loading. Change
+11\. Now simulate unused eager loading. Change
 `app/controllers/posts_controller.rb` and
 `app/views/posts/index.html.erb`
 
@@ -368,7 +373,7 @@ end
 <% end %>
 ```
 
-12\. refresh http://localhost:3000/posts page, then you will see a popup alert box says
+12\. Refresh `http://localhost:3000/posts`, and you will see a popup alert box that says
 
 ```
 The request has unused preload associations as follows:
@@ -377,14 +382,14 @@ The request has N+1 queries as follows:
 None
 ```
 
-In the meanwhile, there's a log appended into `log/bullet.log` file
+Meanwhile, there's a line appended to `log/bullet.log`
 
 ```
 2009-08-25 21:13:22[INFO] Unused preload associations: PATH_INFO: /posts;    model: Post => associations: [comments]·
 Remove from your finder: :include => [:comments]
 ```
 
-13\. simulate counter_cache. Change `app/controllers/posts_controller.rb`
+13\. Simulate counter_cache. Change `app/controllers/posts_controller.rb`
 and `app/views/posts/index.html.erb`
 
 ```ruby
@@ -410,19 +415,18 @@ end
 <% end %>
 ```
 
-14\. refresh http://localhost:3000/posts page, then you will see a popup alert box says
+14\. Refresh `http://localhost:3000/posts`, then you will see a popup alert box that says
 
 ```
 Need counter cache
   Post => [:comments]
 ```
 
-In the meanwhile, there's a log appended into `log/bullet.log` file.
+Meanwhile, there's a line appended to `log/bullet.log`
 
 ```
 2009-09-11 10:07:10[INFO] Need Counter Cache
   Post => [:comments]
 ```
 
-
 Copyright (c) 2009 - 2014 Richard Huang (flyerhzm@gmail.com), released under the MIT license

data/lib/bullet.rb

@@ -96,7 +96,7 @@ module Bullet
     end
 
     def debug(title, message)
-      puts "[Bullet][#{title}] #{message}" if ENV['DEBUG'] == 'true'
+      puts "[Bullet][#{title}] #{message}" if ENV['BULLET_DEBUG'] == 'true'
     end
 
     def start_request
@@ -107,6 +107,7 @@ module Bullet
       Thread.current[:bullet_call_object_associations] = Bullet::Registry::Base.new
       Thread.current[:bullet_possible_objects] = Bullet::Registry::Object.new
       Thread.current[:bullet_impossible_objects] = Bullet::Registry::Object.new
+      Thread.current[:bullet_inversed_objects] = Bullet::Registry::Base.new
       Thread.current[:bullet_eager_loadings] = Bullet::Registry::Association.new
 
       Thread.current[:bullet_counter_possible_objects] ||= Bullet::Registry::Object.new
@@ -118,9 +119,10 @@ module Bullet
       Thread.current[:bullet_notification_collector] = nil
 
       Thread.current[:bullet_object_associations] = nil
+      Thread.current[:bullet_call_object_associations] = nil
       Thread.current[:bullet_possible_objects] = nil
       Thread.current[:bullet_impossible_objects] = nil
-      Thread.current[:bullet_call_object_associations] = nil
+      Thread.current[:bullet_inversed_objects] = nil
       Thread.current[:bullet_eager_loadings] = nil
 
       Thread.current[:bullet_counter_possible_objects] = nil

data/lib/bullet/active_record3.rb

@@ -119,7 +119,7 @@ module Bullet
         alias_method :origin_set_inverse_instance, :set_inverse_instance
         def set_inverse_instance(record, instance)
           if record && we_can_set_the_inverse_on_this?(record)
-            Bullet::Detector::NPlusOneQuery.add_impossible_object(record)
+            Bullet::Detector::NPlusOneQuery.add_inversed_object(record, @reflection.inverse_of.name)
           end
           origin_set_inverse_instance(record, instance)
         end

data/lib/bullet/active_record3x.rb

@@ -108,7 +108,7 @@ module Bullet
         alias_method :origin_set_inverse_instance, :set_inverse_instance
         def set_inverse_instance(record)
           if record && invertible_for?(record)
-            Bullet::Detector::NPlusOneQuery.add_impossible_object(record)
+            Bullet::Detector::NPlusOneQuery.add_inversed_object(record, inverse_reflection_for(record).name)
           end
           origin_set_inverse_instance(record)
         end

data/lib/bullet/active_record4.rb

@@ -98,22 +98,12 @@ module Bullet
         alias_method :origin_reader, :reader
         def reader(force_reload = false)
           result = origin_reader(force_reload)
-          Bullet::Detector::NPlusOneQuery.call_association(@owner, @reflection.name)
+          Bullet::Detector::NPlusOneQuery.call_association(@owner, @reflection.name) unless @inversed
           Bullet::Detector::NPlusOneQuery.add_possible_objects(result)
           result
         end
       end
 
-      ::ActiveRecord::Associations::Association.class_eval do
-        alias_method :origin_set_inverse_instance, :set_inverse_instance
-        def set_inverse_instance(record)
-          if record && invertible_for?(record)
-            Bullet::Detector::NPlusOneQuery.add_impossible_object(record)
-          end
-          origin_set_inverse_instance(record)
-        end
-      end
-
       ::ActiveRecord::Associations::HasManyAssociation.class_eval do
         alias_method :origin_has_cached_counter?, :has_cached_counter?
 

data/lib/bullet/active_record41.rb

@@ -27,6 +27,7 @@ module Bullet
         def preloaders_on(association, records, scope)
           if records.first.class.name !~ /^HABTM_/
             records.each do |record|
+              next unless record
               Bullet::Detector::Association.add_object_associations(record, association)
             end
             Bullet::Detector::UnusedEagerLoading.add_eager_loadings(records, association)

data/lib/bullet/detector/association.rb

@@ -5,7 +5,7 @@ module Bullet
         def add_object_associations(object, associations)
           return unless Bullet.start?
           return if !Bullet.n_plus_one_query_enable? && !Bullet.unused_eager_loading_enable?
-          return unless object.id
+          return unless object.primary_key_value
 
           Bullet.debug("Detector::Association#add_object_associations", "object: #{object.bullet_key}, associations: #{associations}")
           object_associations.add(object.bullet_key, associations)
@@ -14,7 +14,7 @@ module Bullet
         def add_call_object_associations(object, associations)
           return unless Bullet.start?
           return if !Bullet.n_plus_one_query_enable? && !Bullet.unused_eager_loading_enable?
-          return unless object.id
+          return unless object.primary_key_value
 
           Bullet.debug("Detector::Association#add_call_object_associations", "object: #{object.bullet_key}, associations: #{associations}")
           call_object_associations.add(object.bullet_key, associations)
@@ -48,13 +48,19 @@ module Bullet
           # impossible_objects keep the class to objects relationships
           # that the objects may not cause N+1 query.
           # e.g. { Post => ["Post:1", "Post:2"] }
-          # Notice: impossible_objects are not accurate,
           # if find collection returns only one object, then the object is impossible object,
           # impossible_objects are used to avoid treating 1+1 query to N+1 query.
           def impossible_objects
             Thread.current[:bullet_impossible_objects]
           end
 
+          # inversed_objects keeps object relationships
+          # that association is inversed.
+          # e.g. { "Comment:1" => ["post"] }
+          def inversed_objects
+            Thread.current[:bullet_inversed_objects]
+          end
+
           # eager_loadings keep the object relationships
           # that the associations are preloaded by find :include.
           # e.g. { ["Post:1", "Post:2"] => [:comments, :user] }

data/lib/bullet/detector/counter_cache.rb

@@ -5,7 +5,7 @@ module Bullet
         def add_counter_cache(object, associations)
           return unless Bullet.start?
           return unless Bullet.counter_cache_enable?
-          return unless object.id
+          return unless object.primary_key_value
 
           Bullet.debug("Detector::CounterCache#add_counter_cache", "object: #{object.bullet_key}, associations: #{associations}")
           if conditions_met?(object.bullet_key, associations)
@@ -17,7 +17,7 @@ module Bullet
           return unless Bullet.start?
           return unless Bullet.counter_cache_enable?
           objects = Array(object_or_objects)
-          return if objects.map(&:id).compact.empty?
+          return if objects.map(&:primary_key_value).compact.empty?
 
           Bullet.debug("Detector::CounterCache#add_possible_objects", "objects: #{objects.map(&:bullet_key).join(', ')}")
           objects.each { |object| possible_objects.add object.bullet_key }
@@ -26,7 +26,7 @@ module Bullet
         def add_impossible_object(object)
           return unless Bullet.start?
           return unless Bullet.counter_cache_enable?
-          return unless object.id
+          return unless object.primary_key_value
 
           Bullet.debug("Detector::CounterCache#add_impossible_object", "object: #{object.bullet_key}")
           impossible_objects.add object.bullet_key

data/lib/bullet/detector/n_plus_one_query.rb

@@ -10,7 +10,8 @@ module Bullet
         #   if it is, keeps this unpreload associations and caller.
         def call_association(object, associations)
           return unless Bullet.start?
-          return unless object.id
+          return unless object.primary_key_value
+          return if inversed_objects.include?(object.bullet_key, associations)
           add_call_object_associations(object, associations)
 
           Bullet.debug("Detector::NPlusOneQuery#call_association", "object: #{object.bullet_key}, associations: #{associations}")
@@ -24,7 +25,7 @@ module Bullet
           return unless Bullet.start?
           return unless Bullet.n_plus_one_query_enable?
           objects = Array(object_or_objects)
-          return if objects.map(&:id).compact.empty?
+          return if objects.map(&:primary_key_value).compact.empty?
 
           Bullet.debug("Detector::NPlusOneQuery#add_possible_objects", "objects: #{objects.map(&:bullet_key).join(', ')}")
           objects.each { |object| possible_objects.add object.bullet_key }
@@ -33,12 +34,21 @@ module Bullet
         def add_impossible_object(object)
           return unless Bullet.start?
           return unless Bullet.n_plus_one_query_enable?
-          return unless object.id
+          return unless object.primary_key_value
 
           Bullet.debug("Detector::NPlusOneQuery#add_impossible_object", "object: #{object.bullet_key}")
           impossible_objects.add object.bullet_key
         end
 
+        def add_inversed_object(object, association)
+          return unless Bullet.start?
+          return unless Bullet.n_plus_one_query_enable?
+          return unless object.primary_key_value
+
+          Bullet.debug("Detector::NPlusOneQuery#add_inversed_object", "object: #{object.bullet_key}, association: #{association}")
+          inversed_objects.add object.bullet_key, association
+        end
+
         private
           def create_notification(callers, klazz, associations)
             notify_associations = Array(associations) - Bullet.get_whitelist_associations(:n_plus_one_query, klazz)

data/lib/bullet/detector/unused_eager_loading.rb

@@ -22,7 +22,7 @@ module Bullet
         def add_eager_loadings(objects, associations)
           return unless Bullet.start?
           return unless Bullet.unused_eager_loading_enable?
-          return if objects.map(&:id).compact.empty?
+          return if objects.map(&:primary_key_value).compact.empty?
 
           Bullet.debug("Detector::UnusedEagerLoading#add_eager_loadings", "objects: #{objects.map(&:bullet_key).join(', ')}, associations: #{associations}")
           bullet_keys = objects.map(&:bullet_key)

data/lib/bullet/ext/object.rb

@@ -1,9 +1,13 @@
 class Object
   def bullet_key
+    [self.class, self.primary_key_value].join(':')
+  end
+
+  def primary_key_value
     if self.class.respond_to?(:primary_key) && self.class.primary_key
-      "#{self.class}:#{self.send self.class.primary_key}"
+      self.send self.class.primary_key
     else
-      "#{self.class}:#{self.id}"
+      self.id
     end
   end
 end

data/lib/bullet/version.rb

@@ -1,4 +1,4 @@
 # encoding: utf-8
 module Bullet
-  VERSION = "4.11.3"
+  VERSION = "4.12.0"
 end

data/spec/bullet/ext/object_spec.rb

@@ -14,4 +14,18 @@ describe Object do
       end
     end
   end
+
+  context "primary_key_value" do
+    it "should return id" do
+      post = Post.first
+      expect(post.primary_key_value).to eq(post.id)
+    end
+
+    it "should return primary key value" do
+      post = Post.first
+      Post.primary_key = 'name'
+      expect(post.primary_key_value).to eq(post.name)
+      Post.primary_key = 'id'
+    end
+  end
 end

data/spec/integration/active_record3/association_spec.rb

@@ -39,7 +39,7 @@ if !mongoid? && active_record3?
         expect(Bullet::Detector::Association).to be_completely_preloading_associations
       end
 
-      it "should detect non preload comment => post with inverse_of" do
+      it "should not detect non preload comment => post with inverse_of" do
         Post.includes(:comments).each do |post|
           post.comments.each do |comment|
             comment.name

data/spec/models/category.rb

@@ -1,5 +1,5 @@
 class Category < ActiveRecord::Base
-  has_many :posts
+  has_many :posts, inverse_of: :category
   has_many :entries
 
   has_many :submissions

data/spec/models/post.rb

@@ -1,7 +1,7 @@
 class Post < ActiveRecord::Base
   extend Bullet::Dependency
 
-  belongs_to :category
+  belongs_to :category, inverse_of: :posts
   belongs_to :writer
   has_many :comments, inverse_of: :post
 

data/spec/spec_helper.rb

@@ -65,7 +65,7 @@ if active_record?
     end
   end
 
-  if ENV["LOG"]
+  if ENV["BULLET_LOG"]
     require 'logger'
     ActiveRecord::Base.logger = Logger.new(STDOUT)
   end
@@ -96,7 +96,7 @@ if mongoid?
     end
   end
 
-  if ENV["LOG"]
+  if ENV["BULLET_LOG"]
     Mongoid.logger = Logger.new(STDOUT)
     Moped.logger = Logger.new(STDOUT)
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet
 version: !ruby/object:Gem::Version
-  version: 4.11.3
+  version: 4.12.0
 platform: ruby
 authors:
 - Richard Huang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-29 00:00:00.000000000 Z
+date: 2014-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport