150

Everywhere on the internet people mention that using the rails default_scope is a bad idea, and the top hits for default_scope on stackoverflow are about how to overwrite it. This feels messed up, and merits an explicit question (I think).

So: why is using the rails default_scope recommended against?

5 Answers 5

214

Problem 1

Lets consider the basic example:

class Post < ActiveRecord::Base
  default_scope { where(published: true) }
end

The motivation to make the default published: true, might be to make sure you have to be explict when wanting to show unpublished (private) posts. So far so good.

2.1.1 :001 > Post.all
  Post Load (0.2ms)  SELECT "posts".* FROM "posts"  WHERE "posts"."published" = 't'

Well this is pretty much what we expect. Now lets try:

2.1.1 :004 > Post.new
 => #<Post id: nil, title: nil, published: true, created_at: nil, updated_at: nil>

And there we have the first big problem with default scope:

=> default_scope will affect your model initialization

In a newly created instance of such a model, the default_scope will be reflected. So while you might have wanted to be sure to not list unpublished posts by chance, you're now creating published ones by default.

Problem 2

Consider a more elaborate example:

class Post < ActiveRecord::Base
  default_scope { where(published: true) }
  belongs_to :user
end 

class User < ActiveRecord::Base
  has_many :posts
end

Lets get the first users posts:

2.1.1 :001 > User.first.posts
  Post Load (0.3ms)  SELECT "posts".* FROM "posts"  WHERE "posts"."published" = 't' AND "posts"."user_id" = ?  [["user_id", 1]]

This looks like expected (make sure to scroll all the way to the right to see the part about the user_id).

Now we want to get the list of all posts - unpublished included - say for the logged in user's view. You'll realise you have to 'overwrite' or 'undo' the effect of default_scope. After a quick google, you'll likely find out about unscoped. See what happens next:

2.1.1 :002 > User.first.posts.unscoped
  Post Load (0.2ms)  SELECT "posts".* FROM "posts"

=> Unscoped removes ALL scopes that might normally apply to your select, including (but not limited to) associations.

There are multiple ways to overwrite the different effects of the default_scope. Getting that right gets complicated very quickly and I would argue not using the default_scope in the first place, would be a safer choice.

17
  • 2
    To pile on: the only time I found default_scope useful is when you absolutely want to eager load some associations by default. default_scope{eager_load([:category, :comments])} . However!!! If you are doing count query on this model like Product.count, it will eager_load associations for all products. And if you have 50K records, your count query just went from 15ms to 500ms, because while all you want is count, your default_scope will left join everything else.
    – konung
    May 12, 2015 at 14:41
  • 21
    To me it seems that problem is with unscoped instead of default_scope in problem #2
    – Alma Alma
    Jan 22, 2016 at 18:11
  • 4
    @CaptainFogetti Indeed. I still think it's a good idea to present the drawbacks of unscoped as a possible disadvantage of default_scope. In most non trivial cases using default_scope will lead to you needing to use unscoped. This is a second degree caveat (in lack of a better term), which is easy to miss when researching a method.
    – wrtsprt
    Jan 23, 2016 at 19:23
  • 2
    When (almost) always you want to use it. A common use case is "deleted" or "inactive" records.
    – B Seven
    Aug 4, 2016 at 15:12
  • 7
    I guess a good use of default_scope is when you want something to be sorted: default_scope { order(:name) }. Aug 25, 2016 at 9:33
20

Another reason to not use default_scope is when you're deleting an instance of a model that has a 1 to many relation with the default_scope model

Consider for example:

    class User < ActiveRecord::Base
      has_many :posts, dependent: :destroy
    end 

    class Post < ActiveRecord::Base
      default_scope { where(published: true) }
      belongs_to :user
    end

Calling user.destroy will delete all the posts that are published, but it won't delete posts that are unpublished. Hence the database will throw a foreign key violation because it contains records that reference the user you want to remove.

10

default_scope is often recommended against because it is sometimes incorrectly used to limit the result set. A good use of default_scope is to order the result set.

I would stay away from using where in default_scope and rather create a scope for that.

1
  • 1
    The second problem "Unscoped removes ALL scopes that might normally apply to your select, including (but not limited to) associations" still exists even if the default_scope only contains order. This behaviour of unscoped is quite unexpected.
    – Zack Xu
    Nov 8, 2017 at 18:32
3

For me is not a bad idea but must be used with caution!. There is a case where I always wanted to hide certain records when a field is set.

  1. Preferably the default_scope must match with the DB default value (e.g: { where(hidden_id: nil) })
  2. When you are totally sure you want to show those records, there is always the unscoped method that will avoid your default_scope

So it will depend and the real needs.

1

I only find default_scope to be useful only in ordering some parameters to be in asc or desc order in all situation. Otherwise I avoid it like plague

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.