Merge pull request #31336 from openedx/INF-628

fix: course_staff and course_admin added in get_course api
This commit is contained in:
ayesha waris
2022-12-01 16:35:13 +05:00
committed by GitHub
3 changed files with 11 additions and 0 deletions

View File

@@ -28,6 +28,11 @@ from rest_framework.exceptions import PermissionDenied
from rest_framework.request import Request
from rest_framework.response import Response
from common.djangoapps.student.roles import (
CourseInstructorRole,
CourseStaffRole,
)
from lms.djangoapps.course_blocks.api import get_course_blocks
from lms.djangoapps.courseware.courses import get_course_with_access
from lms.djangoapps.courseware.exceptions import CourseAccessRedirect
@@ -340,6 +345,8 @@ def get_course(request, course_key):
}),
"is_group_ta": bool(user_roles & {FORUM_ROLE_GROUP_MODERATOR}),
"is_user_admin": request.user.is_staff,
"is_course_staff": CourseStaffRole(course_key).has_user(request.user),
"is_course_admin": CourseInstructorRole(course_key).has_user(request.user),
"provider": course_config.provider_type,
"enable_in_context": course_config.enable_in_context,
"group_at_subsection": course_config.plugin_configuration.get("group_at_subsection", False),

View File

@@ -201,6 +201,8 @@ class GetCourseTest(ForumsEnableMixin, UrlResetMixin, SharedModuleStoreTestCase)
'group_at_subsection': False,
'provider': 'legacy',
'has_moderation_privileges': False,
"is_course_staff": False,
"is_course_admin": False,
'is_group_ta': False,
'is_user_admin': False,
'user_roles': {'Student'},

View File

@@ -521,6 +521,8 @@ class CourseViewTest(DiscussionAPIViewTestMixin, ModuleStoreTestCase):
"allow_anonymous": True,
"allow_anonymous_to_peers": False,
"has_moderation_privileges": False,
'is_course_admin': False,
'is_course_staff': False,
"is_group_ta": False,
'is_user_admin': False,
"user_roles": ["Student"],