Argument Error: The scope body needs to be callable

Question

I\'m working through the \'Ruby On Rails 3 Essential Training\' and have received a problem when using name scopes. When finding records and using queries withing the Rails

Answer 1

The scope's body needs to be wrapped in something callable like a Proc or Lambda:

scope :visible, -> {
  where(:visible => true)
}

The reason for this is that it ensures the contents of the block is evaluated each time the scope is used.

Answer 2

Yes, indeed, this is rails 4 way of calling scopes. You'd need to change it, if you're upgrading to Rails 4 from Rails 3.

Answer 3

What you are using: scope :visible, where(:visible => true) goes for eager loading, and has been deprecated in Rails 4.

scope :visible, where(:visible => true)

This line of code gets evaluated when the particular class is loaded, and not at the time, this very scope is called.

There are a few cases when this thing does matter, like:

scope :future, where('published_at > ?', Time.now)
scope :future, -> { where('published_at > ?', Time.now) }

In first case, ? will be replaced with the very time the class would have been loaded, but the second & correct case, that time will be used at which the scope would have been called on the class.

Answer 4

I got the same error , while before my solution I had a space between where and ( like below

scope :registered , -> { where ( place_id:  :place_id , is_registered: :true ) }

after i removed the space between where and ( like below i made my page working

scope :registered , -> { where( place_id:  :place_id , is_registered: :true ) }