Skip to content

Commit 7ad67d0

Browse files
refactor: address PR reviews to inlcude only org/course wide access
1 parent 666629f commit 7ad67d0

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

openedx_authz/engine/utils.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77
import logging
88
from collections import defaultdict
99

10+
from django.db.models import Q
11+
1012
from casbin import Enforcer
1113

1214
from openedx_authz.api.data import CourseOverviewData, OrgCourseOverviewGlobData
@@ -228,7 +230,10 @@ def migrate_legacy_course_roles_to_authz(course_access_role_model, course_id_lis
228230
course_access_role_filter["course_id__in"] = course_id_list
229231

230232
legacy_permissions = (
231-
course_access_role_model.objects.filter(**course_access_role_filter).select_related("user").all()
233+
course_access_role_model.objects.filter(**course_access_role_filter)
234+
.filter(Q(course_id="") | Q(course_id__startswith=CourseOverviewData.NAMESPACE))
235+
.select_related("user")
236+
.all()
232237
)
233238

234239
# List to keep track of any permissions that could not be migrated

0 commit comments

Comments
 (0)