Commit 591bb0fd by Waheed Ahmed

Added course image download link.

ECOM-7554
parent b7e2d3be
...@@ -165,13 +165,13 @@ class CourseRunWrapperTests(TestCase): ...@@ -165,13 +165,13 @@ class CourseRunWrapperTests(TestCase):
self.assertEqual(self.wrapped_course_run.course_team_admin, self.course.course_team_admin) self.assertEqual(self.wrapped_course_run.course_team_admin, self.course.course_team_admin)
def test_course_image(self): def test_course_image(self):
""" Verify that the wrapper return the url fo thumbnail course image if exists. """ """ Verify that the wrapper return the course image. """
self.assertIsNone(self.wrapped_course_run.course_image) self.assertIsNone(self.wrapped_course_run.course_image)
self.course.image = make_image_file('test_banner1.jpg') self.course.image = make_image_file('test_banner1.jpg')
self.course.save() self.course.save()
self.assertEqual(self.wrapped_course_run.course_image, self.course.image.thumbnail.url) self.assertEqual(self.wrapped_course_run.course_image, self.course.image)
def test_course_staff(self): def test_course_staff(self):
"""Verify that the wrapper return staff list.""" """Verify that the wrapper return staff list."""
......
...@@ -181,7 +181,9 @@ class CourseRunWrapper(BaseWrapper): ...@@ -181,7 +181,9 @@ class CourseRunWrapper(BaseWrapper):
@property @property
def course_image(self): def course_image(self):
if self.wrapped_obj.course.image: if self.wrapped_obj.course.image:
return self.wrapped_obj.course.image.thumbnail.url return self.wrapped_obj.course.image
return None
@property @property
def course_staff(self): def course_staff(self):
......
...@@ -638,3 +638,9 @@ ...@@ -638,3 +638,9 @@
#id_select_revisions { #id_select_revisions {
max-width: 50%; max-width: 50%;
} }
.download-image {
text-align: center;
margin-top: 10px;
width: 100%;
}
...@@ -170,7 +170,12 @@ ...@@ -170,7 +170,12 @@
{% if not object.course_image %} {% if not object.course_image %}
{% trans "(Required) Not yet added" %} {% trans "(Required) Not yet added" %}
{% else %} {% else %}
<img src="{{ object.course_image }}" alt=""/> <img class="course-image" src="{{ object.course_image.url }}" alt="{% trans 'Course Image' %}">
<div class="download-image">
<a download href="{{ object.course_image.url }}">
{% trans "Download" %}
</a>
</div>
{% endif %} {% endif %}
</div> </div>
</div> </div>
......
...@@ -185,6 +185,23 @@ ...@@ -185,6 +185,23 @@
</div> </div>
<div class="info-item"> <div class="info-item">
<div class="heading"> <div class="heading">
{% trans "Course Image" %}
</div>
<div>
{% if not object.course_image %}
{% trans "(Required) Not yet added" %}
{% else %}
<img class="course-image" src="{{ object.course_image.url }}" alt="{% trans 'Course Image' %}">
<div class="download-image">
<a download href="{{ object.course_image.url }}">
{% trans "Download" %}
</a>
</div>
{% endif %}
</div>
</div>
<div class="info-item">
<div class="heading">
{% trans "Prerequisite" %} {% trans "Prerequisite" %}
{% include "publisher/course_run_detail/_clipboard.html" %} {% include "publisher/course_run_detail/_clipboard.html" %}
</div> </div>
......
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