diff --git a/CHANGELOG.rst b/CHANGELOG.rst index 9ba3178ac5c0..a29ca70a919b 100644 --- a/CHANGELOG.rst +++ b/CHANGELOG.rst @@ -5,6 +5,9 @@ These are notable changes in edx-platform. This is a rolling list of changes, in roughly chronological order, most recent first. Add your entries at or near the top. Include a label indicating the component affected. +LMS: Users with is_staff=True no longer have the STAFF label appear on +their forum posts. + Blades: Video start and end times now function the same for both YouTube and HTML5 videos. If end time is set, the video can still play until the end, after it pauses on the end time. diff --git a/common/static/coffee/src/discussion/utils.coffee b/common/static/coffee/src/discussion/utils.coffee index dd7caeb03371..193b6d1a6d9b 100644 --- a/common/static/coffee/src/discussion/utils.coffee +++ b/common/static/coffee/src/discussion/utils.coffee @@ -26,7 +26,7 @@ class @DiscussionUtil @loadFlagModerator($("#discussion-container").data("flag-moderator")) @isStaff: (user_id) -> - staff = _.union(@roleIds['Staff'], @roleIds['Moderator'], @roleIds['Administrator']) + staff = _.union(@roleIds['Moderator'], @roleIds['Administrator']) _.include(staff, parseInt(user_id)) @isTA: (user_id) -> diff --git a/lms/djangoapps/django_comment_client/tests/test_utils.py b/lms/djangoapps/django_comment_client/tests/test_utils.py index c27503631926..4ba950ace613 100644 --- a/lms/djangoapps/django_comment_client/tests/test_utils.py +++ b/lms/djangoapps/django_comment_client/tests/test_utils.py @@ -39,6 +39,7 @@ def setUp(self): self.course_id = 'edX/toy/2012_Fall' self.student_role = RoleFactory(name='Student', course_id=self.course_id) self.moderator_role = RoleFactory(name='Moderator', course_id=self.course_id) + self.community_ta_role = RoleFactory(name='Community TA', course_id=self.course_id) self.student1 = UserFactory(username='student', email='student@edx.org') self.student1_enrollment = CourseEnrollmentFactory(user=self.student1) self.student_role.users.add(self.student1) @@ -47,10 +48,14 @@ def setUp(self): self.moderator = UserFactory(username='moderator', email='staff@edx.org', is_staff=True) self.moderator_enrollment = CourseEnrollmentFactory(user=self.moderator) self.moderator_role.users.add(self.moderator) + self.community_ta1 = UserFactory(username='community_ta1', email='community_ta1@edx.org') + self.community_ta_role.users.add(self.community_ta1) + self.community_ta2 = UserFactory(username='community_ta2', email='community_ta2@edx.org') + self.community_ta_role.users.add(self.community_ta2) def test_get_role_ids(self): ret = utils.get_role_ids(self.course_id) - expected = {u'Moderator': [3], u'Student': [1, 2], 'Staff': [3]} + expected = {u'Moderator': [3], u'Community TA': [4, 5]} self.assertEqual(ret, expected) def test_has_forum_access(self): diff --git a/lms/djangoapps/django_comment_client/utils.py b/lms/djangoapps/django_comment_client/utils.py index 31d675c8cf60..527a7d0860c8 100644 --- a/lms/djangoapps/django_comment_client/utils.py +++ b/lms/djangoapps/django_comment_client/utils.py @@ -9,7 +9,7 @@ from django.db import connection from django.http import HttpResponse from django.utils import simplejson -from django_comment_common.models import Role +from django_comment_common.models import Role, FORUM_ROLE_STUDENT from django_comment_client.permissions import check_permissions_by_view import mitxmako @@ -42,12 +42,8 @@ def merge_dict(dic1, dic2): def get_role_ids(course_id): - roles = Role.objects.filter(course_id=course_id) - staff = list(User.objects.filter(is_staff=True).values_list('id', flat=True)) - roles_with_ids = {'Staff': staff} - for role in roles: - roles_with_ids[role.name] = list(role.users.values_list('id', flat=True)) - return roles_with_ids + roles = Role.objects.filter(course_id=course_id).exclude(name=FORUM_ROLE_STUDENT) + return dict([(role.name, list(role.users.values_list('id', flat=True))) for role in roles]) def has_forum_access(uname, course_id, rolename):