Commit b87f54a6 by Chris Dodge

return the display names when listing all courses in a group (program)

parent 489a3ee7
...@@ -342,18 +342,24 @@ def group_courses_list(request, group_id): ...@@ -342,18 +342,24 @@ def group_courses_list(request, group_id):
except ObjectDoesNotExist: except ObjectDoesNotExist:
return Response({}, status.HTTP_404_NOT_FOUND) return Response({}, status.HTTP_404_NOT_FOUND)
store = modulestore()
if request.method == 'GET': if request.method == 'GET':
members = CourseGroupRelationship.objects.filter(group=existing_group) members = CourseGroupRelationship.objects.filter(group=existing_group)
response_data['courses'] = [] response_data['courses'] = []
for member in members: for member in members:
response_data['courses'].append(member.course_id) course = store.get_course(member.course_id)
course_data = {
'course_id': member.course_id,
'display_name': course.display_name
}
response_data['courses'].append(course_data)
response_status = status.HTTP_200_OK response_status = status.HTTP_200_OK
else: else:
course_id = request.DATA['course_id'] course_id = request.DATA['course_id']
base_uri = _generate_base_uri(request) base_uri = _generate_base_uri(request)
response_data['uri'] = '{}/{}'.format(base_uri, course_id) response_data['uri'] = '{}/{}'.format(base_uri, course_id)
store = modulestore()
existing_course = store.get_course(course_id) existing_course = store.get_course(course_id)
if not existing_course: if not existing_course:
......
...@@ -574,7 +574,8 @@ class GroupsApiTests(TestCase): ...@@ -574,7 +574,8 @@ class GroupsApiTests(TestCase):
response = self.do_get(test_uri) response = self.do_get(test_uri)
self.assertEqual(response.status_code, 200) self.assertEqual(response.status_code, 200)
self.assertEqual(len(response.data['courses']), 1) self.assertEqual(len(response.data['courses']), 1)
self.assertEqual(response.data['courses'][0], self.test_course_id) self.assertEqual(response.data['courses'][0]['course_id'], self.test_course_id)
self.assertEqual(response.data['courses'][0]['display_name'], self.course.display_name)
def test_group_courses_list_post_duplicate(self): def test_group_courses_list_post_duplicate(self):
data = {'name': self.test_group_name} data = {'name': self.test_group_name}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment