ziafazal/api-add-course-image-to-overview: added

course_image_url to course overview api

added video to overview page

lms/djangoapps/api_manager/courses/tests.py

@@ -654,12 +654,14 @@ class CoursesApiTests(ModuleStoreTestCase):
         self.assertEqual(response.status_code, 200)
         self.assertGreater(len(response.data), 0)
         self.assertEqual(response.data['overview_html'], self.overview.data)
+        self.assertIn(self.course.course_image, response.data['course_image_url'])
 
     def test_courses_overview_get_parsed(self):
         test_uri = self.base_courses_uri + '/' + self.test_course_id + '/overview?parse=true'
         response = self.do_get(test_uri)
         self.assertEqual(response.status_code, 200)
         self.assertGreater(len(response.data), 0)
+        self.assertIn(self.course.course_image, response.data['course_image_url'])
         sections = response.data['sections']
         self.assertEqual(len(sections), 5)
         self.assertIsNotNone(self._find_item_by_class(sections, 'about'))

lms/djangoapps/api_manager/courses/views.py

@@ -367,7 +367,10 @@ def _get_course_data(request, course_key, course_descriptor, depth=0):
     base_uri_without_qs = generate_base_uri(request, True)
     if unicode(course_descriptor.id) not in base_uri_without_qs:
         base_uri_without_qs = '{}/{}'.format(base_uri_without_qs, unicode(course_descriptor.id))
-    data['course_image_url'] = course_image_url(course_descriptor)
+    image_url = ''
+    if hasattr(course_descriptor, 'course_image') and course_descriptor.course_image:
+        image_url = course_image_url(course_descriptor)
+    data['course_image_url'] = image_url
     data['resources'] = []
     resource_uri = '{}/content/'.format(base_uri_without_qs)
     data['resources'].append({'uri': resource_uri})
@@ -837,6 +840,11 @@ class CoursesOverview(SecureAPIView):
             response_data['sections'] = _parse_overview_html(existing_content)
         else:
             response_data['overview_html'] = existing_content
+        image_url = ''
+        if hasattr(course_descriptor, 'course_image') and course_descriptor.course_image:
+            image_url = course_image_url(course_descriptor)
+        response_data['course_image_url'] = image_url
+        response_data['course_video'] = get_course_about_section(course_descriptor, 'video')
         return Response(response_data, status=status.HTTP_200_OK)