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
Next Next commit
Hotfixing find the only budget for emails
  • Loading branch information
@bertocq @javierm
bertocq authored and javierm committed Jun 3, 2019
commit 0d761ddfb1f579b531aa4579f9c061f3cd93c7d0
2 changes: 1 addition & 1 deletion app/controllers/budgets/results_controller.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ def show
private

def load_budget
@budget = Budget.find_by(id: params[:budget_id])
@budget = Budget.find_by(id: params[:budget_id]) || Budget.first
end

def load_heading
Expand Down