Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use find instead of find by #3580

Merged
merged 5 commits into from
Jun 5, 2019
Merged

Use find instead of find by #3580

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
0d761dd
Hotfixing find the only budget for emails
bertocq Jul 13, 2017
998b4d9
Load budgets using slugs
voodoorai2000 Jan 15, 2017
4bd20ee
Use correct param in controller
microweb10 Jan 17, 2019
22076dd
Add method find_by_slug_or_id! to Sluggable module
microweb10 Jan 17, 2019
b122302
Use find instead of find_by_id
microweb10 Jan 17, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
Add method find_by_slug_or_id! to Sluggable module 
Make it easier to find by slug or id for sluggable models. It will
raise a 404 HTML Not found error if the resource is not found.
  • Loading branch information
@microweb10 @javierm
microweb10 authored and javierm committed Jun 3, 2019
commit 22076dd95c1dfab462119bb028f11e116e9a6b1b
4 changes: 4 additions & 0 deletions app/models/concerns/sluggable.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ module Sluggable
def self.find_by_slug_or_id(slug_or_id)
find_by_slug(slug_or_id) || find_by_id(slug_or_id)
end

def self.find_by_slug_or_id!(slug_or_id)
find_by_slug(slug_or_id) || find(slug_or_id)
end
end

def generate_slug
Expand Down