Default "Course About" image
Templates that display the image now check if courses specifies an image, if they don't the default image is displayed Path set in both common.py and aws.py to allow for easy overriding in one place. Addresses SOL-926 Some code provided by Davorin Sego
This commit is contained in:
@@ -126,10 +126,9 @@ def course_image_url(course):
|
||||
url += '/' + course.course_image
|
||||
else:
|
||||
url += '/images/course_image.jpg'
|
||||
elif course.course_image == '':
|
||||
# if course_image is empty the url will be blank as location
|
||||
# of the course_image does not exist
|
||||
url = ''
|
||||
elif not course.course_image:
|
||||
# if course_image is empty, use the default image url from settings
|
||||
url = settings.STATIC_URL + settings.DEFAULT_COURSE_ABOUT_IMAGE_URL
|
||||
else:
|
||||
loc = StaticContent.compute_location(course.id, course.course_image)
|
||||
url = StaticContent.serialize_asset_key_with_slash(loc)
|
||||
|
||||
@@ -794,16 +794,19 @@ class TestHtmlModifiers(ModuleStoreTestCase):
|
||||
self.assertTrue(url.startswith('/static/toy_course_dir/'))
|
||||
self.course.static_asset_path = ""
|
||||
|
||||
@override_settings(DEFAULT_COURSE_ABOUT_IMAGE_URL='test.png')
|
||||
@override_settings(STATIC_URL='static/')
|
||||
@ddt.data(ModuleStoreEnum.Type.mongo, ModuleStoreEnum.Type.split)
|
||||
def test_course_image_for_split_course(self, store):
|
||||
"""
|
||||
for split courses if course_image is empty then course_image_url will be blank
|
||||
for split courses if course_image is empty then course_image_url will be
|
||||
the default image url defined in settings
|
||||
"""
|
||||
self.course = CourseFactory.create(default_store=store)
|
||||
self.course.course_image = ''
|
||||
|
||||
url = course_image_url(self.course)
|
||||
self.assertEqual('', url)
|
||||
self.assertEqual('static/test.png', url)
|
||||
|
||||
def test_get_course_info_section(self):
|
||||
self.course.static_asset_path = "toy_course_dir"
|
||||
|
||||
Reference in New Issue
Block a user