cst1/openstreetmap-website
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Better comments in application controller logic

Browse source
This commit is contained in:
Matt Amos 2012-04-04 16:10:45 +01:00 • committed by Tom Hughes
parent eca8f6e849
commit 1f9301e2e2
1 changed files with 5 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

6
app/controllers/application_controller.rb
View file

@ -113,7 +113,7 @@ class ApplicationController < ActionController::Base
## ##
# require that the user is a moderator, or fill out a helpful error message # require that the user is a moderator, or fill out a helpful error message
# and return them to the blocks index. # and return them to the index for the controller this is wrapped from.
def require_moderator def require_moderator
unless @user.moderator? unless @user.moderator?
if request.get? if request.get?
@ -178,6 +178,10 @@ class ApplicationController < ActionController::Base
# to be used as a before_filter *after* authorize. this checks that # to be used as a before_filter *after* authorize. this checks that
# the user is a moderator and, if not, returns a forbidden error. # the user is a moderator and, if not, returns a forbidden error.
# #
# NOTE: this isn't a very good way of doing it - it duplicates logic
# from require_moderator - but what we really need to do is a fairly
# drastic refactoring based on :format and respond_to? but not a
# good idea to do that in this branch.
def authorize_moderator(errormessage="Access restricted to moderators") def authorize_moderator(errormessage="Access restricted to moderators")
# check user is a moderator # check user is a moderator
unless @user.moderator? unless @user.moderator?