Get an object using scope instead of array Ruby on Rails

Question

In my controller i have the following

  def index
   if !params[:place_id].nil? || !params[:place_id].empty? 
    @restaurant = GoogleSearchedLocation.registered(params[:place_id])
   else
      redirect_to customers_url
    end
  end

While in my model I have the following scope defined

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

Output is expected as below i.e in the form of array

< GoogleSearchedLocation id: 1, place_id: "ChIJSeoh6hkEGTkRsd0e1crAbHU", name: "Dunkin' Donuts", rating: 4.3, longitude: nil, latitude: nil, scope: "GOOGLE", created_at: "2015-10-15 10:59:23", updated_at: "2015-10-15 10:59:23">

I tried to convert it into object in my controller function like below

 @restaurant1= @restaurant[0].each_slice().map{|a| GoogleSearchedLocation.new(a)}

But it's throwing error

undefined method `each_slice' for < GoogleSearchedLocation:0x00000005345cb0>

Can anyone explain the reason for error while we all know the each_slice() function . And kindly suggest a reasonable way to convert this kind of array into object

@restaurant.first should give you the first object of ActiveRecord relation — sameera207
– sameera207, Commented Oct 15, 2015 at 12:25
:), just for clarity, I would use restaurants = GoogleSearchedLocation.registered(params[:place_id]) and @restaurant = restaurants.first, or something like that, coz your scope always returns a collection of objects and its better if your variable name to reflect that — sameera207
– sameera207, Commented Oct 15, 2015 at 12:28

nsave · Accepted Answer · 2015-10-15 12:37:42Z

2

You could simply use GoogleSearchedLocation.registered(params[:place_id]).first to take the first found record.

Also note that for such simple queries there are ActiveRecord helpers like find_by_something. Check out this:

@restaurant = GoogleSearchedLocation.find_by_place_id(params[:place_id])

P.S.

You could use params[:place_id].blank? in your condition:

nil.blank?
# => true

''.blank?
# => true 

'12'.blank?
# => false

edited Oct 15, 2015 at 12:37

answered Oct 15, 2015 at 12:27

nsave

9849 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

sameera207 Over a year ago

actually it doesnt need to check at all, coz it always return an ActiveRecord relation even the id send as nil

Collectives™ on Stack Overflow

Get an object using scope instead of array Ruby on Rails

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related