Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use known translation if exists to get groupname

Browse Source
This commit is contained in:
Don Mitchell
2013-10-21 15:50:18 -04:00
parent f7c979c219
commit 1561ca917f
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
cms/djangoapps/auth/authz.py
View File

@@ -10,6 +10,7 @@ from django.conf import settings
from xmodule.modulestore import Location from xmodule.modulestore import Location
from xmodule.modulestore.locator import CourseLocator, Locator from xmodule.modulestore.locator import CourseLocator, Locator
from xmodule.modulestore.django import loc_mapper
# define a couple of simple roles, we just need ADMIN and EDITOR now for our purposes # define a couple of simple roles, we just need ADMIN and EDITOR now for our purposes
@@ -35,7 +36,11 @@ def get_course_groupname_for_role(location, role):
if isinstance(location, Location): if isinstance(location, Location):
groupnames.append('{0}_{1}'.format(role, location.course)) groupnames.append('{0}_{1}'.format(role, location.course))
elif isinstance(location, CourseLocator): elif isinstance(location, CourseLocator):
groupnames.append('{0}_{1}'.format(role, location.as_old_location_course_id)) old_location = loc_mapper().translate_locator_to_location(location)
if old_location is None:
groupnames.append('{0}_{1}'.format(role, location.as_old_location_course_id))
else:
groupnames.append('{0}_{1}'.format(role, old_location.course_id))
for groupname in groupnames: for groupname in groupnames:
if Group.objects.filter(name=groupname).exists(): if Group.objects.filter(name=groupname).exists():