CODEWITHSUNDEEP
ruby-on-railsruby
mario
mario

Reputation: 63

Rails model block inside lambda scope error

I want to return custom collection on one of my model scope.. But I don't know why it shows error when I use do end block inside my lambda scope.. I am using rails 4.1.0 and ruby 2.1.2..

Here is my scope code inside my model:

scope :date, -> (from_date, to_date) do
  buff_matches = where match_start_time: from_date..to_date
  matches = {}
  buff_matches.each do |match|
    buff_date = match.match_start_time.to_date.to_s
    matches[buff_date] ||= []
    matches[buff_date] << match
  end
  matches
end

It will show an error on this line: buff_matches.each do |match| with error message : SyntaxError: match.rb:15: syntax error, unexpected keyword_do_block, expecting keyword_end.

But if I change my code to be like this :

scope :date, -> (from_date, to_date) do
  buff_matches = where match_start_time: from_date..to_date
  matches = {}
  buff_matches.each { |match|
    buff_date = match.match_start_time.to_date.to_s
    matches[buff_date] ||= []
    matches[buff_date] << match
  }
  matches
end

It will works fine. I want to use do end syntax since it will look cleaner than using curly brace. Do you have any idea why this error happened?

Upvotes: 6

Views: 571

Answers (1)

Joeyjoejoejr
Joeyjoejoejr

Reputation: 914

It seems like you've hit an edge case. I can't really explain why it fails but this fixes it and uses do..end blocks

scope :date, lambda do |from_date, to_date|
  buff_matches = where match_start_time: from_date..to_date
  matches = {}
  buff_matches.each do |match|
    buff_date = match.match_start_time.to_date.to_s
    matches[buff_date] ||= []
    matches[buff_date] << match
  end
  matches
end

Upvotes: 1

Related Questions