Merge pull request #600 from edly-io/manan/EDLY-7237

Added request and course exist check to avoid none type errors.
edly-io · Nov 29, 2024 · 557077e · 557077e
2 parents d8f7aaf + 0c836f7
commit 557077e
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/common/djangoapps/student/auth.py b/common/djangoapps/student/auth.py
@@ -118,7 +118,9 @@ def has_studio_write_access(user, course_key):
     :param course_key: a CourseKey
     """
     request = get_current_request()
-    if not is_course_org_same_as_site_org(request.site, course_key, is_studio=True):
+    site = getattr(request, 'site', None)
+
+    if course_key and site and not is_course_org_same_as_site_org(site, course_key, is_studio=True):
         return False
 
     return bool(STUDIO_EDIT_CONTENT & get_user_permissions(user, course_key))
@@ -140,7 +142,9 @@ def has_studio_read_access(user, course_key):
     there is read-only access to content libraries.
     """
     request = get_current_request()
-    if not is_course_org_same_as_site_org(request.site, course_key, is_studio=True):
+    site = getattr(request, 'site', None)
+
+    if course_key and site and not is_course_org_same_as_site_org(site, course_key, is_studio=True):
         return False
 
     return bool(STUDIO_VIEW_CONTENT & get_user_permissions(user, course_key))