NPO-Accounting/houdini
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Move the nonprofits#show check for super_admin into a before_action

Browse source
This commit is contained in:
Eric 2020-06-18 17:35:22 -05:00 committed by Eric Schultz
parent 0eaede39d0
commit 75543f8b8f
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/controllers/nonprofits_controller.rb
View file

@ -8,15 +8,11 @@ class NonprofitsController < ApplicationController
helper_method :current_nonprofit_user? helper_method :current_nonprofit_user?
before_action :authenticate_nonprofit_user!, only: %i[dashboard dashboard_metrics dashboard_todos payment_history profile_todos recurring_donation_stats update verify_identity] before_action :authenticate_nonprofit_user!, only: %i[dashboard dashboard_metrics dashboard_todos payment_history profile_todos recurring_donation_stats update verify_identity]
before_action :authenticate_super_admin!, only: [:destroy] before_action :authenticate_super_admin!, if: proc {|c| ( c.action_name == "destroy") || (c.action_name == "show" && !current_nonprofit.published) }
# get /nonprofits/:id # get /nonprofits/:id
# get /:state_code/:city/:name # get /:state_code/:city/:name
def show def show
if !current_nonprofit.published && !current_role?(:super_admin)
block_with_sign_in
return
end
@nonprofit = current_nonprofit @nonprofit = current_nonprofit
@url = Format::Url.concat(root_url, @nonprofit.url) @url = Format::Url.concat(root_url, @nonprofit.url)
@supporters = @nonprofit.supporters.not_deleted @supporters = @nonprofit.supporters.not_deleted