Skip to content
Snippets Groups Projects

Fix broken Note scopes with lambdas, 4.0 compat

Merged gitlab-qa-bot requested to merge github/fork/sholden/patch-2 into master

Created by: sholden

Without lambdas, Date.today will be evaluated in the class body. For it to have a running scope of last week etc, it will need to be evaluated each time the scope is called. In Rails 4.0, lambdas will be required for all scopes, so not a bad idea to go ahead and change them all now.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading