diff -r 8060f33a164f -r 38225f2ad3a6 app/soc/views/helper/access.py --- a/app/soc/views/helper/access.py Tue Feb 03 14:19:51 2009 +0000 +++ b/app/soc/views/helper/access.py Tue Feb 03 15:18:40 2009 +0000 @@ -184,12 +184,12 @@ MEMBERSHIP = { 'anyone': 'allow', - 'club_admin': ('checkHasRole', club_admin_logic), - 'club_member': ('checkHasRole', club_member_logic), - 'host': ('checkHasRole', host_logic), - 'org_admin': ('checkHasRole', org_admin_logic), - 'org_mentor': ('checkHasRole', mentor_logic), - 'org_student': 'deny', #('checkHasRole', student_logic), + 'club_admin': ('checkHasActiveRole', club_admin_logic), + 'club_member': ('checkHasActiveRole', club_member_logic), + 'host': ('checkHasActiveRole', host_logic), + 'org_admin': ('checkHasActiveRole', org_admin_logic), + 'org_mentor': ('checkHasActiveRole', mentor_logic), + 'org_student': 'deny', #('checkHasActiveRole', student_logic), 'user': 'checkIsUser', 'user_self': ('checkIsUserSelf', 'scope_path'), } @@ -570,8 +570,8 @@ raise out_of_band.AccessViolation(message_fmt=DEF_NO_ACTIVE_GROUP_MSG) - def checkHasRole(self, django_args, logic, field_name=None): - """Checks that the user has the specified role. + def checkHasActiveRole(self, django_args, logic, field_name=None): + """Checks that the user has the specified active role. """ if not field_name: @@ -674,7 +674,7 @@ self.deny(django_args) new_args = {'scope_path': program.scope_path } - self.checkHasRole(new_args, host_logic) + self.checkHasActiveRole(new_args, host_logic) @allowDeveloper @@ -808,7 +808,7 @@ try: # check if it is my role the user's own role - self.checkHasRole(django_args, role_logic) + self.checkHasActiveRole(django_args, role_logic) except out_of_band.Error: pass