Skip to content

Commit

Permalink
[#2939] Remove Authorizer from user controller
Browse files Browse the repository at this point in the history
  • Loading branch information
tobes committed Oct 9, 2012
1 parent 20d8d7b commit 6302ba1
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions ckan/controllers/user.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import ckan.lib.i18n
from ckan.lib.base import *
from ckan.lib import mailer
from ckan.authz import Authorizer
import ckan.new_authz
from ckan.lib.navl.dictization_functions import DataError, unflatten
from ckan.logic import NotFound, NotAuthorized, ValidationError
from ckan.logic import check_access, get_action
Expand Down Expand Up @@ -49,7 +49,7 @@ def _db_to_edit_form_schema(self):
into a format suitable for the form (optional)'''

def _setup_template_variables(self, context, data_dict):
c.is_sysadmin = Authorizer().is_sysadmin(c.user)
c.is_sysadmin = ckan.new_authz.is_sysadmin(c.user)
try:
user_dict = get_action('user_show')(context, data_dict)
except NotFound:
Expand Down Expand Up @@ -154,7 +154,7 @@ def new(self, data=None, errors=None, error_summary=None):
error_summary = error_summary or {}
vars = {'data': data, 'errors': errors, 'error_summary': error_summary}

c.is_sysadmin = Authorizer().is_sysadmin(c.user)
c.is_sysadmin = ckan.new_authz.is_sysadmin(c.user)
c.form = render(self.new_user_form, extra_vars=vars)
return render('user/new.html')

Expand Down Expand Up @@ -230,7 +230,7 @@ def edit(self, id=None, data=None, errors=None, error_summary=None):

user_obj = context.get('user_obj')

if not (ckan.authz.Authorizer().is_sysadmin(unicode(c.user))
if not (ckan.new_authz.is_sysadmin(c.user)
or c.user == user_obj.name):
abort(401, _('User %s not authorized to edit %s') %
(str(c.user), id))
Expand Down

0 comments on commit 6302ba1

Please sign in to comment.